codex: fix CI failure on PR #14506

Bazel was failing the BTW app tests before they reached any BTW logic because the test helper tried to probe a real terminal on stdout. Add a test-only TUI constructor with synthetic terminal state and route the BTW app tests through it so they no longer depend on an interactive terminal.

Co-authored-by: Codex <noreply@openai.com>

Author: charley-oai

codex-rs/tui/src/app.rs

@@ -7049,11 +7049,7 @@ smart_approvals = true
     }
 
     fn make_test_tui() -> crate::tui::Tui {
-        let terminal = crate::custom_terminal::Terminal::with_options(
-            ratatui::backend::CrosstermBackend::new(std::io::stdout()),
-        )
-        .expect("terminal");
-        crate::tui::Tui::new(terminal)
+        crate::tui::Tui::new_test()
     }
 
     fn app_enabled_in_effective_config(config: &Config, app_id: &str) -> Option<bool> {

codex-rs/tui/src/custom_terminal.rs

@@ -201,6 +201,20 @@ where
         })
     }
 
+    #[cfg(test)]
+    pub fn with_test_options(backend: B, screen_size: Size, cursor_pos: Position) -> Self {
+        Self {
+            backend,
+            buffers: [Buffer::empty(Rect::ZERO), Buffer::empty(Rect::ZERO)],
+            current: 0,
+            hidden_cursor: false,
+            viewport_area: Rect::ZERO,
+            last_known_screen_size: screen_size,
+            last_known_cursor_pos: cursor_pos,
+            visible_history_rows: 0,
+        }
+    }
+
     /// Get a Frame object which provides a consistent view into the terminal state for rendering.
     pub fn get_frame(&mut self) -> Frame<'_> {
         Frame {

codex-rs/tui/src/tui.rs

@@ -286,6 +286,36 @@ impl Tui {
         }
     }
 
+    #[cfg(test)]
+    pub(crate) fn new_test() -> Self {
+        let terminal = CustomTerminal::with_test_options(
+            CrosstermBackend::new(stdout()),
+            ratatui::layout::Size {
+                width: 80,
+                height: 24,
+            },
+            ratatui::layout::Position { x: 0, y: 0 },
+        );
+        let (draw_tx, _) = broadcast::channel(1);
+        let frame_requester = FrameRequester::new(draw_tx.clone());
+
+        Self {
+            frame_requester,
+            draw_tx,
+            event_broker: Arc::new(EventBroker::new()),
+            terminal,
+            pending_history_lines: vec![],
+            alt_saved_viewport: None,
+            #[cfg(unix)]
+            suspend_context: SuspendContext::new(),
+            alt_screen_active: Arc::new(AtomicBool::new(false)),
+            terminal_focused: Arc::new(AtomicBool::new(true)),
+            enhanced_keys_supported: false,
+            notification_backend: None,
+            alt_screen_enabled: true,
+        }
+    }
+
     /// Set whether alternate screen is enabled. When false, enter_alt_screen() becomes a no-op.
     pub fn set_alt_screen_enabled(&mut self, enabled: bool) {
         self.alt_screen_enabled = enabled;