test: guard agent tool call assertions · openclaw/openclaw@8ee024d

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
`@@ -105,7 +105,7 @@ describe("cron tool", () => {`
`105`	`105`	`},`
`106`	`106`	`});`
`107`	`107`
`108`		`-const call = callGatewayMock.mock.calls[0]?.[0] as {`
	`108`	`+const call = callGatewayMock.mock.calls.at(0)?.[0] as {`
`109`	`109`	`params?: { delivery?: TestDelivery };`
`110`	`110`	`};`
`111`	`111`	`return call?.params?.delivery;`
`@@ -534,7 +534,7 @@ describe("cron tool", () => {`
`534`	`534`	`},`
`535`	`535`	`});`
`536`	`536`
`537`		`-const call = callGatewayMock.mock.calls[0]?.[0] as {`
	`537`	`+const call = callGatewayMock.mock.calls.at(0)?.[0] as {`
`538`	`538`	`params?: { agentId?: unknown };`
`539`	`539`	`};`
`540`	`540`	`expect(call?.params?.agentId).toBeNull();`
`@@ -721,7 +721,7 @@ describe("cron tool", () => {`
`721`	`721`	`},`
`722`	`722`	`});`
`723`	`723`
`724`		`-const call = callGatewayMock.mock.calls[0]?.[0] as {`
	`724`	`+const call = callGatewayMock.mock.calls.at(0)?.[0] as {`
`725`	`725`	`method?: string;`
`726`	`726`	`params?: { agentId?: string \| null };`
`727`	`727`	`};`
`@@ -1066,7 +1066,7 @@ describe("cron tool", () => {`
`1066`	`1066`	`name: "flat-name-should-be-ignored",`
`1067`	`1067`	`});`
`1068`	`1068`
`1069`		`-const call = callGatewayMock.mock.calls[0]?.[0] as {`
	`1069`	`+const call = callGatewayMock.mock.calls.at(0)?.[0] as {`
`1070`	`1070`	`params?: { name?: string; payload?: { text?: string } };`
`1071`	`1071`	`};`
`1072`	`1072`	`expect(call?.params?.name).toBe("nested-job");`

Original file line number	Diff line number	Diff line change
`@@ -668,10 +668,8 @@ describe("sessions_spawn tool", () => {`
`668`	`668`	`expect(spawnArgs.task).toBe("resume prior work");`
`669`	`669`	`const spawnContext = mockCallArg(hoisted.spawnSubagentDirectMock, 0, 1, "spawnSubagentDirect");`
`670`	`670`	`expect(spawnContext.agentSessionKey).toBe("agent:main:main");`
`671`		`-expect(hoisted.spawnSubagentDirectMock.mock.calls[0]?.[0]).not.toHaveProperty(`
`672`		`-"resumeSessionId",`
`673`		`-);`
`674`		`-expect(hoisted.spawnSubagentDirectMock.mock.calls[0]?.[0]).not.toHaveProperty("streamTo");`
	`671`	`+expect(spawnArgs).not.toHaveProperty("resumeSessionId");`
	`672`	`+expect(spawnArgs).not.toHaveProperty("streamTo");`
`675`	`673`	`expect(hoisted.spawnAcpDirectMock).not.toHaveBeenCalled();`
`676`	`674`	`});`
`677`	`675`
`@@ -717,10 +715,8 @@ describe("sessions_spawn tool", () => {`
`717`	`715`	`expect(hoisted.spawnAcpDirectMock).not.toHaveBeenCalled();`
`718`	`716`	`const spawnArgs = mockCallArg(hoisted.spawnSubagentDirectMock, 0, 0, "spawnSubagentDirect");`
`719`	`717`	`expect(spawnArgs.task).toBe("analyze file");`
`720`		`-expect(hoisted.spawnSubagentDirectMock.mock.calls[0]?.[0]).not.toHaveProperty(`
`721`		`-"resumeSessionId",`
`722`		`-);`
`723`		`-expect(hoisted.spawnSubagentDirectMock.mock.calls[0]?.[0]).not.toHaveProperty("streamTo");`
	`718`	`+expect(spawnArgs).not.toHaveProperty("resumeSessionId");`
	`719`	`+expect(spawnArgs).not.toHaveProperty("streamTo");`
`724`	`720`	`});`
`725`	`721`
`726`	`722`	`it('treats model="default" as no explicit model override', async () => {`

Original file line number	Diff line number	Diff line change
`@@ -156,7 +156,7 @@ function resultDetails(result: { details?: unknown }): Record<string, unknown> {`
`156`	`156`	`}`
`157`	`157`
`158`	`158`	`function firstMockCallArg(mock: { mock: { calls: unknown[][] } }): unknown {`
`159`		`-const firstCall = mock.mock.calls[0];`
	`159`	`+const firstCall = mock.mock.calls.at(0);`
`160`	`160`	`if (!firstCall) {`
`161`	`161`	`throw new Error("Expected first mock call");`
`162`	`162`	`}`
`@@ -399,7 +399,7 @@ describe("createVideoGenerateTool", () => {`
`399`	`399`	`});`
`400`	`400`
`401`	`401`	`expect((firstMockCallArg(generateSpy) as { timeoutMs?: number }).timeoutMs).toBe(180_000);`
`402`		`-expect((generateSpy.mock.calls[1]?.[0] as { timeoutMs?: number }).timeoutMs).toBe(12_345);`
	`402`	`+expect((generateSpy.mock.calls.at(1)?.[0] as { timeoutMs?: number }).timeoutMs).toBe(12_345);`
`403`	`403`	`expect(resultDetails(defaultResult).timeoutMs).toBe(180_000);`
`404`	`404`	`expect(resultDetails(overrideResult).timeoutMs).toBe(12_345);`
`405`	`405`	`});`
`@@ -1059,7 +1059,7 @@ describe("createVideoGenerateTool", () => {`
`1059`	`1059`	`});`
`1060`	`1060`
`1061`	`1061`	`expect(generateSpy).toHaveBeenCalledTimes(1);`
`1062`		`-const call = generateSpy.mock.calls[0]?.[0] as {`
	`1062`	`+const call = generateSpy.mock.calls.at(0)?.[0] as {`
`1063`	`1063`	`inputImages?: Array<{ role?: string }>;`
`1064`	`1064`	`};`
`1065`	`1065`	`expect(call.inputImages).toHaveLength(2);`
`@@ -1096,7 +1096,7 @@ describe("createVideoGenerateTool", () => {`
`1096`	`1096`	`image: "/tmp/reference.png",`
`1097`	`1097`	`});`
`1098`	`1098`
`1099`		`-const loadCall = vi.mocked(webMedia.loadWebMedia).mock.calls[0];`
	`1099`	`+const loadCall = vi.mocked(webMedia.loadWebMedia).mock.calls.at(0);`
`1100`	`1100`	`expect(loadCall?.[0]).toBe("/tmp/reference.png");`
`1101`	`1101`	`const loadOptions = loadCall?.[1] as { ssrfPolicy?: unknown } \| undefined;`
`1102`	`1102`	`expect(loadOptions?.ssrfPolicy).toEqual({ allowRfc2544BenchmarkRange: true });`

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

推荐订阅源

Recent Commits to openclaw:main