Skip to content

test: stabilize YouTube client CI timeout - #65

Merged
devhims merged 1 commit into
mainfrom
fix/cloudflare-youtube-client-timeout
Sep 20, 2026
Merged

devhims merged 1 commit into
mainfrom
fix/cloudflare-youtube-client-timeout

Conversation

@devhims

@devhims devhims commented Sep 20, 2026

Copy link
Copy Markdown
Owner

Problem / Motivation

The production Worker build after PR #63 failed because one YouTube client normalization test exceeded Vitest's generic 5-second timeout. The same test passes consistently when run alone.

Why it matters

Cloudflare stops the deployment when any test times out, even when the application code and the test assertions are correct.

What changed

The one load-sensitive unavailable-video normalization test now has a 15-second timeout. It intentionally parses an unavailable response across every player profile before returning the first response. Production code and test assertions are unchanged.

Tests

  • The exact test passed 20 out of 20 focused runs before the change.
  • All 25 tests in test/youtube-client.test.ts passed after the change.
  • The complete file finished in 224 ms on the hotfix branch.

Manual verification

Merge this PR and confirm the production Cloudflare Worker build completes.

Screenshots / video

Not applicable. This is a test-runner timeout change.

Issue link

no linked issue: fixes the failed production deployment after PR #63.

@vercel

vercel Bot commented Sep 20, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
video2ctx-web Ready Ready Preview Sep 20, 2026 3:39pm UTC

@devhims

devhims commented Sep 20, 2026

Copy link
Copy Markdown
Owner Author

Intent: Stop a load-sensitive YouTube client test from blocking production Worker deployments while keeping the same assertions and production behavior.
Not a goal: Change YouTube client fallback behavior or increase timeout limits for other tests.

@devhims
devhims merged commit ef30553 into main Sep 20, 2026
6 checks passed
@devhims
devhims deleted the fix/cloudflare-youtube-client-timeout branch September 20, 2026 15:51
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