From 0e38505d3d2b3ed3b9cc6c381e7b5935418e3825 Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Sun, 22 Feb 2026 17:16:09 +0000 Subject: [PATCH] test: collapse duplicate sandbox skill mirroring cases --- src/agents/sandbox-skills.test.ts | 24 ++++++++++-------------- 1 file changed, 10 insertions(+), 14 deletions(-) diff --git a/src/agents/sandbox-skills.test.ts b/src/agents/sandbox-skills.test.ts index 4612fec96a1..d15679b6f3e 100644 --- a/src/agents/sandbox-skills.test.ts +++ b/src/agents/sandbox-skills.test.ts @@ -65,19 +65,15 @@ describe("sandbox skill mirroring", () => { return { context, workspaceDir }; }; - it("copies skills into the sandbox when workspaceAccess is ro", async () => { - const { context } = await runContext("ro"); + it.each(["ro", "none"] as const)( + "copies skills into the sandbox when workspaceAccess is %s", + async (workspaceAccess) => { + const { context } = await runContext(workspaceAccess); - expect(context?.enabled).toBe(true); - const skillPath = path.join(context?.workspaceDir ?? "", "skills", "demo-skill", "SKILL.md"); - await expect(fs.readFile(skillPath, "utf-8")).resolves.toContain("demo-skill"); - }, 20_000); - - it("copies skills into the sandbox when workspaceAccess is none", async () => { - const { context } = await runContext("none"); - - expect(context?.enabled).toBe(true); - const skillPath = path.join(context?.workspaceDir ?? "", "skills", "demo-skill", "SKILL.md"); - await expect(fs.readFile(skillPath, "utf-8")).resolves.toContain("demo-skill"); - }, 20_000); + expect(context?.enabled).toBe(true); + const skillPath = path.join(context?.workspaceDir ?? "", "skills", "demo-skill", "SKILL.md"); + await expect(fs.readFile(skillPath, "utf-8")).resolves.toContain("demo-skill"); + }, + 20_000, + ); });