test: stabilize doctor + sandbox tests

This commit is contained in:
Peter Steinberger
2026-01-11 04:45:04 +01:00
parent 7343597075
commit 30348e41c6
2 changed files with 35 additions and 45 deletions

View File

@@ -130,41 +130,33 @@ describe("sandbox skill mirroring", () => {
return { context, workspaceDir };
};
it(
"copies skills into the sandbox when workspaceAccess is ro",
async () => {
const { context } = await runContext("ro");
it("copies skills into the sandbox when workspaceAccess is ro", async () => {
const { context } = await runContext("ro");
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);
it(
"copies skills into the sandbox when workspaceAccess is none",
async () => {
const { context } = await runContext("none");
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);
});

View File

@@ -635,17 +635,15 @@ describe("doctor", () => {
await doctorCommand(runtime, { nonInteractive: true });
expect(
note.mock.calls.some(
([message, title]) => {
if (title !== "Sandbox") return false;
if (typeof message !== "string") return false;
const normalized = message.replace(/\s+/g, " ");
return (
normalized.includes('agents.list (id "work") sandbox docker') &&
normalized.includes('scope resolves to "shared"')
);
},
),
note.mock.calls.some(([message, title]) => {
if (title !== "Sandbox") return false;
if (typeof message !== "string") return false;
const normalized = message.replace(/\s+/g, " ");
return (
normalized.includes('agents.list (id "work") sandbox docker') &&
normalized.includes('scope resolves to "shared"')
);
}),
).toBe(true);
});