From 193b9d5b8a7b83991d41e5a2c4b1c6f4f6526f35 Mon Sep 17 00:00:00 2001
From: James Long <17031+jlongster@users.noreply.github.com>
Date: Thu, 30 Jul 2026 16:52:59 +0000
Subject: [PATCH] test(tui): defer diff hunk coverage

---
 .../tui/test/component/patch-diff.test.tsx    | 79 -------------------
 packages/tui/test/util/diff.test.ts           | 40 ----------
 2 files changed, 119 deletions(-)
 delete mode 100644 packages/tui/test/component/patch-diff.test.tsx
 delete mode 100644 packages/tui/test/util/diff.test.ts

diff --git a/packages/tui/test/component/patch-diff.test.tsx b/packages/tui/test/component/patch-diff.test.tsx
deleted file mode 100644
index a47ba72bbf..0000000000
--- a/packages/tui/test/component/patch-diff.test.tsx
+++ /dev/null
@@ -1,79 +0,0 @@
-/** @jsxImportSource @opentui/solid */
-import { afterEach, expect, test } from "bun:test"
-import { DiffRenderable, parseColor, type Renderable, SyntaxStyle } from "@opentui/core"
-import { testRender } from "@opentui/solid"
-import { PatchDiff } from "../../src/component/patch-diff"
-
-let app: Awaited<ReturnType<typeof testRender>> | undefined
-
-afterEach(() => {
-  app?.renderer.destroy()
-  app = undefined
-})
-
-test("renders separate diff nodes with a full-width hunk row", async () => {
-  const patch = `--- a/file.ts
-+++ b/file.ts
-@@ -1,2 +1,3 @@
- const first = true
-+const addedFirst = true
- const afterFirst = true
-@@ -20,3 +20,3 @@
- const second = true
--const oldSecond = true
-+const newSecond = true
- const afterSecond = true`
-
-  app = await testRender(
-    () => (
-      <box width={120}>
-        <PatchDiff
-          diff={patch}
-          hunkFg="#888888"
-          lineNumberBg="#222222"
-          view="split"
-          filetype="typescript"
-          syntaxStyle={SyntaxStyle.create()}
-          showLineNumbers={true}
-          width="100%"
-        />
-      </box>
-    ),
-    { width: 120, height: 30 },
-  )
-  await app.waitForFrame((value) => value.includes("@@ -20,3 +20,3 @@"))
-  await app.renderOnce()
-  const frame = app.captureCharFrame()
-  const headerRow = frame.split("\n").findIndex((line) => line.includes("@@ -20,3 +20,3 @@"))
-  const header = frame.split("\n")[headerRow]
-  const firstLine = frame.split("\n").find((line) => line.includes("const first")) ?? ""
-  const secondLine = frame.split("\n").find((line) => line.includes("const second")) ?? ""
-  const background = parseColor("#222222")
-
-  expect(header?.startsWith(" @@ -20,3 +20,3 @@")).toBe(true)
-  expect(header?.trimEnd()).toBe(" @@ -20,3 +20,3 @@")
-  expect(
-    app
-      .captureSpans()
-      .lines[headerRow].spans.every(
-        (span) =>
-          span.bg.r === background.r &&
-          span.bg.g === background.g &&
-          span.bg.b === background.b &&
-          span.bg.a === background.a,
-      ),
-  ).toBe(true)
-  const diffs = findDiffs(app.renderer.root)
-  const gutters = diffs.flatMap((diff) => diff.getChildren().flatMap((side) => side.getChildren().slice(0, 1)))
-  expect(diffs).toHaveLength(2)
-  expect(gutters[0].width).toBeGreaterThan(0)
-  expect(new Set(gutters.map((gutter) => gutter.width)).size).toBe(1)
-  expect(firstLine.search(/\d/)).toBe(secondLine.search(/\d/))
-})
-
-function findDiffs(root: Renderable): DiffRenderable[] {
-  return [
-    ...(root instanceof DiffRenderable ? [root] : []),
-    ...root.getChildren().flatMap((child) => findDiffs(child)),
-  ]
-}
diff --git a/packages/tui/test/util/diff.test.ts b/packages/tui/test/util/diff.test.ts
deleted file mode 100644
index 98553d97f1..0000000000
--- a/packages/tui/test/util/diff.test.ts
+++ /dev/null
@@ -1,40 +0,0 @@
-import { expect, test } from "bun:test"
-import { splitPatchHunks } from "../../src/util/diff"
-
-test("splits a per-file patch into independently renderable hunks", () => {
-  const patch = `--- a/file.ts
-+++ b/file.ts
-@@ -1,3 +1,3 @@
- const first = true
--const oldFirst = true
-+const newFirst = true
- const afterFirst = true
-@@ -20,3 +20,3 @@
- const second = true
--const oldSecond = true
-+const newSecond = true
- const afterSecond = true`
-
-  const hunks = splitPatchHunks(patch)
-
-  expect(hunks).toHaveLength(2)
-  expect(hunks[0].header).toBe("@@ -1,3 +1,3 @@")
-  expect(hunks[1].header).toBe("@@ -20,3 +20,3 @@")
-  expect(hunks[0].rows).toBe(3)
-  expect(hunks[1].rows).toBe(3)
-  expect(hunks[0].patch).toContain("--- a/file.ts\n+++ b/file.ts")
-  expect(hunks[1].patch).toContain("--- a/file.ts\n+++ b/file.ts")
-  expect(hunks[0].patch).not.toContain("const second")
-  expect(hunks[1].patch).not.toContain("const first")
-})
-
-test("keeps patches with one or no hunks intact", () => {
-  const patch = `--- a/file.ts
-+++ b/file.ts
-@@ -1 +1 @@
--old
-+new`
-
-  expect(splitPatchHunks(patch)).toEqual([{ patch }])
-  expect(splitPatchHunks("not a patch")).toEqual([{ patch: "not a patch" }])
-})