Skip to content

Commit 230a045

Browse files
authored
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.
1 parent 3389465 commit 230a045

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

codex-rs/core/tests/suite/tool_parallelism.rs

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -301,8 +301,11 @@ async fn shell_tools_start_before_response_completed_when_stream_delayed() -> an
301301
"perl -MTime::HiRes -e 'print int(Time::HiRes::time()*1000), \"\\n\"' >> \"{}\"",
302302
output_path.display()
303303
);
304+
// Use a non-login shell to avoid slow, user-specific shell init (e.g. zsh profiles)
305+
// from making this timing-based test flaky.
304306
let args = json!({
305307
"command": command,
308+
"login": false,
306309
"timeout_ms": 1_000,
307310
});
308311

0 commit comments

Comments
 (0)