fix(test): deduplicates reset helpers and stabilizes timing test

- resetViewStore delegates to resetViewState (single source of truth)

- increases pooled concurrency test delay from 5ms to 20ms (fixes flake)

Author: wgordon17

tests/helpers/index.tsx

@@ -1,6 +1,6 @@
 import { render } from "@solidjs/testing-library";
 import { MemoryRouter, createMemoryHistory } from "@solidjs/router";
-import { updateViewState } from "../../src/app/stores/view";
+import { resetViewState } from "../../src/app/stores/view";
 import type { Issue, PullRequest, WorkflowRun, ApiError } from "../../src/app/services/api";
 import type { JSX } from "solid-js";
 
@@ -103,17 +103,5 @@ export function renderWithRouter(
 }
 
 export function resetViewStore(): void {
-  updateViewState({
-    lastActiveTab: "issues",
-    sortPreferences: {},
-    ignoredItems: [],
-    globalFilter: { org: null, repo: null },
-    tabFilters: {
-      issues: { role: "all", comments: "all" },
-      pullRequests: { role: "all", reviewDecision: "all", draft: "all", checkStatus: "all", sizeCategory: "all" },
-      actions: { conclusion: "all", event: "all" },
-    },
-    showPrRuns: false,
-    expandedRepos: { issues: {}, pullRequests: {}, actions: {} },
-  });
+  resetViewState();
 }

tests/services/api-optimization.test.ts

@@ -591,7 +591,7 @@ describe("workflow run concurrency", () => {
 
   it("processes repos faster with pooled concurrency than sequential chunks", async () => {
     const repos = makeRepos(30);
-    const DELAY_MS = 5;
+    const DELAY_MS = 20;
 
     const makeTimedOctokit = () => ({
       request: vi.fn(async () => {