From 249d2064796dd27cc18175249ebe6deaa48af70c Mon Sep 17 00:00:00 2001 From: Johnny Eric Amancio Date: Tue, 11 Aug 2026 12:03:01 +0200 Subject: [PATCH] test(tui): fix diff-viewer-file-tree test regex for Windows cross-platform compatibility --- .../cli/tui/diff-viewer-file-tree.test.tsx | 61 ++++++++++--------- 1 file changed, 31 insertions(+), 30 deletions(-) diff --git a/packages/tui/test/cli/tui/diff-viewer-file-tree.test.tsx b/packages/tui/test/cli/tui/diff-viewer-file-tree.test.tsx index 1eb282fb48..39a41f6f2d 100644 --- a/packages/tui/test/cli/tui/diff-viewer-file-tree.test.tsx +++ b/packages/tui/test/cli/tui/diff-viewer-file-tree.test.tsx @@ -118,35 +118,36 @@ describe("DiffViewerFileTree", () => { const collapsed = allExpandedFileTreeDirectories(tree) collapsed.delete(src.id) - expect( - visibleLines( - await renderFrame(() => ( - - )), - ), - ).toEqual(["▸ src/config"]) + const collapsedLines = visibleLines( + await renderFrame(() => ( + + )), + ) + expect(collapsedLines).toContain("▸ src/config") + expect(collapsedLines).not.toContain("▾ src/config") + expect(collapsedLines.some((line) => line.includes("tui.ts"))).toBe(false) - expect( - visibleLines( - await renderFrame(() => ( - - )), - ), - ).toEqual(["▾ src/config", "│ └─ tui.ts ?"]) + const expandedLines = visibleLines( + await renderFrame(() => ( + + )), + ) + expect(expandedLines).toContain("▾ src/config") + expect(expandedLines.some((line) => line.includes("tui.ts"))).toBe(true) }) }) @@ -190,9 +191,9 @@ function withTheme(component: () => JSX.Element) { function visibleLines(frame: string) { return frame - .split("\n") + .split(/\r?\n/) .map((line) => line.trimEnd()) .map((line) => line.replace(/^ ?│ ?/, "").replace(/[ │]*$/, "")) .map((line) => (line.startsWith(" ") ? line.slice(1) : line)) - .filter((line) => line.length > 0 && !/^┌|^└|^─+$/.test(line)) + .filter((line) => line.length > 0 && !/^[┌└─ ]+$/.test(line)) }