From 11ffa134ade2ae34cd08eb9d5bf9b608701ba9f6 Mon Sep 17 00:00:00 2001 From: Thomas Kosiewski Date: Mon, 15 Jun 2026 09:49:56 +0000 Subject: [PATCH] =?UTF-8?q?=F0=9F=A4=96=20tests:=20stabilize=20deep-resear?= =?UTF-8?q?ch=20fanout=20test?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../workflows/builtInWorkflowDefinitions.test.ts | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/src/node/services/workflows/builtInWorkflowDefinitions.test.ts b/src/node/services/workflows/builtInWorkflowDefinitions.test.ts index e908122152..0451fb0a87 100644 --- a/src/node/services/workflows/builtInWorkflowDefinitions.test.ts +++ b/src/node/services/workflows/builtInWorkflowDefinitions.test.ts @@ -11,8 +11,7 @@ import { WorkflowActionRegistry } from "./WorkflowActionRegistry"; import { WorkflowRunStore } from "./WorkflowRunStore"; import { WorkflowRunner, type WorkflowAgentResult, type WorkflowAgentSpec } from "./WorkflowRunner"; -// Keep this above tiny lock timeouts: these fixtures exercise parallel workflow callbacks -// that legitimately contend for the run-store mutation lock under CI coverage load. +// Most fixtures use short leases so stale-run retry behavior stays fast. const BUILT_IN_WORKFLOW_TEST_STALE_LEASE_MS = 100; const deepResearch = BUILT_IN_WORKFLOW_DEFINITIONS.find( @@ -2101,10 +2100,9 @@ describe("built-in deep-research workflow", () => { throw new Error("Expected built-in deep-research workflow"); } using tmp = new DisposableTempDir("deep-research-capped-workflow"); - const runStore = new WorkflowRunStore({ - sessionDir: tmp.path, - staleLeaseMs: BUILT_IN_WORKFLOW_TEST_STALE_LEASE_MS, - }); + // This fixture intentionally creates heavy parallel step persistence; use production + // lease timing so the assertion covers fan-out caps, not 50ms renewal churn. + const runStore = new WorkflowRunStore({ sessionDir: tmp.path }); await runStore.createRun({ id: "wfr_deep_research_capped", workspaceId: "workspace-1", @@ -2240,7 +2238,7 @@ describe("built-in deep-research workflow", () => { expect(structuredOutput.claims).toHaveLength(16); expect(structuredOutput.verification).toHaveLength(16); expect(structuredOutput.stats.claimsExtracted).toBe(75); - }, 10_000); + }, 30_000); }); describe("built-in deep-review-workflow", () => {