test(ui): adds remaining identified-but-unactioned tests

PR/Actions un-ignore round-trip, showPrRuns+ignore combo, timer clear

Author: wgordon17

tests/components/DashboardPage.test.tsx

@@ -322,9 +322,15 @@ describe("DashboardPage — tab badge counts", () => {
     await waitFor(() => {
       expect(screen.getByRole("tab", { name: /Pull Requests/ }).textContent?.replace(/\D+/g, "")).toBe("2");
     });
+
+    // Un-ignore — badge should increment back to 3
+    viewStore.unignoreItem("10");
+    await waitFor(() => {
+      expect(screen.getByRole("tab", { name: /Pull Requests/ }).textContent?.replace(/\D+/g, "")).toBe("3");
+    });
   });
 
-  it("decrements Actions badge on ignore", async () => {
+  it("decrements Actions badge on ignore and increments on un-ignore", async () => {
     vi.mocked(pollService.fetchAllData).mockResolvedValue({
       issues: [],
       pullRequests: [],
@@ -344,6 +350,12 @@ describe("DashboardPage — tab badge counts", () => {
     await waitFor(() => {
       expect(screen.getByRole("tab", { name: /Actions/ }).textContent?.replace(/\D+/g, "")).toBe("1");
     });
+
+    // Un-ignore — badge should increment back to 2
+    viewStore.unignoreItem("20");
+    await waitFor(() => {
+      expect(screen.getByRole("tab", { name: /Actions/ }).textContent?.replace(/\D+/g, "")).toBe("2");
+    });
   });
 
   it("excludes PR-triggered runs from badge count by default", async () => {
@@ -389,6 +401,31 @@ describe("DashboardPage — tab badge counts", () => {
     });
   });
 
+  it("combines showPrRuns and ignore exclusions for Actions badge", async () => {
+    vi.mocked(pollService.fetchAllData).mockResolvedValue({
+      issues: [],
+      pullRequests: [],
+      workflowRuns: [
+        makeWorkflowRun({ id: 20, isPrRun: false }),
+        makeWorkflowRun({ id: 21, isPrRun: true }),
+        makeWorkflowRun({ id: 22, isPrRun: true }),
+      ],
+      errors: [],
+    });
+    viewStore.updateViewState({ showPrRuns: true });
+
+    render(() => <DashboardPage />);
+    await waitFor(() => {
+      expect(screen.getByRole("tab", { name: /Actions/ }).textContent?.replace(/\D+/g, "")).toBe("3");
+    });
+
+    // Ignore one PR-triggered run — badge should drop to 2
+    viewStore.ignoreItem({ id: "21", type: "workflowRun", repo: "owner/repo", title: "CI", ignoredAt: Date.now() });
+    await waitFor(() => {
+      expect(screen.getByRole("tab", { name: /Actions/ }).textContent?.replace(/\D+/g, "")).toBe("2");
+    });
+  });
+
   it("filters badge counts by globalFilter repo", async () => {
     vi.mocked(pollService.fetchAllData).mockResolvedValue({
       issues: [

tests/lib/reorderHighlight.test.ts

@@ -1,4 +1,4 @@
-import { describe, it, expect } from "vitest";
+import { describe, it, expect, vi } from "vitest";
 import { createRoot, createSignal, batch, type Accessor } from "solid-js";
 import { createReorderHighlight } from "../../src/app/lib/reorderHighlight";
 
@@ -183,4 +183,31 @@ describe("createReorderHighlight", () => {
 
     disposeRoot();
   });
+
+  it("clears highlight after 1500ms timeout", () => {
+    vi.useFakeTimers();
+
+    let highlighted!: Accessor<ReadonlySet<string>>;
+    let setOrder!: (v: string[]) => void;
+    let disposeRoot!: () => void;
+
+    createRoot((dispose) => {
+      const [order, _setOrder] = createSignal<string[]>(["a", "b", "c"]);
+      const [locked] = createSignal<string[]>([]);
+      const [ignored] = createSignal(0);
+      setOrder = _setOrder;
+      highlighted = createReorderHighlight(order, locked, ignored);
+      disposeRoot = dispose;
+    });
+
+    setOrder(["c", "a", "b"]);
+    expect(highlighted().size).toBeGreaterThan(0);
+
+    // Advance past the 1500ms clear timeout
+    vi.advanceTimersByTime(1500);
+    expect(highlighted().size).toBe(0);
+
+    disposeRoot();
+    vi.useRealTimers();
+  });
 });