vultr/openclaw
1
0
Fork 0
mirror of https://github.com/openclaw/openclaw.git synced 2026-05-05 15:20:23 +00:00
Code Issues Packages Projects Releases Wiki Activity

test: remove redundant subagent allowlist conversions

Browse Source
This commit is contained in:
Peter Steinberger
2026-04-10 22:03:59 +01:00
parent 1a83731ea1
commit a736b6eede
1 changed files with 7 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
src/agents/openclaw-tools.subagents.sessions-spawn.allowlist.test.ts
View File

@@ -182,9 +182,7 @@ describe("subagent spawn allowlist + sandbox guards", () => {
}); });
const result = await spawn({ agentId: "research" }); const result = await spawn({ agentId: "research" });
expect(result).toMatchObject({ status: "forbidden" }); expect(result).toMatchObject({ status: "forbidden" });
expect(String(result.error ?? "")).toContain( expect(result.error ?? "").toContain("Sandboxed sessions cannot spawn unsandboxed subagents.");
"Sandboxed sessions cannot spawn unsandboxed subagents.",
);
expect(hoisted.callGatewayMock).not.toHaveBeenCalled(); expect(hoisted.callGatewayMock).not.toHaveBeenCalled();
}); });
@@ -199,7 +197,7 @@ describe("subagent spawn allowlist + sandbox guards", () => {
}); });
const result = await spawn({ agentId: "research", sandbox: "require" }); const result = await spawn({ agentId: "research", sandbox: "require" });
expect(result).toMatchObject({ status: "forbidden" }); expect(result).toMatchObject({ status: "forbidden" });
expect(String(result.error ?? "")).toContain('sandbox="require"'); expect(result.error ?? "").toContain('sandbox="require"');
expect(hoisted.callGatewayMock).not.toHaveBeenCalled(); expect(hoisted.callGatewayMock).not.toHaveBeenCalled();
}); });
@@ -212,7 +210,7 @@ describe("subagent spawn allowlist + sandbox guards", () => {
}); });
const result = await spawn({}); const result = await spawn({});
expect(result).toMatchObject({ status: "forbidden" }); expect(result).toMatchObject({ status: "forbidden" });
expect(String(result.error ?? "")).toContain("sessions_spawn requires explicit agentId"); expect(result.error ?? "").toContain("sessions_spawn requires explicit agentId");
expect(hoisted.callGatewayMock).not.toHaveBeenCalled(); expect(hoisted.callGatewayMock).not.toHaveBeenCalled();
}); });
@@ -248,8 +246,8 @@ describe("subagent spawn allowlist + sandbox guards", () => {
}); });
const result = await spawn({ agentId: "Agent not found: xyz" }); const result = await spawn({ agentId: "Agent not found: xyz" });
expect(result).toMatchObject({ status: "error" }); expect(result).toMatchObject({ status: "error" });
expect(String(result.error ?? "")).toContain("Invalid agentId"); expect(result.error ?? "").toContain("Invalid agentId");
expect(String(result.error ?? "")).toContain("agents_list"); expect(result.error ?? "").toContain("agents_list");
expect(hoisted.callGatewayMock).not.toHaveBeenCalled(); expect(hoisted.callGatewayMock).not.toHaveBeenCalled();
}); });
@@ -261,7 +259,7 @@ describe("subagent spawn allowlist + sandbox guards", () => {
}); });
const result = await spawn({ agentId: "../../../etc/passwd" }); const result = await spawn({ agentId: "../../../etc/passwd" });
expect(result).toMatchObject({ status: "error" }); expect(result).toMatchObject({ status: "error" });
expect(String(result.error ?? "")).toContain("Invalid agentId"); expect(result.error ?? "").toContain("Invalid agentId");
expect(hoisted.callGatewayMock).not.toHaveBeenCalled(); expect(hoisted.callGatewayMock).not.toHaveBeenCalled();
}); });
@@ -273,7 +271,7 @@ describe("subagent spawn allowlist + sandbox guards", () => {
}); });
const result = await spawn({ agentId: "a".repeat(65) }); const result = await spawn({ agentId: "a".repeat(65) });
expect(result).toMatchObject({ status: "error" }); expect(result).toMatchObject({ status: "error" });
expect(String(result.error ?? "")).toContain("Invalid agentId"); expect(result.error ?? "").toContain("Invalid agentId");
expect(hoisted.callGatewayMock).not.toHaveBeenCalled(); expect(hoisted.callGatewayMock).not.toHaveBeenCalled();
}); });