test: reuse platform spy helper in infra tests · openclaw...

Recent Commits to openclaw:main

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

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,6 @@`
`1`	`1`	`import { afterEach, describe, expect, it, vi } from "vitest";`
`2`	`2`	`import { captureFullEnv } from "../test-utils/env.js";`
	`3`	`+import { mockProcessPlatform } from "../test-utils/vitest-spies.js";`
`3`	`4`	`import { SUPERVISOR_HINT_ENV_VARS } from "./supervisor-markers.js";`
`4`	`5`
`5`	`6`	`const spawnMock = vi.hoisted(() => vi.fn());`
`@@ -30,16 +31,9 @@ import {`
`30`	`31`	`const originalArgv = [...process.argv];`
`31`	`32`	`const originalExecArgv = [...process.execArgv];`
`32`	`33`	`const envSnapshot = captureFullEnv();`
`33`		`-const originalPlatformDescriptor = Object.getOwnPropertyDescriptor(process, "platform");`
`34`	`34`
`35`		`-function setPlatform(platform: string) {`
`36`		`-if (!originalPlatformDescriptor) {`
`37`		`-return;`
`38`		`-}`
`39`		`-Object.defineProperty(process, "platform", {`
`40`		`- ...originalPlatformDescriptor,`
`41`		`-value: platform,`
`42`		`-});`
	`35`	`+function setPlatform(platform: NodeJS.Platform) {`
	`36`	`+mockProcessPlatform(platform);`
`43`	`37`	`}`
`44`	`38`
`45`	`39`	`afterEach(() => {`
`@@ -50,9 +44,7 @@ afterEach(() => {`
`50`	`44`	`triggerOpenClawRestartMock.mockClear();`
`51`	`45`	`isContainerEnvironmentMock.mockReset();`
`52`	`46`	`isContainerEnvironmentMock.mockReturnValue(false);`
`53`		`-if (originalPlatformDescriptor) {`
`54`		`-Object.defineProperty(process, "platform", originalPlatformDescriptor);`
`55`		`-}`
	`47`	`+vi.restoreAllMocks();`
`56`	`48`	`});`
`57`	`49`
`58`	`50`	`function clearSupervisorHints() {`

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,6 @@`
`1`	`1`	`import { afterEach, beforeAll, beforeEach, describe, expect, it, vi } from "vitest";`
`2`	`2`	`import { captureFullEnv } from "../test-utils/env.js";`
	`3`	`+import { mockProcessPlatform } from "../test-utils/vitest-spies.js";`
`3`	`4`
`4`	`5`	`const spawnSyncMock = vi.hoisted(() => vi.fn());`
`5`	`6`	`const resolveLsofCommandSyncMock = vi.hoisted(() => vi.fn());`
`@@ -29,7 +30,6 @@ let triggerOpenClawRestart: typeof import("./restart.js").triggerOpenClawRestart`
`29`	`30`
`30`	`31`	`let currentTimeMs = 0;`
`31`	`32`	`const envSnapshot = captureFullEnv();`
`32`		`-const originalPlatformDescriptor = Object.getOwnPropertyDescriptor(process, "platform");`
`33`	`33`
`34`	`34`	`beforeAll(async () => {`
`35`	`35`	`({ __testing, cleanStaleGatewayProcessesSync, findGatewayPidsOnPortSync } =`
`@@ -55,20 +55,11 @@ afterEach(() => {`
`55`	`55`	`envSnapshot.restore();`
`56`	`56`	`__testing.setSleepSyncOverride(null);`
`57`	`57`	`__testing.setDateNowOverride(null);`
`58`		`-if (originalPlatformDescriptor) {`
`59`		`-Object.defineProperty(process, "platform", originalPlatformDescriptor);`
`60`		`-}`
`61`	`58`	`vi.restoreAllMocks();`
`62`	`59`	`});`
`63`	`60`
`64`	`61`	`function setPlatform(platform: NodeJS.Platform): void {`
`65`		`-if (!originalPlatformDescriptor) {`
`66`		`-return;`
`67`		`-}`
`68`		`-Object.defineProperty(process, "platform", {`
`69`		`- ...originalPlatformDescriptor,`
`70`		`-value: platform,`
`71`		`-});`
	`62`	`+mockProcessPlatform(platform);`
`72`	`63`	`}`
`73`	`64`
`74`	`65`	`function requireFirstSpawnSyncCall(): [unknown, unknown, unknown] {`

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,6 @@`
`1`	`1`	`import { afterEach, beforeAll, beforeEach, describe, expect, it, vi } from "vitest";`
`2`	`2`	`import { captureEnv } from "../test-utils/env.js";`
	`3`	`+import { mockProcessPlatform } from "../test-utils/vitest-spies.js";`
`3`	`4`
`4`	`5`	`const readFileSyncMock = vi.hoisted(() => vi.fn());`
`5`	`6`	`const readFileMock = vi.hoisted(() => vi.fn());`
`@@ -30,13 +31,8 @@ let isWSL2Sync: typeof import("./wsl.js").isWSL2Sync;`
`30`	`31`	`let isWSL: typeof import("./wsl.js").isWSL;`
`31`	`32`	`let resetWSLStateForTests: typeof import("./wsl.js").resetWSLStateForTests;`
`32`	`33`
`33`		`-const originalPlatformDescriptor = Object.getOwnPropertyDescriptor(process, "platform");`
`34`		`-`
`35`	`34`	`function setPlatform(platform: NodeJS.Platform): void {`
`36`		`-Object.defineProperty(process, "platform", {`
`37`		`-value: platform,`
`38`		`-configurable: true,`
`39`		`-});`
	`35`	`+mockProcessPlatform(platform);`
`40`	`36`	`}`
`41`	`37`
`42`	`38`	`describe("wsl detection", () => {`
`@@ -60,9 +56,7 @@ describe("wsl detection", () => {`
`60`	`56`	`afterEach(() => {`
`61`	`57`	`envSnapshot.restore();`
`62`	`58`	`resetWSLStateForTests();`
`63`		`-if (originalPlatformDescriptor) {`
`64`		`-Object.defineProperty(process, "platform", originalPlatformDescriptor);`
`65`		`-}`
	`59`	`+vi.restoreAllMocks();`
`66`	`60`	`});`
`67`	`61`
`68`	`62`	`it.each([`

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

推荐订阅源

Recent Commits to openclaw:main