Skip to content

test: add AI metadata extraction integration tests (#52) - #40

Merged
kelleyglenn merged 4 commits into
masterfrom
feature/52-ai-extraction-tests
Feb 22, 2026
Merged

test: add AI metadata extraction integration tests (#52)#40
kelleyglenn merged 4 commits into
masterfrom
feature/52-ai-extraction-tests

Conversation

@kelleyglenn

Copy link
Copy Markdown
Owner

Summary

  • Add API integration tests for the /videos/extract endpoint (auth, validation, real extraction shape)
  • Add E2E browser tests for the Auto-fill with AI UX flow (button visibility, preview trigger, extraction completion)

Depends on:

Closes #52

Test plan

  • npm run test:api passes — extraction boundary tests (401/403, 400, 422) and real extraction shape validation
  • npm run test:e2e passes — Auto-fill button visibility, preview trigger, extraction completion across Chromium/Firefox/WebKit

🤖 Generated with Claude Code

kelleyglenn and others added 4 commits February 21, 2026 14:00
Add API boundary tests for /videos/extract endpoint (auth, validation,
error handling) and E2E browser tests for the Auto-fill with AI button
in the video submission form. Tests require ANTHROPIC_API_KEY to be
configured with a funded account.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
getByText('AI suggestions applied') matched both the toast and the
review banner, causing strict mode violations. Use period suffix to
target the toast specifically.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The video-extract integration test fails because the CI workflow
doesn't pass ANTHROPIC_API_KEY to docker compose, so video-service
can't call the Claude API for metadata extraction.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Include status code and response body in assertion messages for the
extract and auto-approve tests so CI failures show the actual error.
Add a step to dump backend service logs on failure.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@kelleyglenn
kelleyglenn merged commit 933fdde into master Feb 22, 2026
3 of 4 checks passed
@kelleyglenn
kelleyglenn deleted the feature/52-ai-extraction-tests branch February 22, 2026 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant