Skip to content

Commit 1b10a42

Browse files
author
Paulo Aboim Pinto
committed
fix(tui): assert normalized footer paths in spillover tests
8f2b622 normalized the model-facing artifact footer to forward slashes (platform-independent). Two unit tests still asserted the raw OS path against that footer, failing on the Windows lane: - truncate::adaptive_evidence_footer_names_artifact_path_and_recovery - subagent::subagent_tool_results_spill_to_disk_and_stay_bounded_inline (upstream fa7c4b0) Assert the normalized form via format_artifact_relative_path, matching what the footer actually emits.
1 parent 43a55ce commit 1b10a42

2 files changed

Lines changed: 6 additions & 2 deletions

File tree

crates/tui/src/tools/subagent/tests.rs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13652,7 +13652,9 @@ fn subagent_tool_results_spill_to_disk_and_stay_bounded_inline() {
1365213652
assert!(inline.contains("full output at"));
1365313653
assert!(inline.contains(crate::tools::truncate::SPILLOVER_RECOVERY_HINT));
1365413654
assert!(inline.contains("\n…\n"));
13655-
assert!(inline.contains(&path.display().to_string()));
13655+
assert!(inline.contains(&crate::artifacts::format_artifact_relative_path(
13656+
&path
13657+
)));
1365613658
assert!(!inline.contains("Exact evidence retained"));
1365713659
assert!(inline.contains("retrieve_tool_result"), "{inline}");
1365813660
// Full output remains recoverable from disk.

crates/tui/src/tools/truncate.rs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1687,7 +1687,9 @@ mod tests {
16871687
assert!(result.content.contains("of output omitted ("));
16881688
assert!(result.content.contains(" lines)"));
16891689
assert!(result.content.contains("full output at"));
1690-
assert!(result.content.contains(&path.display().to_string()));
1690+
assert!(result.content.contains(
1691+
&crate::artifacts::format_artifact_relative_path(&path)
1692+
));
16911693
assert!(result.content.contains(SPILLOVER_RECOVERY_HINT));
16921694
assert!(!result.content.contains(SPILLOVER_PREVIEW_HINT));
16931695

0 commit comments

Comments
 (0)