test: guard agent three-hit mock calls · openclaw/openclaw@d2ae92a

Recent Commits to openclaw:main

steipete · 2026-05-12 · via Recent Commits to openclaw:main

Original file line number	Diff line number	Diff line change
`@@ -185,10 +185,10 @@ describe("startAcpSpawnParentStreamRelay", () => {`
`185`	`185`	`expect(progressOptions?.contextKey).toBe("acp-spawn:run-cron:progress");`
`186`	`186`	`expect(progressOptions?.sessionKey).toBe("global");`
`187`	`187`	`expect(progressOptions?.trusted).toBe(false);`
`188`		`-const heartbeatOptions = requestHeartbeatMock.mock.calls[0]?.[0];`
	`188`	`+const heartbeatOptions = requestHeartbeatMock.mock.calls.at(0)?.[0];`
`189`	`189`	`expect(heartbeatOptions?.agentId).toBe("ops");`
`190`	`190`	`expect(heartbeatOptions?.reason).toBe("acp:spawn:stream");`
`191`		`-expect(requestHeartbeatMock.mock.calls[0]?.[0]).not.toHaveProperty("sessionKey");`
	`191`	`+expect(requestHeartbeatMock.mock.calls.at(0)?.[0]).not.toHaveProperty("sessionKey");`
`192`	`192`	`relay.dispose();`
`193`	`193`	`});`
`194`	`194`
`@@ -421,7 +421,7 @@ describe("startAcpSpawnParentStreamRelay", () => {`
`421`	`421`	`sessionKey: "agent:codex:acp:child-1",`
`422`	`422`	`});`
`423`	`423`	`expect(resolveSessionFilePathMock).toHaveBeenCalledTimes(1);`
`424`		`-const [sessionId, entry, options] = resolveSessionFilePathMock.mock.calls[0] as [`
	`424`	`+const [sessionId, entry, options] = resolveSessionFilePathMock.mock.calls.at(0) as [`
`425`	`425`	`string,`
`426`	`426`	`{ sessionId?: unknown },`
`427`	`427`	`{ storePath?: unknown },`

Original file line number	Diff line number	Diff line change
`@@ -364,7 +364,7 @@ function expectSessionPatchFields(expected: Record<string, unknown>): void {`
`364`	`364`	`}`
`365`	`365`
`366`	`366`	`function expectInitializeSessionFields(expected: Record<string, unknown>): Record<string, unknown> {`
`367`		`-return expectRecordFields(hoisted.initializeSessionMock.mock.calls[0]?.[0], expected);`
	`367`	`+return expectRecordFields(hoisted.initializeSessionMock.mock.calls.at(0)?.[0], expected);`
`368`	`368`	`}`
`369`	`369`
`370`	`370`	`function expectBindingCallFields(expected: {`
`@@ -1942,7 +1942,7 @@ describe("spawnAcpDirect", () => {`
`1942`	`1942`	`expect(accepted.streamLogPath).toBeUndefined();`
`1943`	`1943`	`expect(hoisted.startAcpSpawnParentStreamRelayMock).not.toHaveBeenCalled();`
`1944`	`1944`	`if (expectTranscriptPersistence) {`
`1945`		`-expectRecordFields(hoisted.resolveSessionTranscriptFileMock.mock.calls[0]?.[0], {`
	`1945`	`+expectRecordFields(hoisted.resolveSessionTranscriptFileMock.mock.calls.at(0)?.[0], {`
`1946`	`1946`	`sessionId: "sess-123",`
`1947`	`1947`	`storePath: "/tmp/codex-sessions.json",`
`1948`	`1948`	`agentId: "codex",`
`@@ -2162,7 +2162,7 @@ describe("spawnAcpDirect", () => {`
`2162`	`2162`	`expect(relayRuns).toContain(agentCall?.params?.idempotencyKey);`
`2163`	`2163`	`expect(relayRuns).toContain(accepted.runId);`
`2164`	`2164`	`const streamPathInput = expectRecordFields(`
`2165`		`-hoisted.resolveAcpSpawnStreamLogPathMock.mock.calls[0]?.[0],`
	`2165`	`+hoisted.resolveAcpSpawnStreamLogPathMock.mock.calls.at(0)?.[0],`
`2166`	`2166`	`{},`
`2167`	`2167`	`);`
`2168`	`2168`	`expect(streamPathInput.childSessionKey).toMatch(/^agent:codex:acp:/);`

Original file line number	Diff line number	Diff line change
`@@ -64,15 +64,15 @@ function expectExecTarget(`
`64`	`64`	`}`
`65`	`65`
`66`	`66`	`function requireSystemEventCall(): [string, Record<string, unknown>] {`
`67`		`-const call = enqueueSystemEventMock.mock.calls[0];`
	`67`	`+const call = enqueueSystemEventMock.mock.calls.at(0);`
`68`	`68`	`if (!call) {`
`69`	`69`	`throw new Error("expected system event call");`
`70`	`70`	`}`
`71`	`71`	`return call as [string, Record<string, unknown>];`
`72`	`72`	`}`
`73`	`73`
`74`	`74`	`function requireHeartbeatCall(): Record<string, unknown> {`
`75`		`-const call = requestHeartbeatMock.mock.calls[0];`
	`75`	`+const call = requestHeartbeatMock.mock.calls.at(0);`
`76`	`76`	`if (!call) {`
`77`	`77`	`throw new Error("expected heartbeat call");`
`78`	`78`	`}`
`@@ -537,7 +537,7 @@ describe("emitExecSystemEvent", () => {`
`537`	`537`	`expect(heartbeatParams.agentId).toBe("ops");`
`538`	`538`	`expect(heartbeatParams.coalesceMs).toBe(0);`
`539`	`539`	`expect(heartbeatParams.reason).toBe("exec-event");`
`540`		`-expect(requestHeartbeatMock.mock.calls[0]?.[0]).not.toHaveProperty("sessionKey");`
	`540`	`+expect(requestHeartbeatMock.mock.calls.at(0)?.[0]).not.toHaveProperty("sessionKey");`
`541`	`541`	`});`
`542`	`542`
`543`	`543`	`it("keeps wake unscoped for non-agent session keys", () => {`

Original file line number	Diff line number	Diff line change
`@@ -78,7 +78,7 @@ describe("executePreparedCliRun supervisor output capture", () => {`
`78`	`78`	`});`
`79`	`79`
`80`	`80`	`const result = await executePreparedCliRun(buildPreparedCliRunContext({ output: "text" }));`
`81`		`-const spawnInput = supervisorSpawnMock.mock.calls[0]?.[0] as SupervisorSpawnInput;`
	`81`	`+const spawnInput = supervisorSpawnMock.mock.calls.at(0)?.[0] as SupervisorSpawnInput;`
`82`	`82`
`83`	`83`	`expect(spawnInput.captureOutput).toBe(false);`
`84`	`84`	`expect(result.rawText).toBe(fullText);`
`@@ -107,7 +107,7 @@ describe("executePreparedCliRun supervisor output capture", () => {`
`107`	`107`	`await expect(`
`108`	`108`	`executePreparedCliRun(buildPreparedCliRunContext({ output: "text" })),`
`109`	`109`	`).rejects.toThrow("CLI stdout exceeded");`
`110`		`-const spawnInput = supervisorSpawnMock.mock.calls[0]?.[0] as SupervisorSpawnInput;`
	`110`	`+const spawnInput = supervisorSpawnMock.mock.calls.at(0)?.[0] as SupervisorSpawnInput;`
`111`	`111`
`112`	`112`	`expect(spawnInput.captureOutput).toBe(false);`
`113`	`113`	`});`
`@@ -281,7 +281,7 @@ describe("executePreparedCliRun supervisor output capture", () => {`
`281`	`281`	`const result = await executePreparedCliRun(`
`282`	`282`	`buildPreparedCliRunContext({ output: "jsonl", provider: "claude-cli" }),`
`283`	`283`	`);`
`284`		`-const spawnInput = supervisorSpawnMock.mock.calls[0]?.[0] as SupervisorSpawnInput;`
	`284`	`+const spawnInput = supervisorSpawnMock.mock.calls.at(0)?.[0] as SupervisorSpawnInput;`
`285`	`285`
`286`	`286`	`expect(spawnInput.captureOutput).toBe(false);`
`287`	`287`	`expect(result.text).toBe("Hello world");`

Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ describe("resolveVisibleModelCatalog", () => {`
`32`	`32`	`});`
`33`	`33`
`34`	`34`	`expect(createProviderAuthCheckerMock).toHaveBeenCalledTimes(1);`
`35`		`-const checkerOptions = createProviderAuthCheckerMock.mock.calls[0]?.[0];`
	`35`	`+const checkerOptions = createProviderAuthCheckerMock.mock.calls.at(0)?.[0];`
`36`	`36`	`expect(checkerOptions?.cfg).toBe(cfg);`
`37`	`37`	`expect(checkerOptions?.allowPluginSyntheticAuth).toBe(false);`
`38`	`38`	`expect(checkerOptions?.discoverExternalCliAuth).toBe(false);`
`@@ -70,7 +70,7 @@ describe("resolveVisibleModelCatalog", () => {`
`70`	`70`	`});`
`71`	`71`
`72`	`72`	`expect(createProviderAuthCheckerMock).toHaveBeenCalledTimes(1);`
`73`		`-expect(createProviderAuthCheckerMock.mock.calls[0]?.[0]).toMatchObject({`
	`73`	`+expect(createProviderAuthCheckerMock.mock.calls.at(0)?.[0]).toMatchObject({`
`74`	`74`	`allowPluginSyntheticAuth: true,`
`75`	`75`	`discoverExternalCliAuth: true,`
`76`	`76`	`});`
`@@ -105,7 +105,7 @@ describe("resolveVisibleModelCatalog", () => {`
`105`	`105`	`});`
`106`	`106`
`107`	`107`	`expect(createProviderAuthCheckerMock).toHaveBeenCalledTimes(1);`
`108`		`-expect(createProviderAuthCheckerMock.mock.calls[0]?.[0]).toMatchObject({`
	`108`	`+expect(createProviderAuthCheckerMock.mock.calls.at(0)?.[0]).toMatchObject({`
`109`	`109`	`allowPluginSyntheticAuth: true,`
`110`	`110`	`discoverExternalCliAuth: true,`
`111`	`111`	`});`

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

推荐订阅源

Recent Commits to openclaw:main