Remove OpenTUI re-tests from the overlay list suite

Author: TheGreatAxios

src/tui/overlay-list.test.ts

@@ -1,9 +1,9 @@
 /**
- * Windowing semantics of the shell's overlay list, which delegates selection
- * clamping and scroll-keep-visible to @opentui/core's SelectRenderable.
- * Ported from list-viewport.test.ts when the hand-rolled viewport kit was
- * deleted; assertions are behavioural (active stays inside the window) rather
- * than pinning the old edge-follow scroll rule.
+ * Selection continuity of the shell's overlay list across count and height
+ * changes. Scroll clamping and windowing itself are delegated to
+ * @opentui/core's SelectRenderable and are not re-tested here; what this pins
+ * is the wrapper's own reshape/setCount logic that carries the live selection
+ * across a rebuild so a resize does not snap the cursor back.
  */
 import { describe, expect, test } from "bun:test";
 import { withTestRenderer } from "./harness";
@@ -31,74 +31,7 @@ async function withList(
   });
 }
 
-describe("createOverlayList", () => {
-  test("empty list shows nothing", async () => {
-    await withList({ count: 0, items: 5 }, (list) => {
-      expect(list.count).toBe(0);
-      expect(list.activeIndex).toBe(0);
-      expect(list.visibleRange()).toEqual({ start: 0, end: 0 });
-    });
-  });
-
-  test("short list fits entirely", async () => {
-    await withList({ count: 3, items: 10 }, (list) => {
-      expect(list.offset).toBe(0);
-      expect(windowOf(list)).toEqual([0, 1, 2]);
-    });
-  });
-
-  test("tall list starts at top", async () => {
-    await withList({ count: 30, items: 5 }, (list) => {
-      expect(list.offset).toBe(0);
-      expect(windowOf(list)).toEqual([0, 1, 2, 3, 4]);
-    });
-  });
-
-  test("initial activeIndex deep in the list scrolls the window to it", async () => {
-    await withList({ count: 30, items: 5, activeIndex: 20 }, (list) => {
-      expect(list.activeIndex).toBe(20);
-      expect(windowOf(list)).toContain(20);
-      expect(list.offset).toBeGreaterThan(0);
-    });
-  });
-
-  test("clamps negative and oversized activeIndex", async () => {
-    await withList({ count: 10, items: 3, activeIndex: 99 }, (list) => {
-      expect(list.activeIndex).toBe(9);
-    });
-  });
-});
-
-describe("move / page / jump", () => {
-  test("moves down and keeps active visible", async () => {
-    await withList({ count: 20, items: 4 }, (list) => {
-      for (let i = 0; i < 6; i++) list.move(1);
-      expect(list.activeIndex).toBe(6);
-      activeVisible(list);
-    });
-  });
-
-  test("moves up and keeps active visible", async () => {
-    await withList({ count: 20, items: 4, activeIndex: 10 }, (list) => {
-      list.move(-3);
-      expect(list.activeIndex).toBe(7);
-      activeVisible(list);
-    });
-  });
-
-  test("clamps at ends", async () => {
-    await withList({ count: 5, items: 3, activeIndex: 4 }, (list) => {
-      list.move(1);
-      expect(list.activeIndex).toBe(4);
-      list.move(1);
-      expect(list.activeIndex).toBe(4);
-      list.move(-99);
-      expect(list.activeIndex).toBe(0);
-      list.move(-1);
-      expect(list.activeIndex).toBe(0);
-    });
-  });
-
+describe("page / jump", () => {
   test("page steps by the window height minus one", async () => {
     await withList({ count: 30, items: 5 }, (list) => {
       list.page(1);