fix(qa-lab): report mock cleanup startup failures · openc...

Recent Commits to openclaw:main

vincentkoc · 2026-06-20 · via Recent Commits to openclaw:main

Original file line number	Diff line number	Diff line change
`@@ -193,6 +193,45 @@ describe("startQaLiveLaneGateway", () => {`
`193`	`193`	`expect(gatewayStop).toHaveBeenCalledTimes(1);`
`194`	`194`	`});`
`195`	`195`
	`196`	`+it("stops the mock server when gateway startup fails", async () => {`
	`197`	`+startQaGatewayChild.mockRejectedValueOnce(new Error("gateway failed"));`
	`198`	`+`
	`199`	`+await expect(`
	`200`	`+startQaLiveLaneGateway({`
	`201`	`+repoRoot: "/tmp/openclaw-repo",`
	`202`	`+transport: createStubTransport(),`
	`203`	`+transportBaseUrl: "http://127.0.0.1:43123",`
	`204`	`+providerMode: "mock-openai",`
	`205`	`+primaryModel: "mock-openai/gpt-5.5",`
	`206`	`+alternateModel: "mock-openai/gpt-5.5-alt",`
	`207`	`+controlUiEnabled: false,`
	`208`	`+}),`
	`209`	`+).rejects.toThrow("gateway failed");`
	`210`	`+`
	`211`	`+expect(mockStop).toHaveBeenCalledTimes(1);`
	`212`	`+});`
	`213`	`+`
	`214`	`+it("reports mock cleanup failures after gateway startup failures", async () => {`
	`215`	`+startQaGatewayChild.mockRejectedValueOnce(new Error("gateway failed"));`
	`216`	`+mockStop.mockRejectedValueOnce(new Error("mock stuck"));`
	`217`	`+`
	`218`	`+await expect(`
	`219`	`+startQaLiveLaneGateway({`
	`220`	`+repoRoot: "/tmp/openclaw-repo",`
	`221`	`+transport: createStubTransport(),`
	`222`	`+transportBaseUrl: "http://127.0.0.1:43123",`
	`223`	`+providerMode: "mock-openai",`
	`224`	`+primaryModel: "mock-openai/gpt-5.5",`
	`225`	`+alternateModel: "mock-openai/gpt-5.5-alt",`
	`226`	`+controlUiEnabled: false,`
	`227`	`+}),`
	`228`	`+).rejects.toThrow(`
	`229`	`+"failed to start QA live lane gateway:\ngateway startup failed: gateway failed\nmock provider stop failed: mock stuck",`
	`230`	`+);`
	`231`	`+`
	`232`	`+expect(mockStop).toHaveBeenCalledTimes(1);`
	`233`	`+});`
	`234`	`+`
`196`	`235`	`it("still stops the mock server when gateway shutdown fails", async () => {`
`197`	`236`	`gatewayStop.mockRejectedValueOnce(new Error("gateway down"));`
`198`	`237`	`const harness = await startQaLiveLaneGateway({`

Original file line number	Diff line number	Diff line change
`@@ -128,7 +128,18 @@ export async function startQaLiveLaneGateway(params: {`
`128`	`128`	`},`
`129`	`129`	`};`
`130`	`130`	`} catch (error) {`
`131`		`-await mock?.stop().catch(() => {});`
	`131`	`+if (mock) {`
	`132`	`+try {`
	`133`	`+await mock.stop();`
	`134`	`+} catch (cleanupError) {`
	`135`	`+const errors: string[] = [];`
	`136`	`+appendLiveLaneIssue(errors, "gateway startup failed", error);`
	`137`	`+appendLiveLaneIssue(errors, "mock provider stop failed", cleanupError);`
	`138`	+throw new Error(`failed to start QA live lane gateway:\n${errors.join("\n")}`, {
	`139`	`+cause: cleanupError,`
	`140`	`+});`
	`141`	`+}`
	`142`	`+}`
`132`	`143`	`throw error;`
`133`	`144`	`}`
`134`	`145`	`}`

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

推荐订阅源

Recent Commits to openclaw:main