From 49990a1b1371ed6dd0c331f656939d726f9b5a44 Mon Sep 17 00:00:00 2001 From: gsxdsm Date: Tue, 19 May 2026 19:10:08 -0700 Subject: [PATCH] test(dashboard): align PR-reviews/PR-merged tests with current signatures - routes-pr-reviews mock store now provides updatePrInfoByNumber and addPrInfo (refresh route uses both for the multi-PR refresh path). - pr-merged-auto-done.integration calls store.init() and passes refreshPrInBackground a PrInfo[] array (the signature changed when multi-PR support landed). Fixes 2 failing tests. Co-Authored-By: Claude Opus 4.7 (1M context) --- .../src/__tests__/pr-merged-auto-done.integration.test.ts | 3 ++- packages/dashboard/src/__tests__/routes-pr-reviews.test.ts | 2 ++ 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/packages/dashboard/src/__tests__/pr-merged-auto-done.integration.test.ts b/packages/dashboard/src/__tests__/pr-merged-auto-done.integration.test.ts index 696d72e27..bcda78efc 100644 --- a/packages/dashboard/src/__tests__/pr-merged-auto-done.integration.test.ts +++ b/packages/dashboard/src/__tests__/pr-merged-auto-done.integration.test.ts @@ -38,6 +38,7 @@ describe("pr merged refresh auto-done", () => { rootDir = mkdtempSync(join(tmpdir(), "fn-4762-pr-merged-root-")); globalDir = mkdtempSync(join(tmpdir(), "fn-4762-pr-merged-global-")); store = new TaskStore(rootDir, globalDir, { inMemoryDb: true }); + await store.init(); }); afterEach(() => { @@ -60,7 +61,7 @@ describe("pr merged refresh auto-done", () => { commentCount: 0, }); - await refreshPrInBackground(store, task.id, (await store.getTask(task.id)).prInfo!); + await refreshPrInBackground(store, task.id, [(await store.getTask(task.id)).prInfo!]); const updated = await store.getTask(task.id); expect(updated.column).toBe("done"); diff --git a/packages/dashboard/src/__tests__/routes-pr-reviews.test.ts b/packages/dashboard/src/__tests__/routes-pr-reviews.test.ts index b42852fe5..d5714a7df 100644 --- a/packages/dashboard/src/__tests__/routes-pr-reviews.test.ts +++ b/packages/dashboard/src/__tests__/routes-pr-reviews.test.ts @@ -34,6 +34,8 @@ function createStore(task: Task): TaskStore { return { getTask: vi.fn().mockResolvedValue(task), updatePrInfo: vi.fn().mockResolvedValue(undefined), + updatePrInfoByNumber: vi.fn().mockResolvedValue(undefined), + addPrInfo: vi.fn().mockResolvedValue(undefined), addComment: vi.fn().mockResolvedValue(task), moveTask: vi.fn().mockResolvedValue({ ...task, column: "todo" }), upsertTaskDocument: vi.fn().mockResolvedValue({ key: "review-feedback" }),