From b9137f00f3e839c42bc43dbd175b8bed6421fe9a Mon Sep 17 00:00:00 2001 From: CodeWhale Bot Date: Wed, 23 Sep 2026 09:11:39 -0700 Subject: [PATCH] test(runtime-api): build expected diagnostics paths with Path::join diagnostics_list_and_read_bounded_windows compared the crash directory the API reports (built with Path::join) against a fixture path written as "cwhome/crashes", which only matches on Unix. Windows CI failed on unrelated PRs (#6433 job 107246026498, #6434 job 107246133133): left `...\cwhome\crashes`, right `...\cwhome/crashes`. The product is correct; the fixture now joins components. Evidence: the test passes locally (1 passed, 0 failed); Windows acceptance is the next hosted run. Co-Authored-By: Claude Opus 5.5 (1M context) --- crates/tui/src/runtime_api/tests.rs | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/crates/tui/src/runtime_api/tests.rs b/crates/tui/src/runtime_api/tests.rs index 85e61e6a54..915dcd256f 100644 --- a/crates/tui/src/runtime_api/tests.rs +++ b/crates/tui/src/runtime_api/tests.rs @@ -17582,9 +17582,13 @@ async fn diagnostics_list_and_read_bounded_windows() -> Result<()> { // An explicit profile owns both logs and crashes, even when ambient // legacy crash files exist outside it. let _user_home = crate::test_support::EnvVarGuard::set("HOME", tmp.path().join("userhome")); - let logs = tmp.path().join("cwhome/logs"); - let crashes = tmp.path().join("cwhome/crashes"); - let ambient_crashes = tmp.path().join("userhome/.deepseek/crashes"); + let logs = tmp.path().join("cwhome").join("logs"); + let crashes = tmp.path().join("cwhome").join("crashes"); + let ambient_crashes = tmp + .path() + .join("userhome") + .join(".deepseek") + .join("crashes"); fs::create_dir_all(&ambient_crashes)?; fs::write( ambient_crashes.join("ambient-only.log"),