test: tighten agent tool assertions · openclaw/openclaw@d...

Recent Commits to openclaw:main

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

Original file line number	Diff line number	Diff line change
`@@ -197,9 +197,7 @@ describe("TOOL_POLICY_CONFORMANCE", () => {`
`197`	`197`
`198`	`198`	`it("is JSON-serializable", () => {`
`199`	`199`	`const serialized = JSON.stringify(TOOL_POLICY_CONFORMANCE);`
`200`		`-expect(JSON.parse(serialized)).toMatchObject({`
`201`		`-toolGroups: TOOL_GROUPS,`
`202`		`-});`
	`200`	`+expect(JSON.parse(serialized)).toEqual({ toolGroups: TOOL_GROUPS });`
`203`	`201`	`});`
`204`	`202`	`});`
`205`	`203`

Original file line number	Diff line number	Diff line change
`@@ -401,11 +401,12 @@ describe("resolveEffectiveToolInventory", () => {`
`401`	`401`	`modelId: "grok-test",`
`402`	`402`	`});`
`403`	`403`
`404`		`-expect(createToolsMock).toHaveBeenCalledWith(`
`405`		`-expect.objectContaining({`
`406`		`-allowGatewaySubagentBinding: true,`
`407`		`-modelCompat: { supportsTools: true, nativeWebSearchTool: true },`
`408`		`-}),`
`409`		`-);`
	`404`	`+expect(createToolsMock).toHaveBeenCalledTimes(1);`
	`405`	`+const createToolsOptions = createToolsMock.mock.calls[0]?.[0];`
	`406`	`+expect(createToolsOptions?.allowGatewaySubagentBinding).toBe(true);`
	`407`	`+expect(createToolsOptions?.modelCompat).toEqual({`
	`408`	`+supportsTools: true,`
	`409`	`+nativeWebSearchTool: true,`
	`410`	`+});`
`410`	`411`	`});`
`411`	`412`	`});`

Original file line number	Diff line number	Diff line change
`@@ -58,19 +58,15 @@ function expectAllowedApply(`
`58`	`58`
`59`	`59`	`describe("gateway config mutation guard coverage", () => {`
`60`	`60`	`it("keeps a narrow allowlist of agent-tunable config paths", () => {`
`61`		`-expect(ALLOWED_GATEWAY_CONFIG_PATHS_FOR_TEST).toEqual(`
`62`		`-expect.arrayContaining([`
`63`		`-"agents.defaults.systemPromptOverride",`
`64`		`-"agents.defaults.model",`
`65`		`-"agents.defaults.subagents.thinking",`
`66`		`-"agents.list[].id",`
`67`		`-"agents.list[].model",`
`68`		`-"agents.list[].subagents.thinking",`
`69`		`-"channels.*.requireMention",`
`70`		`-"messages.visibleReplies",`
`71`		`-"messages.groupChat.visibleReplies",`
`72`		`-]),`
`73`		`-);`
	`61`	`+expect(ALLOWED_GATEWAY_CONFIG_PATHS_FOR_TEST).toContain("agents.defaults.systemPromptOverride");`
	`62`	`+expect(ALLOWED_GATEWAY_CONFIG_PATHS_FOR_TEST).toContain("agents.defaults.model");`
	`63`	`+expect(ALLOWED_GATEWAY_CONFIG_PATHS_FOR_TEST).toContain("agents.defaults.subagents.thinking");`
	`64`	`+expect(ALLOWED_GATEWAY_CONFIG_PATHS_FOR_TEST).toContain("agents.list[].id");`
	`65`	`+expect(ALLOWED_GATEWAY_CONFIG_PATHS_FOR_TEST).toContain("agents.list[].model");`
	`66`	`+expect(ALLOWED_GATEWAY_CONFIG_PATHS_FOR_TEST).toContain("agents.list[].subagents.thinking");`
	`67`	`+expect(ALLOWED_GATEWAY_CONFIG_PATHS_FOR_TEST).toContain("channels.*.requireMention");`
	`68`	`+expect(ALLOWED_GATEWAY_CONFIG_PATHS_FOR_TEST).toContain("messages.visibleReplies");`
	`69`	`+expect(ALLOWED_GATEWAY_CONFIG_PATHS_FOR_TEST).toContain("messages.groupChat.visibleReplies");`
`74`	`70`	`});`
`75`	`71`
`76`	`72`	`it("allows documented subagent thinking default edits via config.patch", () => {`

Original file line number	Diff line number	Diff line change
`@@ -182,7 +182,7 @@ describe("createSessionVisibilityGuard", () => {`
`182`	`182`	`a2aPolicy: createAgentToAgentPolicy({} as unknown as OpenClawConfig),`
`183`	`183`	`});`
`184`	`184`
`185`		`-expect(guard.check("agent:codex:acp:child-1")).toMatchObject({ allowed: false });`
	`185`	`+expect(guard.check("agent:codex:acp:child-1").allowed).toBe(false);`
`186`	`186`	`expect(callGateway).not.toHaveBeenCalled();`
`187`	`187`
`188`	`188`	`sessionsResolutionTesting.setDepsForTest();`

Original file line number	Diff line number	Diff line change
`@@ -190,13 +190,12 @@ describe("sessions-list-tool", () => {`
`190`	`190`	`const result = await tool.execute("call-3", {});`
`191`	`191`	`const details = getSessionsListDetails(result);`
`192`	`192`
`193`		`-expect(details.sessions?.[0]).toMatchObject({`
`194`		`-thinkingLevel: "high",`
`195`		`-fastMode: true,`
`196`		`-verboseLevel: "on",`
`197`		`-reasoningLevel: "deep",`
`198`		`-elevatedLevel: "on",`
`199`		`-responseUsage: "full",`
`200`		`-});`
	`193`	`+const session = details.sessions?.[0];`
	`194`	`+expect(session?.thinkingLevel).toBe("high");`
	`195`	`+expect(session?.fastMode).toBe(true);`
	`196`	`+expect(session?.verboseLevel).toBe("on");`
	`197`	`+expect(session?.reasoningLevel).toBe("deep");`
	`198`	`+expect(session?.elevatedLevel).toBe("on");`
	`199`	`+expect(session?.responseUsage).toBe("full");`
`201`	`200`	`});`
`202`	`201`	`});`

Original file line number	Diff line number	Diff line change
`@@ -25,11 +25,8 @@ describe("web_search signal plumbing", () => {`
`25`	`25`
`26`	`26`	`await tool?.execute("call-search", { query: "openclaw" }, controller.signal);`
`27`	`27`
`28`		`-expect(mocks.runWebSearch).toHaveBeenCalledWith(`
`29`		`-expect.objectContaining({`
`30`		`-args: { query: "openclaw" },`
`31`		`-signal: controller.signal,`
`32`		`-}),`
`33`		`-);`
	`28`	`+expect(mocks.runWebSearch).toHaveBeenCalledTimes(1);`
	`29`	`+expect(mocks.runWebSearch.mock.calls[0]?.[0]?.args).toEqual({ query: "openclaw" });`
	`30`	`+expect(mocks.runWebSearch.mock.calls[0]?.[0]?.signal).toBe(controller.signal);`
`34`	`31`	`});`
`35`	`32`	`});`

Original file line number	Diff line number	Diff line change
`@@ -180,11 +180,9 @@ describe("web_fetch extraction fallbacks", () => {`
`180`	`180`
`181`	`181`	`expect(details.text).toMatch(/<<<EXTERNAL_UNTRUSTED_CONTENT id="[a-f0-9]{16}">>>/);`
`182`	`182`	`expect(details.text).toContain("Ignore previous instructions");`
`183`		`-expect(details.externalContent).toMatchObject({`
`184`		`-untrusted: true,`
`185`		`-source: "web_fetch",`
`186`		`-wrapped: true,`
`187`		`-});`
	`183`	`+expect(details.externalContent?.untrusted).toBe(true);`
	`184`	`+expect(details.externalContent?.source).toBe("web_fetch");`
	`185`	`+expect(details.externalContent?.wrapped).toBe(true);`
`188`	`186`	`// contentType is protocol metadata, not user content - should NOT be wrapped`
`189`	`187`	`expect(details.contentType).toBe("text/plain");`
`190`	`188`	`expect(details.length).toBe(details.text?.length);`

Original file line number	Diff line number	Diff line change
`@@ -106,14 +106,13 @@ describe("video generation task status", () => {`
`106`	`106`	`expect(buildVideoGenerationTaskStatusText(activeTask, { duplicateGuard: true })).toContain(`
`107`	`107`	`"Do not call video_generate again for this request.",`
`108`	`108`	`);`
`109`		`-expect(buildVideoGenerationTaskStatusDetails(activeTask)).toMatchObject({`
`110`		`-active: true,`
`111`		`-existingTask: true,`
`112`		`-status: "running",`
`113`		`-taskKind: VIDEO_GENERATION_TASK_KIND,`
`114`		`-provider: "openai",`
`115`		`-progressSummary: "Generating video",`
`116`		`-});`
	`109`	`+const details = buildVideoGenerationTaskStatusDetails(activeTask);`
	`110`	`+expect(details.active).toBe(true);`
	`111`	`+expect(details.existingTask).toBe(true);`
	`112`	`+expect(details.status).toBe("running");`
	`113`	`+expect(details.taskKind).toBe(VIDEO_GENERATION_TASK_KIND);`
	`114`	`+expect(details.provider).toBe("openai");`
	`115`	`+expect(details.progressSummary).toBe("Generating video");`
`117`	`116`	`});`
`118`	`117`
`119`	`118`	`it("builds prompt context for active session work", () => {`

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

推荐订阅源

Recent Commits to openclaw:main