From 8d31370b0ba44f299510028a052d70d16ae95998 Mon Sep 17 00:00:00 2001 From: gsxdsm Date: Wed, 3 Jun 2026 16:29:09 -0700 Subject: [PATCH] FN-5965: use temporary root in worktree recovery test Harden executor worktree recovery coverage against fixed-path collisions. - create a unique temporary root for the stale unregistered worktree test case - derive the stale worktree path from that temporary root instead of a shared /tmp/test path - initialize TaskExecutor with the same temporary root so setup and assertions stay aligned Files changed: packages/engine/src/__tests__/executor-worktree.test.ts | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) Fusion-Task-Id: FN-5965 Fusion-Task-Lineage: 4436bd02-b7f7-4e62-8514-0aef6cb67813 --- packages/engine/src/__tests__/executor-worktree.test.ts | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/packages/engine/src/__tests__/executor-worktree.test.ts b/packages/engine/src/__tests__/executor-worktree.test.ts index 52159f72ba..56a05d23eb 100644 --- a/packages/engine/src/__tests__/executor-worktree.test.ts +++ b/packages/engine/src/__tests__/executor-worktree.test.ts @@ -1695,7 +1695,8 @@ describe("TaskExecutor worktree recovery", () => { it("removes existing directory that is not a registered worktree", async () => { const store = createMockStore(); const fs = await import("node:fs/promises"); - const staleWorktreePath = "/tmp/test/.worktrees/swift-falcon"; + const tempRoot = await fs.mkdtemp("/tmp/executor-worktree-"); + const staleWorktreePath = `${tempRoot}/.worktrees/swift-falcon`; // Directory exists but is not registered mockedExistsSync.mockReturnValue(true); @@ -1712,7 +1713,7 @@ describe("TaskExecutor worktree recovery", () => { return Buffer.from(""); }); - const executor = new TaskExecutor(store, "/tmp/test"); + const executor = new TaskExecutor(store, tempRoot); await executor.execute(makeTask()); expect(