test(dashboard): update routes-approval expectations for 5-pending fixture
The fixture got a 6th approval (apr-6, worktrunk_install) for the new worktrunk-installer tests, but the prior pendingCount assertion still expected 4 pending. Update to 5. Also add getSettings stub to the project-context store so the worktrunk-install approve handler doesn't crash on the new code path. Fixes 2 failing tests in routes-approval.test.ts. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -103,6 +103,7 @@ describe("approval routes", async () => {
|
|||||||
getDatabase: () => ({}),
|
getDatabase: () => ({}),
|
||||||
getFusionDir: () => "/tmp/fusion",
|
getFusionDir: () => "/tmp/fusion",
|
||||||
getTask: async () => state.task,
|
getTask: async () => state.task,
|
||||||
|
getSettings: async () => ({ worktrunk: {} }),
|
||||||
pauseTask: async (_id: string, paused: boolean) => {
|
pauseTask: async (_id: string, paused: boolean) => {
|
||||||
state.task = { ...state.task, paused, pausedByAgentId: paused ? state.task.pausedByAgentId : undefined };
|
state.task = { ...state.task, paused, pausedByAgentId: paused ? state.task.pausedByAgentId : undefined };
|
||||||
},
|
},
|
||||||
@@ -240,9 +241,9 @@ describe("approval routes", async () => {
|
|||||||
const app = createApp();
|
const app = createApp();
|
||||||
const res = await get(app, "/api/approvals?status=pending");
|
const res = await get(app, "/api/approvals?status=pending");
|
||||||
expect(res.status).toBe(200);
|
expect(res.status).toBe(200);
|
||||||
expect(res.body.total).toBe(4);
|
expect(res.body.total).toBe(5);
|
||||||
expect(res.body.pendingCount).toBe(4);
|
expect(res.body.pendingCount).toBe(5);
|
||||||
expect(res.body.requests).toHaveLength(4);
|
expect(res.body.requests).toHaveLength(5);
|
||||||
expect(res.body.requests[0]).toMatchObject({
|
expect(res.body.requests[0]).toMatchObject({
|
||||||
id: "apr-1",
|
id: "apr-1",
|
||||||
actionCategory: "command_execution",
|
actionCategory: "command_execution",
|
||||||
|
|||||||
Reference in New Issue
Block a user