github/openclaw
1
1
Fork 0
mirror of https://github.com/openclaw/openclaw.git synced 2026-02-19 18:39:20 -05:00
Code Issues Packages Projects Releases Wiki Activity

Tests: use agents_list in tools invoke gateway test

Browse Source 
What: swap sessions_list for agents_list in /tools/invoke tests
Why: avoid nested gateway calls that can hang under CI; still validates tool invocation + allowlist
Tests: not run (CI should cover)
This commit is contained in:
Josh Palmer
2026-01-30 18:17:12 +01:00
parent 09be5d45d5
commit 6522de6ce0
1 changed files with 21 additions and 21 deletions
Download Patch File Download Diff File Expand all files Collapse all files

42
src/gateway/tools-invoke-http.test.ts
View File

@@ -25,13 +25,13 @@ const resolveGatewayToken = (): string => {
describe("POST /tools/invoke", () => {
it("invokes a tool and returns {ok:true,result}", async () => {
// Allow the sessions_list tool for main agent.
// Allow the agents_list tool for main agent.
testState.agentsConfig = {
list: [
{
id: "main",
tools: {
allow: ["sessions_list"],
allow: ["agents_list"],
},
},
],
@@ -46,7 +46,7 @@ describe("POST /tools/invoke", () => {
const res = await fetch(`http://127.0.0.1:${port}/tools/invoke`, {
method: "POST",
headers: { "content-type": "application/json", authorization: `Bearer ${token}` },
body: JSON.stringify({ tool: "sessions_list", action: "json", args: {}, sessionKey: "main" }),
body: JSON.stringify({ tool: "agents_list", action: "json", args: {}, sessionKey: "main" }),
});
expect(res.status).toBe(200);
@@ -63,10 +63,10 @@ describe("POST /tools/invoke", () => {
list: [{ id: "main" }],
} as any;
// minimal profile does NOT include sessions_list, but alsoAllow should.
// minimal profile does NOT include agents_list, but alsoAllow should.
const { writeConfigFile } = await import("../config/config.js");
await writeConfigFile({
tools: { profile: "minimal", alsoAllow: ["sessions_list"] },
tools: { profile: "minimal", alsoAllow: ["agents_list"] },
} as any);
const port = await getFreePort();
@@ -76,7 +76,7 @@ describe("POST /tools/invoke", () => {
const res = await fetch(`http://127.0.0.1:${port}/tools/invoke`, {
method: "POST",
headers: { "content-type": "application/json", authorization: `Bearer ${token}` },
body: JSON.stringify({ tool: "sessions_list", action: "json", args: {}, sessionKey: "main" }),
body: JSON.stringify({ tool: "agents_list", action: "json", args: {}, sessionKey: "main" }),
});
expect(res.status).toBe(200);
@@ -94,7 +94,7 @@ describe("POST /tools/invoke", () => {
await fs.mkdir(path.dirname(CONFIG_PATH), { recursive: true });
await fs.writeFile(
CONFIG_PATH,
JSON.stringify({ tools: { alsoAllow: ["sessions_list"] } }, null, 2),
JSON.stringify({ tools: { alsoAllow: ["agents_list"] } }, null, 2),
"utf-8",
);
@@ -105,7 +105,7 @@ describe("POST /tools/invoke", () => {
const res = await fetch(`http://127.0.0.1:${port}/tools/invoke`, {
method: "POST",
headers: { "content-type": "application/json", authorization: `Bearer ${token}` },
body: JSON.stringify({ tool: "sessions_list", action: "json", args: {}, sessionKey: "main" }),
body: JSON.stringify({ tool: "agents_list", action: "json", args: {}, sessionKey: "main" }),
});
expect(res.status).toBe(200);
@@ -121,7 +121,7 @@ describe("POST /tools/invoke", () => {
{
id: "main",
tools: {
allow: ["sessions_list"],
allow: ["agents_list"],
},
},
],
@@ -139,7 +139,7 @@ describe("POST /tools/invoke", () => {
"content-type": "application/json",
authorization: "Bearer secret",
},
body: JSON.stringify({ tool: "sessions_list", action: "json", args: {}, sessionKey: "main" }),
body: JSON.stringify({ tool: "agents_list", action: "json", args: {}, sessionKey: "main" }),
});
expect(res.status).toBe(200);
@@ -171,7 +171,7 @@ describe("POST /tools/invoke", () => {
{
id: "main",
tools: {
allow: ["sessions_list"],
allow: ["agents_list"],
},
},
],
@@ -185,7 +185,7 @@ describe("POST /tools/invoke", () => {
method: "POST",
headers: { "content-type": "application/json", authorization: `Bearer ${token}` },
body: JSON.stringify({
tool: "sessions_list",
tool: "agents_list",
action: "json",
args: {},
sessionKey: "main",
@@ -206,7 +206,7 @@ describe("POST /tools/invoke", () => {
{
id: "main",
tools: {
allow: ["sessions_list"],
allow: ["agents_list"],
},
},
],
@@ -221,7 +221,7 @@ describe("POST /tools/invoke", () => {
const res = await fetch(`http://127.0.0.1:${port}/tools/invoke`, {
method: "POST",
headers: { "content-type": "application/json" },
body: JSON.stringify({ tool: "sessions_list", action: "json", args: {}, sessionKey: "main" }),
body: JSON.stringify({ tool: "agents_list", action: "json", args: {}, sessionKey: "main" }),
});
expect(res.status).toBe(401);
@@ -235,7 +235,7 @@ describe("POST /tools/invoke", () => {
{
id: "main",
tools: {
deny: ["sessions_list"],
deny: ["agents_list"],
},
},
],
@@ -248,7 +248,7 @@ describe("POST /tools/invoke", () => {
const res = await fetch(`http://127.0.0.1:${port}/tools/invoke`, {
method: "POST",
headers: { "content-type": "application/json", authorization: `Bearer ${token}` },
body: JSON.stringify({ tool: "sessions_list", action: "json", args: {}, sessionKey: "main" }),
body: JSON.stringify({ tool: "agents_list", action: "json", args: {}, sessionKey: "main" }),
});
expect(res.status).toBe(404);
@@ -262,7 +262,7 @@ describe("POST /tools/invoke", () => {
{
id: "main",
tools: {
allow: ["sessions_list"],
allow: ["agents_list"],
},
},
],
@@ -280,7 +280,7 @@ describe("POST /tools/invoke", () => {
const res = await fetch(`http://127.0.0.1:${port}/tools/invoke`, {
method: "POST",
headers: { "content-type": "application/json", authorization: `Bearer ${token}` },
body: JSON.stringify({ tool: "sessions_list", action: "json", args: {}, sessionKey: "main" }),
body: JSON.stringify({ tool: "agents_list", action: "json", args: {}, sessionKey: "main" }),
});
expect(res.status).toBe(404);
@@ -294,14 +294,14 @@ describe("POST /tools/invoke", () => {
{
id: "main",
tools: {
deny: ["sessions_list"],
deny: ["agents_list"],
},
},
{
id: "ops",
default: true,
tools: {
allow: ["sessions_list"],
allow: ["agents_list"],
},
},
],
@@ -311,7 +311,7 @@ describe("POST /tools/invoke", () => {
const port = await getFreePort();
const server = await startGatewayServer(port, { bind: "loopback" });
const payload = { tool: "sessions_list", action: "json", args: {} };
const payload = { tool: "agents_list", action: "json", args: {} };
const token = resolveGatewayToken();
const resDefault = await fetch(`http://127.0.0.1:${port}/tools/invoke`, {