romtuck/openclaw
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

test: update conflicting test to reflect API requirement

Browse Source 
The API requires tool calls to have arguments field present (even if empty).
Updated test to verify arguments IS synthesized when missing.
This commit is contained in:
Mimi 2026-01-29 00:34:48 +08:00
parent bc96c27942
commit 422fee7643
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/agents/pi-embedded-helpers.sanitize-session-messages-images.keeps-tool-call-tool-result-ids-unchanged.test.ts
View File

@ -86,7 +86,7 @@ describe("sanitizeSessionMessagesImages", () => {
expect(toolResult.role).toBe("toolResult");
expect(toolResult.toolCallId).toBe("call123fc456");
});
it("does not synthesize tool call input when missing", async () => {
it("synthesizes tool call arguments when missing (API requires it)", async () => {
const input = [
{
role: "assistant",
@ -98,7 +98,8 @@ describe("sanitizeSessionMessagesImages", () => {
const assistant = out[0] as { content?: Array<Record<string, unknown>> };
const toolCall = assistant.content?.find((b) => b.type === "toolCall");
expect(toolCall).toBeTruthy();
expect("input" in (toolCall ?? {})).toBe(false);
expect("arguments" in (toolCall ?? {})).toBe(false);
// API requires arguments field to be present (even if empty)
expect("arguments" in (toolCall ?? {})).toBe(true);
expect(toolCall?.arguments).toEqual({});
});
});