test: guard agent eight-hit mock calls · openclaw/opencla...

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
`@@ -761,7 +761,7 @@ describe("agentCommand – LiveSessionModelSwitchError retry", () => {`
`761`	`761`
`762`	`762`	`expect(state.runWithModelFallbackMock).toHaveBeenCalledTimes(2);`
`763`	`763`
`764`		`-const secondCall = state.runWithModelFallbackMock.mock.calls[1]?.[0] as`
	`764`	`+const secondCall = state.runWithModelFallbackMock.mock.calls.at(1)?.[0] as`
`765`	`765`	`\| FallbackRunnerParams`
`766`	`766`	`\| undefined;`
`767`	`767`	`expect(secondCall?.provider).toBe("openai");`
`@@ -912,8 +912,8 @@ describe("agentCommand – LiveSessionModelSwitchError retry", () => {`
`912`	`912`	`await runBasicAgentCommand();`
`913`	`913`
`914`	`914`	`expect(state.trajectoryRecordEventMock).toHaveBeenCalledTimes(1);`
`915`		`-expect(state.trajectoryRecordEventMock.mock.calls[0]?.[0]).toBe("model.fallback_step");`
`916`		`-expectRecordFields(state.trajectoryRecordEventMock.mock.calls[0]?.[1], {`
	`915`	`+expect(state.trajectoryRecordEventMock.mock.calls.at(0)?.[0]).toBe("model.fallback_step");`
	`916`	`+expectRecordFields(state.trajectoryRecordEventMock.mock.calls.at(0)?.[1], {`
`917`	`917`	`fallbackStepType: "fallback_step",`
`918`	`918`	`fallbackStepFromModel: "ollama/llama3",`
`919`	`919`	`fallbackStepToModel: "openai/gpt-5.4",`
`@@ -1125,7 +1125,7 @@ describe("agentCommand – LiveSessionModelSwitchError retry", () => {`
`1125`	`1125`
`1126`	`1126`	`await runBasicAgentCommand();`
`1127`	`1127`
`1128`		`-const attemptParams = state.runAgentAttemptMock.mock.calls[0]?.[0] as`
	`1128`	`+const attemptParams = state.runAgentAttemptMock.mock.calls.at(0)?.[0] as`
`1129`	`1129`	`\| { skillsSnapshot?: Record<string, unknown> }`
`1130`	`1130`	`\| undefined;`
`1131`	`1131`	`expectRecordFields(attemptParams?.skillsSnapshot, {`
`@@ -1176,13 +1176,13 @@ describe("agentCommand – LiveSessionModelSwitchError retry", () => {`
`1176`	`1176`	`code: "empty_result",`
`1177`	`1177`	`});`
`1178`	`1178`	`expect(state.runAgentAttemptMock).toHaveBeenCalledTimes(2);`
`1179`		`-expectRecordFields(state.runAgentAttemptMock.mock.calls[1]?.[0], {`
	`1179`	`+expectRecordFields(state.runAgentAttemptMock.mock.calls.at(1)?.[0], {`
`1180`	`1180`	`providerOverride: "openai",`
`1181`	`1181`	`modelOverride: "gpt-5.4",`
`1182`	`1182`	`isFallbackRetry: true,`
`1183`	`1183`	`});`
`1184`	`1184`	`const deliveryParams = requireRecord(`
`1185`		`-state.deliverAgentCommandResultMock.mock.calls[0]?.[0],`
	`1185`	`+state.deliverAgentCommandResultMock.mock.calls.at(0)?.[0],`
`1186`	`1186`	`"delivery params",`
`1187`	`1187`	`);`
`1188`	`1188`	`const result = requireRecord(deliveryParams.result, "delivery result");`
`@@ -1261,15 +1261,15 @@ describe("agentCommand – LiveSessionModelSwitchError retry", () => {`
`1261`	`1261`	`});`
`1262`	`1262`
`1263`	`1263`	`expect(state.acpRunTurnMock).toHaveBeenCalledTimes(1);`
`1264`		`-const runTurnParams = state.acpRunTurnMock.mock.calls[0]?.[0] as { text?: string };`
	`1264`	`+const runTurnParams = state.acpRunTurnMock.mock.calls.at(0)?.[0] as { text?: string };`
`1265`	`1265`	`expect(runTurnParams.text).toContain("A background task completed.");`
`1266`	`1266`	`expect(runTurnParams.text).toContain("inspect ACP delivery");`
`1267`	`1267`	`expect(runTurnParams.text).toContain("child output");`
`1268`	`1268`	`expect(runTurnParams.text).not.toContain(INTERNAL_RUNTIME_CONTEXT_BEGIN);`
`1269`	`1269`	`expect(runTurnParams.text).not.toContain(INTERNAL_RUNTIME_CONTEXT_END);`
`1270`	`1270`
`1271`	`1271`	`expect(state.persistAcpTurnTranscriptMock).toHaveBeenCalledTimes(1);`
`1272`		`-const transcriptParams = state.persistAcpTurnTranscriptMock.mock.calls[0]?.[0] as {`
	`1272`	`+const transcriptParams = state.persistAcpTurnTranscriptMock.mock.calls.at(0)?.[0] as {`
`1273`	`1273`	`body?: string;`
`1274`	`1274`	`transcriptBody?: string;`
`1275`	`1275`	`};`

Original file line number	Diff line number	Diff line change
`@@ -71,7 +71,7 @@ vi.mock("./pi-embedded.js", () => ({`
`71`	`71`
`72`	`72`	`function capturedEmbeddedRunParams(): RunEmbeddedPiAgentParams {`
`73`	`73`	`expect(runEmbeddedPiAgentMock).toHaveBeenCalledTimes(1);`
`74`		`-const call = runEmbeddedPiAgentMock.mock.calls[0];`
	`74`	`+const call = runEmbeddedPiAgentMock.mock.calls.at(0);`
`75`	`75`	`if (!call) {`
`76`	`76`	`throw new Error("expected runEmbeddedPiAgent to be called");`
`77`	`77`	`}`
`@@ -261,7 +261,7 @@ describe("Auth profile runtime contract - Pi and CLI adapter", () => {`
`261`	`261`	`});`
`262`	`262`
`263`	`263`	`expect(runCliAgentMock).toHaveBeenCalledTimes(1);`
`264`		`-expect(runCliAgentMock.mock.calls[0]?.[0]?.authProfileId).toBe(`
	`264`	`+expect(runCliAgentMock.mock.calls.at(0)?.[0]?.authProfileId).toBe(`
`265`	`265`	`expectedForwardedAuthProfile({`
`266`	`266`	`provider: AUTH_PROFILE_RUNTIME_CONTRACT.codexCliProvider,`
`267`	`267`	`authProfileProvider: AUTH_PROFILE_RUNTIME_CONTRACT.openAiCodexProvider,`
`@@ -281,7 +281,7 @@ describe("Auth profile runtime contract - Pi and CLI adapter", () => {`
`281`	`281`	`});`
`282`	`282`
`283`	`283`	`expect(runCliAgentMock).toHaveBeenCalledTimes(1);`
`284`		`-expect(runCliAgentMock.mock.calls[0]?.[0]?.authProfileId).toBe(`
	`284`	`+expect(runCliAgentMock.mock.calls.at(0)?.[0]?.authProfileId).toBe(`
`285`	`285`	`expectedForwardedAuthProfile({`
`286`	`286`	`provider: AUTH_PROFILE_RUNTIME_CONTRACT.codexCliProvider,`
`287`	`287`	`authProfileProvider: AUTH_PROFILE_RUNTIME_CONTRACT.codexCliProvider,`
`@@ -301,7 +301,7 @@ describe("Auth profile runtime contract - Pi and CLI adapter", () => {`
`301`	`301`	`});`
`302`	`302`
`303`	`303`	`expect(runCliAgentMock).toHaveBeenCalledTimes(1);`
`304`		`-expect(runCliAgentMock.mock.calls[0]?.[0]?.authProfileId).toBeUndefined();`
	`304`	`+expect(runCliAgentMock.mock.calls.at(0)?.[0]?.authProfileId).toBeUndefined();`
`305`	`305`	`});`
`306`	`306`
`307`	`307`	`it("does not leak an OpenAI Codex auth profile into an unrelated CLI provider", async () => {`
`@@ -314,7 +314,7 @@ describe("Auth profile runtime contract - Pi and CLI adapter", () => {`
`314`	`314`	`});`
`315`	`315`
`316`	`316`	`expect(runCliAgentMock).toHaveBeenCalledTimes(1);`
`317`		`-expect(runCliAgentMock.mock.calls[0]?.[0]?.authProfileId).toBeUndefined();`
	`317`	`+expect(runCliAgentMock.mock.calls.at(0)?.[0]?.authProfileId).toBeUndefined();`
`318`	`318`	`});`
`319`	`319`
`320`	`320`	`it("does not let a configured Codex harness leak OpenAI Codex auth into unrelated CLI providers", async () => {`
`@@ -338,7 +338,7 @@ describe("Auth profile runtime contract - Pi and CLI adapter", () => {`
`338`	`338`	`});`
`339`	`339`
`340`	`340`	`expect(runCliAgentMock).toHaveBeenCalledTimes(1);`
`341`		`-expect(runCliAgentMock.mock.calls[0]?.[0]?.authProfileId).toBeUndefined();`
	`341`	`+expect(runCliAgentMock.mock.calls.at(0)?.[0]?.authProfileId).toBeUndefined();`
`342`	`342`	`});`
`343`	`343`
`344`	`344`	`it("forwards an OpenAI Codex auth profile through the embedded Pi path", async () => {`
`@@ -351,7 +351,7 @@ describe("Auth profile runtime contract - Pi and CLI adapter", () => {`
`351`	`351`	`});`
`352`	`352`
`353`	`353`	`expect(runEmbeddedPiAgentMock).toHaveBeenCalledTimes(1);`
`354`		`-expect(runEmbeddedPiAgentMock.mock.calls[0]?.[0]?.authProfileId).toBe(`
	`354`	`+expect(runEmbeddedPiAgentMock.mock.calls.at(0)?.[0]?.authProfileId).toBe(`
`355`	`355`	`AUTH_PROFILE_RUNTIME_CONTRACT.openAiCodexProfileId,`
`356`	`356`	`);`
`357`	`357`	`});`
`@@ -366,7 +366,7 @@ describe("Auth profile runtime contract - Pi and CLI adapter", () => {`
`366`	`366`	`});`
`367`	`367`
`368`	`368`	`expect(runEmbeddedPiAgentMock).toHaveBeenCalledTimes(1);`
`369`		`-expect(runEmbeddedPiAgentMock.mock.calls[0]?.[0]?.authProfileId).toBe(`
	`369`	`+expect(runEmbeddedPiAgentMock.mock.calls.at(0)?.[0]?.authProfileId).toBe(`
`370`	`370`	`expectedForwardedAuthProfile({`
`371`	`371`	`provider: AUTH_PROFILE_RUNTIME_CONTRACT.openAiCodexProvider,`
`372`	`372`	`authProfileProvider: AUTH_PROFILE_RUNTIME_CONTRACT.codexCliProvider,`

Original file line number	Diff line number	Diff line change
`@@ -246,7 +246,7 @@ function mockActiveTranscript(messages: unknown[]) {`
`246`	`246`	`async function runMathSideQuestionAndCaptureContext() {`
`247`	`247`	`mockDoneAnswer(MATH_ANSWER);`
`248`	`248`	`await runMathSideQuestion();`
`249`		`-const [, context] = streamSimpleMock.mock.calls[0] ?? [];`
	`249`	`+const [, context] = streamSimpleMock.mock.calls.at(0) ?? [];`
`250`	`250`	`return context;`
`251`	`251`	`}`
`252`	`252`
`@@ -474,7 +474,7 @@ describe("runBtwSideQuestion", () => {`
`474`	`474`	`const result = await runSideQuestion();`
`475`	`475`
`476`	`476`	`expect(result).toEqual({ text: "Final answer." });`
`477`		`-const ensureArgs = ensureOpenClawModelsJsonMock.mock.calls[0];`
	`477`	`+const ensureArgs = ensureOpenClawModelsJsonMock.mock.calls.at(0);`
`478`	`478`	`expect(ensureArgs?.[1]).toBe(DEFAULT_AGENT_DIR);`
`479`	`479`	`expect(ensureArgs?.[2]).toEqual({ workspaceDir: "/tmp/workspace" });`
`480`	`480`	`});`
`@@ -523,7 +523,7 @@ describe("runBtwSideQuestion", () => {`
`523`	`523`	`expect(sideQuestionParams.agentId).toBe("main");`
`524`	`524`	`expect(sideQuestionParams.workspaceDir).toBe("/tmp/workspace");`
`525`	`525`	`expect(sideQuestionParams.authProfileId).toBe("openai-codex:work");`
`526`		`-expect(codexSideQuestionMock.mock.calls[0]?.[0].sessionFile).toContain("session-1.jsonl");`
	`526`	`+expect(codexSideQuestionMock.mock.calls.at(0)?.[0].sessionFile).toContain("session-1.jsonl");`
`527`	`527`	`expect(streamSimpleMock).not.toHaveBeenCalled();`
`528`	`528`	`expect(registerProviderStreamForModelMock).not.toHaveBeenCalled();`
`529`	`529`	`});`
`@@ -589,7 +589,7 @@ describe("runBtwSideQuestion", () => {`
`589`	`589`
`590`	`590`	`expect(result).toEqual({ text: "Copilot answer." });`
`591`	`591`	`const runtimeAuthParams = expectRecordFields(`
`592`		`-prepareProviderRuntimeAuthMock.mock.calls[0]?.[0],`
	`592`	`+prepareProviderRuntimeAuthMock.mock.calls.at(0)?.[0],`
`593`	`593`	`{`
`594`	`594`	`provider: "github-copilot",`
`595`	`595`	`workspaceDir: "/tmp/workspace",`
`@@ -603,7 +603,7 @@ describe("runBtwSideQuestion", () => {`
`603`	`603`	`authMode: "token",`
`604`	`604`	`profileId: "profile-1",`
`605`	`605`	`});`
`606`		`-const [streamModel, , streamOptions] = streamSimpleMock.mock.calls[0] ?? [];`
	`606`	`+const [streamModel, , streamOptions] = streamSimpleMock.mock.calls.at(0) ?? [];`
`607`	`607`	`expectRecordFields(streamModel, {`
`608`	`608`	`provider: "github-copilot",`
`609`	`609`	`id: "gpt-5.4",`
`@@ -632,7 +632,7 @@ describe("runBtwSideQuestion", () => {`
`632`	`632`
`633`	`633`	`expect(result).toEqual({ text: "Ollama Cloud answer." });`
`634`	`634`	`const registerParams = expectRecordFields(`
`635`		`-registerProviderStreamForModelMock.mock.calls[0]?.[0],`
	`635`	`+registerProviderStreamForModelMock.mock.calls.at(0)?.[0],`
`636`	`636`	`{`
`637`	`637`	`workspaceDir: "/tmp/workspace",`
`638`	`638`	`},`
`@@ -661,7 +661,7 @@ describe("runBtwSideQuestion", () => {`
`661`	`661`
`662`	`662`	`await runSideQuestion();`
`663`	`663`
`664`		`-const [, , options] = streamSimpleMock.mock.calls[0] ?? [];`
	`664`	`+const [, , options] = streamSimpleMock.mock.calls.at(0) ?? [];`
`665`	`665`	`const onPayload = (options as { onPayload?: (payload: unknown) => void })?.onPayload;`
`666`	`666`	`const payloadWithEmptyTools = { messages: [], tools: [] as unknown[] };`
`667`	`667`
`@@ -712,7 +712,7 @@ describe("runBtwSideQuestion", () => {`
`712`	`712`	`const result = await runSideQuestion({ resolvedThinkLevel: "adaptive" });`
`713`	`713`
`714`	`714`	`expect(result).toEqual({ text: "Final answer." });`
`715`		`-const [, , options] = streamSimpleMock.mock.calls[0] ?? [];`
	`715`	`+const [, , options] = streamSimpleMock.mock.calls.at(0) ?? [];`
`716`	`716`	`expect((options as { reasoning?: unknown } \| undefined)?.reasoning).toBeUndefined();`
`717`	`717`	`});`
`718`	`718`

Original file line number	Diff line number	Diff line change
`@@ -486,7 +486,7 @@ describe("installContextEngineLoopHook", () => {`
`486`	`486`	`await callTransform(agent, messages);`
`487`	`487`
`488`	`488`	`expect(engine.afterTurn).toHaveBeenCalledTimes(1);`
`489`		`-const afterTurnParams = engine.afterTurn.mock.calls[0]?.[0];`
	`489`	`+const afterTurnParams = engine.afterTurn.mock.calls.at(0)?.[0];`
`490`	`490`	`expect(afterTurnParams?.prePromptMessageCount).toBe(1);`
`491`	`491`	`expect(afterTurnParams?.messages).toBe(messages);`
`492`	`492`	`expect(engine.assemble).toHaveBeenCalledTimes(1);`
`@@ -508,7 +508,7 @@ describe("installContextEngineLoopHook", () => {`
`508`	`508`	`await callTransform(agent, messages);`
`509`	`509`
`510`	`510`	`expect(engine.afterTurn).toHaveBeenCalledTimes(1);`
`511`		`-const afterTurnParams = engine.afterTurn.mock.calls[0]?.[0];`
	`511`	`+const afterTurnParams = engine.afterTurn.mock.calls.at(0)?.[0];`
`512`	`512`	`expect(afterTurnParams?.prePromptMessageCount).toBe(1);`
`513`	`513`	`expect(afterTurnParams?.runtimeContext).toEqual({`
`514`	`514`	`provider: "anthropic",`
`@@ -551,7 +551,7 @@ describe("installContextEngineLoopHook", () => {`
`551`	`551`	`await callTransform(agent, withNew);`
`552`	`552`
`553`	`553`	`expect(engine.afterTurn).toHaveBeenCalledTimes(1);`
`554`		`-const afterTurnParams = engine.afterTurn.mock.calls[0]?.[0];`
	`554`	`+const afterTurnParams = engine.afterTurn.mock.calls.at(0)?.[0];`
`555`	`555`	`expect(afterTurnParams?.prePromptMessageCount).toBe(2);`
`556`	`556`	`expect(afterTurnParams?.messages).toBe(withNew);`
`557`	`557`	`expect(engine.assemble).toHaveBeenCalledTimes(1);`
`@@ -572,8 +572,8 @@ describe("installContextEngineLoopHook", () => {`
`572`	`572`	`await callTransform(agent, batch2);`
`573`	`573`
`574`	`574`	`expect(engine.afterTurn).toHaveBeenCalledTimes(2);`
`575`		`-expect(engine.afterTurn.mock.calls[0]?.[0]?.prePromptMessageCount).toBe(2);`
`576`		`-expect(engine.afterTurn.mock.calls[1]?.[0]?.prePromptMessageCount).toBe(4);`
	`575`	`+expect(engine.afterTurn.mock.calls.at(0)?.[0]?.prePromptMessageCount).toBe(2);`
	`576`	`+expect(engine.afterTurn.mock.calls.at(1)?.[0]?.prePromptMessageCount).toBe(4);`
`577`	`577`	`});`
`578`	`578`
`579`	`579`	`it("reports the latest delivered afterTurn checkpoint", async () => {`
`@@ -761,8 +761,8 @@ describe("installContextEngineLoopHook", () => {`
`761`	`761`	`await callTransform(agent, batch2);`
`762`	`762`
`763`	`763`	`expect(engine.ingestBatch).toHaveBeenCalledTimes(2);`
`764`		`-expect(engine.ingestBatch?.mock.calls[0]?.[0]?.messages).toEqual(batch1.slice(2));`
`765`		`-expect(engine.ingestBatch?.mock.calls[1]?.[0]?.messages).toEqual(batch2.slice(4));`
	`764`	`+expect(engine.ingestBatch?.mock.calls.at(0)?.[0]?.messages).toEqual(batch1.slice(2));`
	`765`	`+expect(engine.ingestBatch?.mock.calls.at(1)?.[0]?.messages).toEqual(batch2.slice(4));`
`766`	`766`	`expect(engine.assemble).toHaveBeenCalledTimes(2);`
`767`	`767`	`});`
`768`	`768`
`@@ -776,7 +776,7 @@ describe("installContextEngineLoopHook", () => {`
`776`	`776`	`await callTransform(agent, messages);`
`777`	`777`
`778`	`778`	`expect(engine.ingest).toHaveBeenCalledTimes(1);`
`779`		`-const ingestParams = engine.ingest.mock.calls[0]?.[0];`
	`779`	`+const ingestParams = engine.ingest.mock.calls.at(0)?.[0];`
`780`	`780`	`expect(ingestParams?.sessionId).toBe(sessionId);`
`781`	`781`	`expect(ingestParams?.sessionKey).toBe(sessionKey);`
`782`	`782`	`expect(ingestParams?.message).toBe(toolResult);`

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

推荐订阅源

Recent Commits to openclaw:main