From 1173d2ef939a8281e125d581c5045daee4e90bd4 Mon Sep 17 00:00:00 2001 From: Dylan Hurd Date: Sun, 1 Feb 2026 20:29:49 -0800 Subject: [PATCH] Increase Windows shell test timeout budget --- codex-rs/core/tests/suite/shell_command.rs | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/codex-rs/core/tests/suite/shell_command.rs b/codex-rs/core/tests/suite/shell_command.rs index 1df65056da..5d67180f94 100644 --- a/codex-rs/core/tests/suite/shell_command.rs +++ b/codex-rs/core/tests/suite/shell_command.rs @@ -20,6 +20,10 @@ use test_case::test_case; /// Use this timeout if, empirically, a test seems to need more time than the /// default. const MEDIUM_TIMEOUT: Duration = Duration::from_secs(5); +#[cfg(target_os = "windows")] +const DEFAULT_SHELL_TIMEOUT_MS: i64 = 5_000; +#[cfg(not(target_os = "windows"))] +const DEFAULT_SHELL_TIMEOUT_MS: i64 = 2_000; fn shell_responses_with_timeout( call_id: &str, @@ -50,7 +54,7 @@ fn shell_responses_with_timeout( } fn shell_responses(call_id: &str, command: &str, login: Option) -> Vec { - shell_responses_with_timeout(call_id, command, login, 2_000) + shell_responses_with_timeout(call_id, command, login, DEFAULT_SHELL_TIMEOUT_MS) } async fn shell_command_harness_with(