From 230a045ac9f7dd3e8209ea059e5d539d1119759e Mon Sep 17 00:00:00 2001 From: Thibault Sottiaux Date: Wed, 7 Jan 2026 01:26:47 -0800 Subject: [PATCH] chore: stabilize core tool parallelism test (#8805) Set login=false for the shell tool in the timing-based parallelism test so it does not depend on slow user login shells, making the test deterministic without user-facing changes. This prevents occasional flakes when running locally. --- codex-rs/core/tests/suite/tool_parallelism.rs | 3 +++ 1 file changed, 3 insertions(+) diff --git a/codex-rs/core/tests/suite/tool_parallelism.rs b/codex-rs/core/tests/suite/tool_parallelism.rs index 5a5b44320..71a6031fe 100644 --- a/codex-rs/core/tests/suite/tool_parallelism.rs +++ b/codex-rs/core/tests/suite/tool_parallelism.rs @@ -301,8 +301,11 @@ async fn shell_tools_start_before_response_completed_when_stream_delayed() -> an "perl -MTime::HiRes -e 'print int(Time::HiRes::time()*1000), \"\\n\"' >> \"{}\"", output_path.display() ); + // Use a non-login shell to avoid slow, user-specific shell init (e.g. zsh profiles) + // from making this timing-based test flaky. let args = json!({ "command": command, + "login": false, "timeout_ms": 1_000, });