From c05cf3493a2c7a07b8bc5ece6e3c77aba1c0b1c7 Mon Sep 17 00:00:00 2001 From: Vincent Koc Date: Sun, 12 Apr 2026 05:28:06 +0100 Subject: [PATCH] fix(test): skip heavy-check lock for scoped tooling targets --- scripts/test-projects.test-support.mjs | 1 - test/scripts/test-projects.test.ts | 31 ++++---------------------- 2 files changed, 4 insertions(+), 28 deletions(-) diff --git a/scripts/test-projects.test-support.mjs b/scripts/test-projects.test-support.mjs index 567785791c0..b5f6f3431e1 100644 --- a/scripts/test-projects.test-support.mjs +++ b/scripts/test-projects.test-support.mjs @@ -760,7 +760,6 @@ export function shouldAcquireLocalHeavyCheckLock(runSpecs, env = process.env) { } return !( - env.OPENCLAW_TEST_PROJECTS_SERIAL === "1" && runSpecs.length === 1 && runSpecs[0]?.config === TOOLING_VITEST_CONFIG && runSpecs[0]?.watchMode === false && diff --git a/test/scripts/test-projects.test.ts b/test/scripts/test-projects.test.ts index 16d2e4f803c..a650ac1ec1f 100644 --- a/test/scripts/test-projects.test.ts +++ b/test/scripts/test-projects.test.ts @@ -197,7 +197,7 @@ describe("scripts/test-projects changed-target routing", () => { }); describe("scripts/test-projects local heavy-check lock", () => { - it("skips the lock for a single scoped serial tooling run", () => { + it("skips the lock for a single scoped tooling run", () => { expect( shouldAcquireLocalHeavyCheckLock( [ @@ -207,31 +207,12 @@ describe("scripts/test-projects local heavy-check lock", () => { watchMode: false, }, ], - { - ...process.env, - OPENCLAW_TEST_PROJECTS_SERIAL: "1", - }, + process.env, ), ).toBe(false); }); - it("keeps the lock for non-tooling or non-serial runs", () => { - expect( - shouldAcquireLocalHeavyCheckLock( - [ - { - config: "test/vitest/vitest.tooling.config.ts", - includePatterns: ["test/scripts/committer.test.ts"], - watchMode: false, - }, - ], - { - ...process.env, - OPENCLAW_TEST_PROJECTS_SERIAL: "0", - }, - ), - ).toBe(true); - + it("keeps the lock for non-tooling runs", () => { expect( shouldAcquireLocalHeavyCheckLock( [ @@ -241,10 +222,7 @@ describe("scripts/test-projects local heavy-check lock", () => { watchMode: false, }, ], - { - ...process.env, - OPENCLAW_TEST_PROJECTS_SERIAL: "1", - }, + process.env, ), ).toBe(true); }); @@ -262,7 +240,6 @@ describe("scripts/test-projects local heavy-check lock", () => { { ...process.env, OPENCLAW_TEST_PROJECTS_FORCE_LOCK: "1", - OPENCLAW_TEST_PROJECTS_SERIAL: "1", }, ), ).toBe(true);