Skip to content

Commit a21e0ed

Browse files
committed
fix: increase e2e test step timeout on Windows to fix flaky CI
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
1 parent 665d553 commit a21e0ed

1 file changed

Lines changed: 7 additions & 2 deletions

File tree

  • crates/vite_task_bin/tests/e2e_snapshots

crates/vite_task_bin/tests/e2e_snapshots/main.rs

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,13 @@ use vite_path::{AbsolutePath, AbsolutePathBuf, RelativePathBuf};
1717
use vite_str::Str;
1818
use vite_workspace::find_workspace_root;
1919

20-
/// Timeout for each step in e2e tests
21-
const STEP_TIMEOUT: Duration = Duration::from_secs(20);
20+
/// Timeout for each step in e2e tests.
21+
/// Windows CI needs a longer timeout due to Git Bash startup overhead and slower I/O.
22+
const STEP_TIMEOUT: Duration = if cfg!(windows) {
23+
Duration::from_secs(60)
24+
} else {
25+
Duration::from_secs(20)
26+
};
2227

2328
/// Screen size for the PTY terminal. Large enough to avoid line wrapping.
2429
const SCREEN_SIZE: ScreenSize = ScreenSize { rows: 500, cols: 500 };

0 commit comments

Comments
 (0)