From bb95ec3ec602dfc7762fd92e2746606df9dfea21 Mon Sep 17 00:00:00 2001 From: Eric Traut Date: Sun, 29 Mar 2026 17:54:17 -0600 Subject: [PATCH] [codex] Normalize Windows path in MCP startup snapshot test (#16204) ## Summary A Windows-only snapshot assertion in the app-server MCP startup warning test compared the raw rendered path, so CI saw `C:\tmp\project` instead of the normalized `/tmp/project` snapshot fixture. ## Fix Route that snapshot assertion through the existing `normalize_snapshot_paths(...)` helper so the test remains platform-stable. --- codex-rs/tui/src/chatwidget/tests.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/codex-rs/tui/src/chatwidget/tests.rs b/codex-rs/tui/src/chatwidget/tests.rs index b4950dad7..ebb92909e 100644 --- a/codex-rs/tui/src/chatwidget/tests.rs +++ b/codex-rs/tui/src/chatwidget/tests.rs @@ -11760,7 +11760,7 @@ async fn app_server_mcp_startup_failure_renders_warning_history() { assert_snapshot!( "app_server_mcp_startup_failure_renders_warning_history", - term.backend().vt100().screen().contents() + normalize_snapshot_paths(term.backend().vt100().screen().contents()) ); }