Skip to content

fix: increase e2e test step timeout on Windows to fix flaky CI#252

Merged
branchseer merged 2 commits intomainfrom
claude/fix-windows-flaky-tests-885OI
Mar 15, 2026
Merged

fix: increase e2e test step timeout on Windows to fix flaky CI#252
branchseer merged 2 commits intomainfrom
claude/fix-windows-flaky-tests-885OI

Conversation

@branchseer
Copy link
Member

Summary

  • Increases the per-step timeout for e2e snapshot tests from 20s to 60s on Windows, while keeping 20s on Unix
  • Windows CI runners using Git Bash have significantly more overhead for shell startup and process execution
  • The 20-second timeout was frequently exceeded, causing flaky [timeout] failures in tests like summary-output and filter-unmatched

Failure patterns observed

Pattern Tests affected Frequency
[timeout] on first step summary-output, filter-unmatched Most common (3+ occurrences)
[127] command not found exit-codes Occasional

Test plan

  • Compiles locally (cargo check -p vite_task_bin --test e2e_snapshots)
  • Validate by re-running Windows CI at least 10 times to confirm flakiness is resolved

https://claude.ai/code/session_01VzFJkUBQ9t6zTmH7ZJFXgt

claude added 2 commits March 15, 2026 09:29
Windows CI runners with Git Bash have significantly more overhead for
shell startup and process execution. The 20-second per-step timeout was
frequently exceeded on Windows, causing flaky failures in tests like
"multi task with cache hits shows compact summary" where `[timeout]`
appeared instead of actual output. Increase to 60 seconds on Windows
while keeping 20 seconds on Unix.

https://claude.ai/code/session_01VzFJkUBQ9t6zTmH7ZJFXgt
@branchseer branchseer merged commit 1d7128e into main Mar 15, 2026
8 checks passed
@branchseer branchseer deleted the claude/fix-windows-flaky-tests-885OI branch March 15, 2026 10:43
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.

2 participants