test: guard agent six-hit mock calls · openclaw/openclaw@2b295a5

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
`@@ -310,8 +310,8 @@ async function expectFallsBackToHaiku(params: {`
`310`	`310`
`311`	`311`	`expect(result.result).toBe("ok");`
`312`	`312`	`expect(run).toHaveBeenCalledTimes(2);`
`313`		`-expect(run.mock.calls[1]?.[0]).toBe("anthropic");`
`314`		`-expect(run.mock.calls[1]?.[1]).toBe("claude-haiku-3-5");`
	`313`	`+expect(run.mock.calls.at(1)?.[0]).toBe("anthropic");`
	`314`	`+expect(run.mock.calls.at(1)?.[1]).toBe("claude-haiku-3-5");`
`315`	`315`	`}`
`316`	`316`
`317`	`317`	`function createOverrideFailureRun(params: {`
`@@ -678,7 +678,7 @@ describe("runWithModelFallback", () => {`
`678`	`678`
`679`	`679`	`expect(result.result).toEqual({ payloads: [{ text: "fallback ok" }] });`
`680`	`680`	`expect(run).toHaveBeenCalledTimes(2);`
`681`		`-expect(run.mock.calls[1]).toEqual(["anthropic", "claude-haiku-3-5"]);`
	`681`	`+expect(run.mock.calls.at(1)).toEqual(["anthropic", "claude-haiku-3-5"]);`
`682`	`682`	`expect(result.attempts[0]?.provider).toBe("openai-codex");`
`683`	`683`	`expect(result.attempts[0]?.model).toBe("gpt-5.4");`
`684`	`684`	`expect(result.attempts[0]?.reason).toBe("format");`
`@@ -878,7 +878,7 @@ describe("runWithModelFallback", () => {`
`878`	`878`	`});`
`879`	`879`
`880`	`880`	`expect(onError).toHaveBeenCalledTimes(1);`
`881`		`-const errorCall = requireRecord(onError.mock.calls[0]?.[0], "onError payload");`
	`881`	`+const errorCall = requireRecord(onError.mock.calls.at(0)?.[0], "onError payload");`
`882`	`882`	`expect(errorCall.provider).toBe("openai");`
`883`	`883`	`expect(errorCall.model).toBe("gpt-4.1-mini");`
`884`	`884`	`expect(errorCall.attempt).toBe(1);`
`@@ -1257,7 +1257,7 @@ describe("runWithModelFallback", () => {`
`1257`	`1257`
`1258`	`1258`	`expect(result.result).toBe("ok");`
`1259`	`1259`	`expect(run).toHaveBeenCalledTimes(2);`
`1260`		`-expect(run.mock.calls[1]).toEqual(testCase.expectedFallback);`
	`1260`	`+expect(run.mock.calls.at(1)).toEqual(testCase.expectedFallback);`
`1261`	`1261`	`if (testCase.expectedReason) {`
`1262`	`1262`	`expect(result.attempts).toHaveLength(1);`
`1263`	`1263`	`expect(result.attempts[0]?.reason).toBe(testCase.expectedReason);`
`@@ -1357,7 +1357,7 @@ describe("runWithModelFallback", () => {`
`1357`	`1357`
`1358`	`1358`	`expect(result.result).toBe("ok");`
`1359`	`1359`	`expect(run).toHaveBeenCalledTimes(1);`
`1360`		`-expect(run.mock.calls[0]?.[0]).toBe("openrouter");`
	`1360`	`+expect(run.mock.calls.at(0)?.[0]).toBe("openrouter");`
`1361`	`1361`	`expect(result.attempts).toStrictEqual([]);`
`1362`	`1362`	`});`
`1363`	`1363`

Original file line number	Diff line number	Diff line change
`@@ -144,7 +144,7 @@ describe("compaction lifecycle logging", () => {`
`144`	`144`	`aborted: false,`
`145`	`145`	`});`
`146`	`146`
`147`		`-expect(info.mock.calls[0]?.[0]).toBe("embedded run auto-compaction start");`
	`147`	`+expect(info.mock.calls.at(0)?.[0]).toBe("embedded run auto-compaction start");`
`148`	`148`	`const startMeta = loggedInfoMetaAt(info, 0);`
`149`	`149`	`expect(startMeta.event).toBe("embedded_run_compaction_start");`
`150`	`150`	`expect(startMeta.reason).toBe("threshold");`
`@@ -153,7 +153,7 @@ describe("compaction lifecycle logging", () => {`
`153`	`153`	`"embedded run auto-compaction start: runId=run-test reason=threshold",`
`154`	`154`	`);`
`155`	`155`
`156`		`-expect(info.mock.calls[1]?.[0]).toBe("embedded run auto-compaction complete");`
	`156`	`+expect(info.mock.calls.at(1)?.[0]).toBe("embedded run auto-compaction complete");`
`157`	`157`	`const endMeta = loggedInfoMetaAt(info, 1);`
`158`	`158`	`expect(endMeta.event).toBe("embedded_run_compaction_end");`
`159`	`159`	`expect(endMeta.reason).toBe("threshold");`
`@@ -194,7 +194,7 @@ describe("compaction lifecycle logging", () => {`
`194`	`194`	`aborted: false,`
`195`	`195`	`});`
`196`	`196`
`197`		`-expect(info.mock.calls[0]?.[0]).toBe("embedded run manual compaction start");`
	`197`	`+expect(info.mock.calls.at(0)?.[0]).toBe("embedded run manual compaction start");`
`198`	`198`	`const startMeta = loggedInfoMetaAt(info, 0);`
`199`	`199`	`expect(startMeta.event).toBe("embedded_run_compaction_start");`
`200`	`200`	`expect(startMeta.reason).toBe("manual");`
`@@ -203,7 +203,7 @@ describe("compaction lifecycle logging", () => {`
`203`	`203`	`"embedded run manual compaction start: runId=run-test reason=manual",`
`204`	`204`	`);`
`205`	`205`
`206`		`-expect(info.mock.calls[1]?.[0]).toBe("embedded run manual compaction incomplete");`
	`206`	`+expect(info.mock.calls.at(1)?.[0]).toBe("embedded run manual compaction incomplete");`
`207`	`207`	`const endMeta = loggedInfoMetaAt(info, 1);`
`208`	`208`	`expect(endMeta.event).toBe("embedded_run_compaction_end");`
`209`	`209`	`expect(endMeta.reason).toBe("manual");`
`@@ -242,7 +242,7 @@ describe("compaction lifecycle logging", () => {`
`242`	`242`	`aborted: false,`
`243`	`243`	`});`
`244`	`244`
`245`		`-expect(info.mock.calls[0]?.[0]).toBe("embedded run auto-compaction start");`
	`245`	`+expect(info.mock.calls.at(0)?.[0]).toBe("embedded run auto-compaction start");`
`246`	`246`	`const startMeta = loggedInfoMetaAt(info, 0);`
`247`	`247`	`expect(startMeta.event).toBe("embedded_run_compaction_start");`
`248`	`248`	`expect(startMeta.reason).toBe("threshold");`
`@@ -251,7 +251,7 @@ describe("compaction lifecycle logging", () => {`
`251`	`251`	`"embedded run auto-compaction start: runId=run-test reason=threshold",`
`252`	`252`	`);`
`253`	`253`
`254`		`-expect(info.mock.calls[1]?.[0]).toBe("embedded run auto-compaction complete");`
	`254`	`+expect(info.mock.calls.at(1)?.[0]).toBe("embedded run auto-compaction complete");`
`255`	`255`	`const endMeta = loggedInfoMetaAt(info, 1);`
`256`	`256`	`expect(endMeta.event).toBe("embedded_run_compaction_end");`
`257`	`257`	`expect(endMeta.reason).toBe("threshold");`

Original file line number	Diff line number	Diff line change
`@@ -57,8 +57,8 @@ async function handleAgentEndAndReadWarnMeta(ctx: EmbeddedPiSubscribeContext) {`
`57`	`57`
`58`	`58`	`const warn = vi.mocked(ctx.log.warn);`
`59`	`59`	`expect(warn).toHaveBeenCalledTimes(1);`
`60`		`-expect(warn.mock.calls[0]?.[0]).toBe("embedded run agent end");`
`61`		`-return readRecord(warn.mock.calls[0]?.[1]);`
	`60`	`+expect(warn.mock.calls.at(0)?.[0]).toBe("embedded run agent end");`
	`61`	`+return readRecord(warn.mock.calls.at(0)?.[1]);`
`62`	`62`	`}`
`63`	`63`
`64`	`64`	`function readRecord(value: unknown): Record<string, unknown> {`
`@@ -138,7 +138,7 @@ describe("handleAgentEnd", () => {`
`138`	`138`	`await handleAgentEnd(ctx);`
`139`	`139`
`140`	`140`	`const warn = vi.mocked(ctx.log.warn);`
`141`		`-const meta = readRecord(warn.mock.calls[0]?.[1]);`
	`141`	`+const meta = readRecord(warn.mock.calls.at(0)?.[1]);`
`142`	`142`	`expect(meta.consoleMessage).toBe(`
`143`	`143`	`"embedded run agent end: runId=run-1 isError=true model=claude sonnet 4 provider=anthropic]8;;https://evil.test error=LLM request failed: connection refused by the provider endpoint. rawError=connection refused",`
`144`	`144`	`);`
`@@ -163,7 +163,7 @@ describe("handleAgentEnd", () => {`
`163`	`163`	`await handleAgentEnd(ctx);`
`164`	`164`
`165`	`165`	`const warn = vi.mocked(ctx.log.warn);`
`166`		`-const meta = readRecord(warn.mock.calls[0]?.[1]);`
	`166`	`+const meta = readRecord(warn.mock.calls.at(0)?.[1]);`
`167`	`167`	`expect(meta.event).toBe("embedded_run_agent_end");`
`168`	`168`	`expect(meta.error).toBe("x-api-key: ***");`
`169`	`169`	`expect(meta.rawErrorPreview).toBe("x-api-key: ***");`
`@@ -189,7 +189,7 @@ describe("handleAgentEnd", () => {`
`189`	`189`
`190`	`190`	`await handleAgentEnd(ctx);`
`191`	`191`
`192`		`-const meta = readRecord(vi.mocked(ctx.log.warn).mock.calls[0]?.[1]);`
	`192`	`+const meta = readRecord(vi.mocked(ctx.log.warn).mock.calls.at(0)?.[1]);`
`193`	`193`	`expect(meta.failoverReason).toBe("auth");`
`194`	`194`	`expect(meta.providerRuntimeFailureKind).toBe("auth_scope");`
`195`	`195`	`expect(meta.httpCode).toBe("401");`
`@@ -207,7 +207,7 @@ describe("handleAgentEnd", () => {`
`207`	`207`
`208`	`208`	`await handleAgentEnd(ctx);`
`209`	`209`
`210`		`-const warnMeta = vi.mocked(ctx.log.warn).mock.calls[0]?.[1];`
	`210`	`+const warnMeta = vi.mocked(ctx.log.warn).mock.calls.at(0)?.[1];`
`211`	`211`	`const meta = readRecord(warnMeta);`
`212`	`212`	`expect(meta.providerRuntimeFailureKind).toBe("auth_html_403");`
`213`	`213`	`expect(meta.rawErrorPreview).toBe("403 <!DOCTYPE html><html><body>Access denied</body></html>");`

Original file line number	Diff line number	Diff line change
`@@ -94,7 +94,7 @@ function expectSingleBlockReplyText(params: {`
`94`	`94`	`text: string;`
`95`	`95`	`}) {`
`96`	`96`	`expect(params.onBlockReply).toHaveBeenCalledTimes(1);`
`97`		`-expect(params.onBlockReply.mock.calls[0]?.[0]?.text).toBe(params.text);`
	`97`	`+expect(params.onBlockReply.mock.calls.at(0)?.[0]?.text).toBe(params.text);`
`98`	`98`	`expect(params.subscription.assistantTexts).toEqual([params.text]);`
`99`	`99`	`}`
`100`	`100`
`@@ -110,8 +110,8 @@ describe("subscribeEmbeddedPiSession", () => {`
`110`	`110`	`await vi.waitFor(() => {`
`111`	`111`	`expect(onBlockReply).toHaveBeenCalledTimes(1);`
`112`	`112`	`});`
`113`		`-const payload = onBlockReply.mock.calls[0][0];`
`114`		`-expect(payload.text).toBe("Hello block");`
	`113`	`+const payload = onBlockReply.mock.calls.at(0)?.[0];`
	`114`	`+expect(payload?.text).toBe("Hello block");`
`115`	`115`	`expect(subscription.assistantTexts).toEqual(["Hello block"]);`
`116`	`116`
`117`	`117`	`const assistantMessage = {`
`@@ -147,7 +147,7 @@ describe("subscribeEmbeddedPiSession", () => {`
`147`	`147`	`await vi.waitFor(() => {`
`148`	`148`	`expect(onBlockReply).toHaveBeenCalledTimes(1);`
`149`	`149`	`});`
`150`		`-expect(onBlockReply.mock.calls[0]?.[0]?.text).toBe("Final visible reply.");`
	`150`	`+expect(onBlockReply.mock.calls.at(0)?.[0]?.text).toBe("Final visible reply.");`
`151`	`151`	`expect(subscription.assistantTexts).toEqual(["Final visible reply."]);`
`152`	`152`	`});`
`153`	`153`
`@@ -198,7 +198,7 @@ describe("subscribeEmbeddedPiSession", () => {`
`198`	`198`	`await Promise.resolve();`
`199`	`199`
`200`	`200`	`expect(onBlockReply).toHaveBeenCalledTimes(1);`
`201`		`-expect(onBlockReply.mock.calls[0]?.[0]?.text).toBe("Legacy answer");`
	`201`	`+expect(onBlockReply.mock.calls.at(0)?.[0]?.text).toBe("Legacy answer");`
`202`	`202`	`expect(subscription.assistantTexts).toEqual(["Legacy answer"]);`
`203`	`203`
`204`	`204`	`emit({`
`@@ -260,7 +260,7 @@ describe("subscribeEmbeddedPiSession", () => {`
`260`	`260`	`await Promise.resolve();`
`261`	`261`
`262`	`262`	`expect(onBlockReply).toHaveBeenCalledTimes(1);`
`263`		`-expect(onBlockReply.mock.calls[0]?.[0]?.text).toBe("Hello world");`
	`263`	`+expect(onBlockReply.mock.calls.at(0)?.[0]?.text).toBe("Hello world");`
`264`	`264`	`expect(subscription.assistantTexts).toEqual(["Hello world"]);`
`265`	`265`	`});`
`266`	`266`
`@@ -286,7 +286,7 @@ describe("subscribeEmbeddedPiSession", () => {`
`286`	`286`	`await Promise.resolve();`
`287`	`287`
`288`	`288`	`expect(onBlockReply).toHaveBeenCalledTimes(1);`
`289`		`-expect(onBlockReply.mock.calls[0]?.[0]?.text).toBe("Done.");`
	`289`	`+expect(onBlockReply.mock.calls.at(0)?.[0]?.text).toBe("Done.");`
`290`	`290`	`expect(subscription.assistantTexts).toEqual(["Done."]);`
`291`	`291`	`});`
`292`	`292`

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

推荐订阅源

Recent Commits to openclaw:main