fix(test): guard force runner help · openclaw/openclaw@f6...

Recent Commits to openclaw:main

vincentkoc · 2026-05-30 · via Recent Commits to openclaw:main

Original file line number	Diff line number	Diff line change
`@@ -2,9 +2,31 @@`
`2`	`2`	`import { spawnSync } from "node:child_process";`
`3`	`3`	`import os from "node:os";`
`4`	`4`	`import path from "node:path";`
	`5`	`+import { pathToFileURL } from "node:url";`
`5`	`6`	`import { forceFreePort, type PortProcess } from "../src/cli/ports.js";`
`6`	`7`	`import { resolveGatewayPort } from "../src/config/config.js";`
`7`	`8`
	`9`	`+function usage(): string {`
	`10`	`+return [`
	`11`	`+"Usage: node --import tsx scripts/test-force.ts",`
	`12`	`+"",`
	`13`	`+"Clears the configured OpenClaw gateway port, then runs the local test suite.",`
	`14`	`+"",`
	`15`	`+"Options:",`
	`16`	`+" -h, --help Show this help.",`
	`17`	`+].join("\n");`
	`18`	`+}`
	`19`	`+`
	`20`	`+function parseArgs(argv: readonly string[]): { help: boolean } {`
	`21`	`+for (const arg of argv) {`
	`22`	`+if (arg === "--help" \|\| arg === "-h") {`
	`23`	`+return { help: true };`
	`24`	`+}`
	`25`	+throw new Error(`unknown argument: ${arg}\n\n${usage()}`);
	`26`	`+}`
	`27`	`+return { help: false };`
	`28`	`+}`
	`29`	`+`
`8`	`30`	`function killGatewayListeners(port: number): PortProcess[] {`
`9`	`31`	`try {`
`10`	`32`	`const killed = forceFreePort(port);`
`@@ -42,7 +64,13 @@ function runTests() {`
`42`	`64`	`process.exit(result.status ?? 1);`
`43`	`65`	`}`
`44`	`66`
`45`		`-function main() {`
	`67`	`+export function main(argv: readonly string[] = process.argv.slice(2)) {`
	`68`	`+const args = parseArgs(argv);`
	`69`	`+if (args.help) {`
	`70`	`+console.log(usage());`
	`71`	`+return;`
	`72`	`+}`
	`73`	`+`
`46`	`74`	`const port = resolveGatewayPort(undefined, process.env);`
`47`	`75`
`48`	`76`	console.log(`🧹 test:force - clearing gateway on port ${port}`);
`@@ -55,4 +83,11 @@ function main() {`
`55`	`83`	`runTests();`
`56`	`84`	`}`
`57`	`85`
`58`		`-main();`
	`86`	`+if (process.argv[1] && import.meta.url === pathToFileURL(process.argv[1]).href) {`
	`87`	`+try {`
	`88`	`+main();`
	`89`	`+} catch (error) {`
	`90`	`+console.error(error instanceof Error ? error.message : String(error));`
	`91`	`+process.exit(2);`
	`92`	`+}`
	`93`	`+}`

Original file line number	Diff line number	Diff line change
`@@ -408,6 +408,7 @@ const TOOLING_SOURCE_TEST_TARGETS = new Map([`
`408`	`408`	`["scripts/docker-e2e-timings.mjs", ["test/scripts/docker-e2e-helper-cli.test.ts"]],`
`409`	`409`	`["scripts/kova-ci-summary.mjs", ["test/scripts/kova-ci-summary.test.ts"]],`
`410`	`410`	`["scripts/test-extension-batch.mjs", ["test/scripts/test-extension.test.ts"]],`
	`411`	`+["scripts/test-force.ts", ["test/scripts/test-force.test.ts"]],`
`411`	`412`	`["scripts/zai-fallback-repro.ts", ["test/scripts/zai-fallback-repro.test.ts"]],`
`412`	`413`	`["scripts/lib/extension-test-plan.mjs", ["test/scripts/test-extension.test.ts"]],`
`413`	`414`	`["scripts/lib/vitest-batch-runner.mjs", ["test/scripts/test-extension.test.ts"]],`

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,31 @@`
	`1`	`+import { spawnSync } from "node:child_process";`
	`2`	`+import { describe, expect, it } from "vitest";`
	`3`	`+`
	`4`	`+function runTestForce(...args: string[]) {`
	`5`	`+return spawnSync(process.execPath, ["--import", "tsx", "scripts/test-force.ts", ...args], {`
	`6`	`+cwd: process.cwd(),`
	`7`	`+encoding: "utf8",`
	`8`	`+});`
	`9`	`+}`
	`10`	`+`
	`11`	`+describe("scripts/test-force.ts", () => {`
	`12`	`+it("prints help without clearing ports or running tests", () => {`
	`13`	`+const result = runTestForce("--help");`
	`14`	`+`
	`15`	`+expect(result.status).toBe(0);`
	`16`	`+expect(result.stderr).toBe("");`
	`17`	`+expect(result.stdout).toContain("Usage: node --import tsx scripts/test-force.ts");`
	`18`	`+expect(result.stdout).not.toContain("test:force - clearing gateway");`
	`19`	`+expect(result.stdout).not.toContain("running pnpm test");`
	`20`	`+});`
	`21`	`+`
	`22`	`+it("rejects unknown arguments before clearing ports or running tests", () => {`
	`23`	`+const result = runTestForce("--bogus");`
	`24`	`+`
	`25`	`+expect(result.status).toBe(2);`
	`26`	`+expect(result.stderr).toContain("unknown argument: --bogus");`
	`27`	`+expect(result.stderr).toContain("Usage: node --import tsx scripts/test-force.ts");`
	`28`	`+expect(result.stdout).not.toContain("test:force - clearing gateway");`
	`29`	`+expect(result.stdout).not.toContain("running pnpm test");`
	`30`	`+});`
	`31`	`+});`

Original file line number	Diff line number	Diff line change
`@@ -300,6 +300,13 @@ describe("scripts/test-projects changed-target routing", () => {`
`300`	`300`	`});`
`301`	`301`	`});`
`302`	`302`
	`303`	`+it("keeps force-test runner edits on its safe CLI tests", () => {`
	`304`	`+expect(resolveChangedTestTargetPlan(["scripts/test-force.ts"])).toEqual({`
	`305`	`+mode: "targets",`
	`306`	`+targets: ["test/scripts/test-force.test.ts"],`
	`307`	`+});`
	`308`	`+});`
	`309`	`+`
`303`	`310`	`it("keeps sharded oxlint runner edits on oxlint runner tests", () => {`
`304`	`311`	`expect(resolveChangedTestTargetPlan(["scripts/run-oxlint-shards.mjs"])).toEqual({`
`305`	`312`	`mode: "targets",`
`@@ -312,6 +319,7 @@ describe("scripts/test-projects changed-target routing", () => {`
`312`	`319`	`findUnmatchedExplicitTestTargets([`
`313`	`320`	`"scripts/check-dynamic-import-warts.mjs",`
`314`	`321`	`"scripts/run-oxlint-shards.mjs",`
	`322`	`+"scripts/test-force.ts",`
`315`	`323`	`"scripts/tsdown-build.mjs",`
`316`	`324`	`]),`
`317`	`325`	`).toEqual([]);`
`@@ -320,9 +328,16 @@ describe("scripts/test-projects changed-target routing", () => {`
`320`	`328`	`buildVitestRunPlans([`
`321`	`329`	`"scripts/check-dynamic-import-warts.mjs",`
`322`	`330`	`"scripts/run-oxlint-shards.mjs",`
	`331`	`+"scripts/test-force.ts",`
`323`	`332`	`"scripts/tsdown-build.mjs",`
`324`	`333`	`]),`
`325`	`334`	`).toEqual([`
	`335`	`+{`
	`336`	`+config: "test/vitest/vitest.unit-fast.config.ts",`
	`337`	`+forwardedArgs: [],`
	`338`	`+includePatterns: ["test/scripts/test-force.test.ts"],`
	`339`	`+watchMode: false,`
	`340`	`+},`
`326`	`341`	`{`
`327`	`342`	`config: "test/vitest/vitest.tooling.config.ts",`
`328`	`343`	`forwardedArgs: [],`

此内容由惯性聚合(RSS阅读器)自动聚合整理，仅供阅读参考。原文来自 — 版权归原作者所有。

推荐订阅源

Recent Commits to openclaw:main