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) <noreply@anthropic.com>
This commit is contained in:
gsxdsm
2026-05-19 19:10:08 -07:00
parent 836da032e6
commit 49990a1b13
2 changed files with 4 additions and 1 deletions

View File

@@ -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");

View File

@@ -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" }),