From 2c2d99f0e3794f3cc4807b21b8dff8c424883ae1 Mon Sep 17 00:00:00 2001 From: Andrew Ambrosino Date: Tue, 10 Nov 2026 19:00:57 +0000 Subject: [PATCH] Update the safety consent test for the current dialog (#60645) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Pass `can_save: true` when opening the consent dialog and assert that the popup displays “Include logs and session history?”. GitOrigin-RevId: d6035a9a1b3e5114a6e92a6fd31547f02c4980fc --- codex-rs/tui/src/chatwidget/tests/status_and_layout.rs | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/codex-rs/tui/src/chatwidget/tests/status_and_layout.rs b/codex-rs/tui/src/chatwidget/tests/status_and_layout.rs index 9cc90c4c12..15498ec20e 100644 --- a/codex-rs/tui/src/chatwidget/tests/status_and_layout.rs +++ b/codex-rs/tui/src/chatwidget/tests/status_and_layout.rs @@ -243,8 +243,11 @@ async fn daybreak_refusal_offers_enable_for_the_next_turn() { } )) ); - chat.open_feedback_consent(crate::app_event::FeedbackCategory::SafetyCheck); - assert!(render_bottom_popup(&chat, /*width*/ 80).contains("Upload logs?")); + chat.open_feedback_consent( + crate::app_event::FeedbackCategory::SafetyCheck, + /*can_save*/ true, + ); + assert!(render_bottom_popup(&chat, /*width*/ 80).contains("Include logs and session history?")); chat.handle_key_event(KeyCode::Esc.into()); assert!(render_bottom_popup(&chat, /*width*/ 80).contains("1. Overview")); chat.handle_key_event(KeyCode::Char('4').into());