romtuck/openclaw
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

test(gateway): update exec-approval test for two-phase response

Browse Source 
Add assertion for immediate 'accepted' response before final decision.
This commit is contained in:
rshirali 2026-01-28 14:54:13 +01:00
parent b1d409738b
commit 0ccb5584ee
1 changed files with 8 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/gateway/server-methods/exec-approval.test.ts
View File

@ -82,6 +82,13 @@ describe("exec approval handlers", () => {
const id = (requested?.payload as { id?: string })?.id ?? ""; const id = (requested?.payload as { id?: string })?.id ?? "";
expect(id).not.toBe(""); expect(id).not.toBe("");
// First response should be "accepted" (registration confirmation)
expect(respond).toHaveBeenCalledWith(
true,
expect.objectContaining({ status: "accepted", id }),
undefined,
);
const resolveRespond = vi.fn(); const resolveRespond = vi.fn();
await handlers["exec.approval.resolve"]({ await handlers["exec.approval.resolve"]({
params: { id, decision: "allow-once" }, params: { id, decision: "allow-once" },
@ -97,6 +104,7 @@ describe("exec approval handlers", () => {
await requestPromise; await requestPromise;
expect(resolveRespond).toHaveBeenCalledWith(true, { ok: true }, undefined); expect(resolveRespond).toHaveBeenCalledWith(true, { ok: true }, undefined);
// Second response should contain the decision
expect(respond).toHaveBeenCalledWith( expect(respond).toHaveBeenCalledWith(
true, true,
expect.objectContaining({ id, decision: "allow-once" }), expect.objectContaining({ id, decision: "allow-once" }),