Skip to content

Commit bd632f5

Browse files
committed
Expect native path separators in scan headers
scan_hosted_paths_run_once_per_project_directory compared the per-directory `== apps/a ==` header against a literal forward-slash path, but scan prints the directory glob matched, which Windows spells `apps\a`. The Windows test leg failed on this since 62f07c7, and because every e2e job waits on `test`, the whole e2e tier was skipped on v5 PRs. Build the expected header from path components. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AfGnDQMy2FvpExneXmR1qp
1 parent 90ad98d commit bd632f5

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

‎crates/socket-patch-cli/tests/covgap_commands_scan_mod.rs‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1484,8 +1484,9 @@ async fn scan_hosted_paths_run_once_per_project_directory() {
14841484

14851485
let (code, stdout, stderr) = run_scan_human(tmp.path(), &mock.uri(), &["apps/*"]);
14861486
assert_eq!(code, 0, "stdout={stdout}; stderr={stderr}");
1487-
for app in ["apps/a", "apps/b"] {
1488-
let header = format!("== {} ==", std::path::Path::new(app).display());
1487+
// glob rebuilds matches with the native separator (`apps\a` on Windows).
1488+
for app in ["a", "b"] {
1489+
let header = format!("== {} ==", Path::new("apps").join(app).display());
14891490
assert!(stdout.contains(&header), "missing {header:?}: {stdout}");
14901491
}
14911492
assert_eq!(stdout.matches("Redirected 0 packages").count(), 2, "{stdout}");

0 commit comments

Comments
 (0)