test: add coverage for generate_animated_card and /animated endpoints #189

New Issue

cal · 2026-04-07T03:21:49Z

cal commented

2026-04-07 03:21:49 +00:00

From PR #188 review feedback — the generate_animated_card function and /animated endpoints currently lack test coverage.

Scope:

Unit tests for generate_animated_card() (mock Playwright, verify frame capture logic)
Integration tests for the /animated API endpoints (mock render pipeline, verify response shape)
Edge cases: missing card, invalid parameters, render failures

Context: These were added as part of the card rendering pipeline but shipped without tests. Adding coverage will catch regressions as the pipeline evolves (e.g., APNG effects in database#186).

From PR #188 review feedback — the `generate_animated_card` function and `/animated` endpoints currently lack test coverage. **Scope:** - Unit tests for `generate_animated_card()` (mock Playwright, verify frame capture logic) - Integration tests for the `/animated` API endpoints (mock render pipeline, verify response shape) - Edge cases: missing card, invalid parameters, render failures **Context:** These were added as part of the card rendering pipeline but shipped without tests. Adding coverage will catch regressions as the pipeline evolves (e.g., APNG effects in database#186).

Claude added the

ai-working

label 2026-04-07 04:01:07 +00:00

Sign in to join this conversation.

No Milestone

No project

No Assignees

1 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: cal/paper-dynasty-database#189