From fdb5981c133bac3e5738221a5aaef7f920b4bb63 Mon Sep 17 00:00:00 2001 From: xli-oai Date: Fri, 15 May 2026 13:12:42 -0700 Subject: [PATCH] Revert unrelated turn_start test change --- codex-rs/app-server/tests/suite/v2/turn_start.rs | 14 ++++---------- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/codex-rs/app-server/tests/suite/v2/turn_start.rs b/codex-rs/app-server/tests/suite/v2/turn_start.rs index c05e1febb5..314be8a6ff 100644 --- a/codex-rs/app-server/tests/suite/v2/turn_start.rs +++ b/codex-rs/app-server/tests/suite/v2/turn_start.rs @@ -330,16 +330,10 @@ async fn turn_start_emits_thread_scoped_warning_notification_for_trimmed_skills( let warning: WarningNotification = serde_json::from_value(params).expect("deserialize warning notification"); assert_eq!(warning.thread_id.as_deref(), Some(thread.id.as_str())); - let warning_prefix = - "Exceeded skills context budget of 2%. All skill descriptions were removed and "; - let warning_suffix = " additional skills were not included in the model-visible skills list."; - let omitted_skill_count = warning - .message - .strip_prefix(warning_prefix) - .and_then(|message| message.strip_suffix(warning_suffix)) - .expect("warning should include omitted skill count") - .parse::()?; - assert!(omitted_skill_count > 0); + assert_eq!( + warning.message, + "Exceeded skills context budget of 2%. All skill descriptions were removed and 7 additional skills were not included in the model-visible skills list." + ); timeout( DEFAULT_READ_TIMEOUT,