root/fusion
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

perf(dashboard): hoist agent-store setup to beforeAll in routes.test

Browse Source 
The PATCH /tasks/:id/assign and Task checkout routes describe blocks
re-initialized a real AgentStore (sqlite init + createAgent) in their
beforeEach hooks, even though no test in either block mutates the agent
rows. Moving the agent setup to beforeAll while keeping the per-test
mock-store reset cuts ~20 sqlite init cycles across these two blocks.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
gsxdsm
2026-04-23 22:37:42 -07:00
parent fcf47d2c77
commit 9ddb475947
1 changed files with 14 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
packages/dashboard/src/routes.test.ts
View File

@@ -2965,7 +2965,9 @@ describe("PATCH /tasks/:id/assign and GET /agents/:id/tasks", () => {
let agentId: string;
let store: TaskStore;
beforeEach(async () => {
// Agent store init + createAgent is ~50ms per call; hoisted to beforeAll
// because no test in this block mutates the agent row.
beforeAll(async () => {
tempDir = mkdtempSync(join(tmpdir(), "kb-routes-task-assign-"));
fusionDir = join(tempDir, ".fusion");
mkdirSync(fusionDir, { recursive: true });
@@ -2978,16 +2980,18 @@ describe("PATCH /tasks/:id/assign and GET /agents/:id/tasks", () => {
role: "executor",
});
agentId = agent.id;
}, 30_000);
beforeEach(() => {
store = createMockStore({
getFusionDir: vi.fn().mockReturnValue(fusionDir),
updateTask: vi.fn(),
listTasks: vi.fn().mockResolvedValue([]),
selectNextTaskForAgent: vi.fn().mockResolvedValue(null),
} as any);
}, 30_000);
});
afterEach(() => {
afterAll(() => {
rmSync(tempDir, { recursive: true, force: true });
});
@@ -3204,7 +3208,9 @@ describe("Task checkout routes", () => {
let agentBId: string;
let taskState: TaskDetail;
beforeEach(async () => {
// Agent store init + createAgent is ~50ms per call; hoisted to beforeAll
// because no test in this block mutates the agent rows.
beforeAll(async () => {
tempDir = mkdtempSync(join(tmpdir(), "kb-routes-task-checkout-"));
fusionDir = join(tempDir, ".fusion");
mkdirSync(fusionDir, { recursive: true });
@@ -3224,7 +3230,9 @@ describe("Task checkout routes", () => {
agentAId = agentA.id;
agentBId = agentB.id;
}, 30_000);
beforeEach(() => {
taskState = {
...FAKE_TASK_DETAIL,
id: "FN-300",
@@ -3257,9 +3265,9 @@ describe("Task checkout routes", () => {
}),
logEntry: vi.fn().mockResolvedValue(undefined),
} as any);
}, 30_000);
});
afterEach(() => {
afterAll(() => {
rmSync(tempDir, { recursive: true, force: true });
});