let5see/clawdbot
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

test: use absolute exec path for allowlist

Browse Source
This commit is contained in:
Peter Steinberger
2026-01-22 09:20:38 +00:00
parent 4c8806ad38
commit 646ea6ef0b
1 changed files with 1 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/agents/bash-tools.exec.approval-id.test.ts
View File

@@ -80,10 +80,6 @@ describe("exec approvals", () => {
if (process.platform !== "win32") { if (process.platform !== "win32") {
await fs.chmod(exePath, 0o755); await fs.chmod(exePath, 0o755);
} }
const execEnv: Record<string, string> = { PATH: binDir };
if (process.platform === "win32") {
execEnv.PATHEXT = ".CMD";
}
const approvalsFile = { const approvalsFile = {
version: 1, version: 1,
defaults: { security: "allowlist", ask: "on-miss", askFallback: "deny" }, defaults: { security: "allowlist", ask: "on-miss", askFallback: "deny" },
@@ -117,8 +113,7 @@ describe("exec approvals", () => {
}); });
const result = await tool.execute("call2", { const result = await tool.execute("call2", {
command: `${exeName} --help`, command: `"${exePath}" --help`,
env: execEnv,
}); });
expect(result.details.status).toBe("completed"); expect(result.details.status).toBe("completed");
expect(calls).toContain("exec.approvals.node.get"); expect(calls).toContain("exec.approvals.node.get");