From a8e3e1ea0149277d2f8195ad74988669ad6024fd Mon Sep 17 00:00:00 2001 From: IYODA Atsushi Date: Thu, 19 Feb 2026 12:55:04 +0900 Subject: [PATCH] fix(test): correct browserProvider assertion to match actual behavior MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit When browserProvider is not set, agent-browser skill should NOT resolve. Test assertions were inverted — expected 'Skills not found' but asserted the opposite. --- src/tools/delegate-task/tools.test.ts | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/src/tools/delegate-task/tools.test.ts b/src/tools/delegate-task/tools.test.ts index 8108dce8..8ac91db6 100644 --- a/src/tools/delegate-task/tools.test.ts +++ b/src/tools/delegate-task/tools.test.ts @@ -2679,11 +2679,9 @@ describe("sisyphus-task", () => { toolContext ) - // then - skill content should be injected - expect(result).not.toContain("Skills not found") - expect(promptBody).toBeDefined() - expect(promptBody.system).toContain("") - expect(String(promptBody.system).startsWith("")).toBe(false) + // then - agent-browser skill should NOT resolve without browserProvider + expect(result).toContain("Skills not found") + expect(result).toContain("agent-browser") }) })