Skip to content

test: add per-file timeout and reduce CI job timeout#32840

Merged
bartlomieju merged 1 commit intodenoland:mainfrom
bartlomieju:fix/wpt-timeout
Mar 19, 2026
Merged

test: add per-file timeout and reduce CI job timeout#32840
bartlomieju merged 1 commit intodenoland:mainfrom
bartlomieju:fix/wpt-timeout

Conversation

@bartlomieju
Copy link
Member

Summary

  • Add a 2-minute per-file timeout wrapping the entire runSingleTest (including generateBundle fetch calls) using Promise.race, so the runner always moves on even if the WPT server becomes unresponsive
  • Reduce WPT CI job timeout from 240 minutes to 30 minutes

Test plan

  • WPT CI job completes without hanging

🤖 Generated with Claude Code

The WPT runner could hang indefinitely when generateBundle's fetch
to the WPT server becomes unresponsive. Wrap runSingleTest in a
2-minute Promise.race so the runner always moves on.

Also reduce the CI job timeout from 240m to 30m.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@bartlomieju bartlomieju changed the title fix(wpt): add per-file timeout and reduce CI job timeout test: add per-file timeout and reduce CI job timeout Mar 19, 2026
@bartlomieju bartlomieju merged commit 8ab268c into denoland:main Mar 19, 2026
113 checks passed
@bartlomieju bartlomieju deleted the fix/wpt-timeout branch March 19, 2026 18:57
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