From 16648c8d1c120fb3d9a8fc54bee9ae60ee778bd3 Mon Sep 17 00:00:00 2001 From: starr-openai Date: Wed, 6 May 2026 21:32:20 -0700 Subject: [PATCH] Make realtime sideband failure test deterministic Use the existing mock server as the sideband failure endpoint instead of relying on an OS-level connection refusal from 127.0.0.1:1. Disable retries in this failure-path test so Windows CI does not spend the default retry budget before emitting the expected error/close events. Co-authored-by: Codex --- codex-rs/core/tests/suite/realtime_conversation.rs | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/codex-rs/core/tests/suite/realtime_conversation.rs b/codex-rs/core/tests/suite/realtime_conversation.rs index ff273a77c4..f22d82ad7c 100644 --- a/codex-rs/core/tests/suite/realtime_conversation.rs +++ b/codex-rs/core/tests/suite/realtime_conversation.rs @@ -749,11 +749,13 @@ async fn conversation_webrtc_sideband_connect_failure_closes_with_error() -> Res ) .mount(&server) .await; - let mut builder = test_codex().with_config(|config| { + let realtime_base_url = server.uri(); + let mut builder = test_codex().with_config(move |config| { config.experimental_realtime_ws_backend_prompt = Some("backend prompt".to_string()); config.experimental_realtime_ws_model = Some("realtime-test-model".to_string()); config.experimental_realtime_ws_startup_context = Some(String::new()); - config.experimental_realtime_ws_base_url = Some("http://127.0.0.1:1".to_string()); + config.experimental_realtime_ws_base_url = Some(realtime_base_url); + config.model_provider.request_max_retries = Some(0); config.realtime.version = RealtimeWsVersion::V1; }); let test = builder.build(&server).await?;