Skip to content

Commit 010c7f7

Browse files
committed
fix(qa): disable pnpm verify in cpu scenarios
1 parent 69891cf commit 010c7f7

2 files changed

Lines changed: 20 additions & 6 deletions

File tree

scripts/check-gateway-cpu-scenarios.mjs

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -247,6 +247,7 @@ function buildPrivateQaEnv(env, qaState) {
247247
OPENCLAW_ENABLE_PRIVATE_QA_CLI: "1",
248248
OPENCLAW_RUN_NODE_SKIP_DTS_BUILD: env.OPENCLAW_RUN_NODE_SKIP_DTS_BUILD ?? "1",
249249
OPENCLAW_TEST_DISABLE_UPDATE_CHECK: env.OPENCLAW_TEST_DISABLE_UPDATE_CHECK ?? "1",
250+
PNPM_CONFIG_VERIFY_DEPS_BEFORE_RUN: env.PNPM_CONFIG_VERIFY_DEPS_BEFORE_RUN ?? "false",
250251
};
251252
}
252253

@@ -264,7 +265,11 @@ function createQaState(outputDir) {
264265

265266
async function runGatewayCpuScenarios(options, params = {}) {
266267
const repoRoot = params.cwd ?? process.cwd();
267-
const baseEnv = params.env ?? process.env;
268+
const inputEnv = params.env ?? process.env;
269+
const baseEnv = {
270+
...inputEnv,
271+
PNPM_CONFIG_VERIFY_DEPS_BEFORE_RUN: inputEnv.PNPM_CONFIG_VERIFY_DEPS_BEFORE_RUN ?? "false",
272+
};
268273
fs.mkdirSync(options.outputDir, { recursive: true });
269274

270275
const startupOutput = path.join(options.outputDir, "gateway-startup-bench.json");
@@ -284,7 +289,7 @@ async function runGatewayCpuScenarios(options, params = {}) {
284289
"startup build",
285290
process.execPath,
286291
["scripts/ensure-cli-startup-build.mjs"],
287-
{},
292+
{ env: baseEnv },
288293
params,
289294
);
290295
steps.push(startupBuild);
@@ -305,7 +310,7 @@ async function runGatewayCpuScenarios(options, params = {}) {
305310
startupOutput,
306311
...options.startupCases.flatMap((id) => ["--case", id]),
307312
],
308-
{},
313+
{ env: baseEnv },
309314
params,
310315
)
311316
: { name: "startup bench", signal: null, status: 1 },

test/scripts/check-gateway-cpu-scenarios.test.ts

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -121,9 +121,14 @@ describe("gateway CPU scenario guard", () => {
121121
});
122122

123123
it("prepares CLI startup artifacts before running the startup bench", async () => {
124+
type SpawnCall = {
125+
args: string[];
126+
command: string;
127+
env?: Record<string, string | undefined>;
128+
};
124129
const outputDir = makeTempRoot();
125130
const startupOutput = path.join(outputDir, "gateway-startup-bench.json");
126-
const calls: Array<{ command: string; args: string[] }> = [];
131+
const calls: SpawnCall[] = [];
127132
const options = testing.parseArgs([
128133
"--output-dir",
129134
outputDir,
@@ -136,8 +141,8 @@ describe("gateway CPU scenario guard", () => {
136141

137142
const result = await testing.runGatewayCpuScenarios(options, {
138143
silent: true,
139-
spawnSync: (command: string, args: string[]) => {
140-
calls.push({ command, args });
144+
spawnSync: (command: string, args: string[], opts?: { env?: Record<string, string> }) => {
145+
calls.push({ args, command, env: opts?.env });
141146
if (args.includes("scripts/bench-gateway-startup.ts")) {
142147
writeFileSync(startupOutput, `${JSON.stringify({ results: [{ id: "default" }] })}\n`);
143148
}
@@ -151,6 +156,8 @@ describe("gateway CPU scenario guard", () => {
151156
"--import",
152157
]);
153158
expect(calls[1]?.args).toContain("scripts/bench-gateway-startup.ts");
159+
expect(calls[0]?.env?.PNPM_CONFIG_VERIFY_DEPS_BEFORE_RUN).toBe("false");
160+
expect(calls[1]?.env?.PNPM_CONFIG_VERIFY_DEPS_BEFORE_RUN).toBe("false");
154161
});
155162

156163
it("fails successful startup benches that do not write a report", async () => {
@@ -297,6 +304,7 @@ describe("gateway CPU scenario guard", () => {
297304
OPENCLAW_RUN_NODE_SKIP_DTS_BUILD: "1",
298305
OPENCLAW_STATE_DIR: path.join(outputDir, "qa-state-root", "state"),
299306
OPENCLAW_TEST_DISABLE_UPDATE_CHECK: "1",
307+
PNPM_CONFIG_VERIFY_DEPS_BEFORE_RUN: "false",
300308
USERPROFILE: path.join(outputDir, "qa-state-root", "home"),
301309
});
302310
expect(calls[0]?.env?.OPENCLAW_BUNDLED_PLUGIN_BUILD_IDS).toBeUndefined();
@@ -344,6 +352,7 @@ describe("gateway CPU scenario guard", () => {
344352
OPENCLAW_CONFIG_PATH: path.join(outputDir, "qa-state-root", "state", "openclaw.json"),
345353
OPENCLAW_HOME: path.join(outputDir, "qa-state-root", "home"),
346354
OPENCLAW_STATE_DIR: path.join(outputDir, "qa-state-root", "state"),
355+
PNPM_CONFIG_VERIFY_DEPS_BEFORE_RUN: "false",
347356
USERPROFILE: path.join(outputDir, "qa-state-root", "home"),
348357
});
349358
expect(calls[0]?.env?.HOME).not.toBe("/real/user/home");

0 commit comments

Comments
 (0)