From ca38b19e4dd7ab505e381da86e99a9605c6b61e5 Mon Sep 17 00:00:00 2001 From: Rohit Arunachalam Date: Thu, 25 Jun 2026 03:27:00 -0700 Subject: [PATCH] Assert sleep blocks the next clock read --- .../app-server/tests/suite/v2/current_time.rs | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/codex-rs/app-server/tests/suite/v2/current_time.rs b/codex-rs/app-server/tests/suite/v2/current_time.rs index d62ca3b11f..627b6f073e 100644 --- a/codex-rs/app-server/tests/suite/v2/current_time.rs +++ b/codex-rs/app-server/tests/suite/v2/current_time.rs @@ -161,14 +161,16 @@ async fn external_sleep_notification_waits_for_wake_and_ignores_stale_wakes() -> assert!(!params.sleep_id.is_empty()); let sleep_id = params.sleep_id; + // The second currentTime/read marks the next model-request boundary, so reaching it here + // would mean clock.sleep returned before currentTime/wake released the turn. + let second_read_before_wake = timeout( + Duration::from_millis(100), + app_server.read_stream_until_request_message(), + ) + .await; assert!( - timeout( - Duration::from_millis(100), - app_server.read_stream_until_notification_message("turn/completed"), - ) - .await - .is_err(), - "turn completed before currentTime/wake" + second_read_before_wake.is_err(), + "second currentTime/read arrived before currentTime/wake" ); let wake_params = CurrentTimeWakeParams {