vultr/openclaw
1
0
Fork 0
mirror of https://github.com/openclaw/openclaw.git synced 2026-04-10 16:51:13 +00:00
Code Issues Packages Projects Releases Wiki Activity

test: align Slack approval fallback coverage

Browse Source
This commit is contained in:
Gustavo Madeira Santana
2026-04-01 12:21:43 -04:00
parent 4d1cfb983e
commit 463d39d686
1 changed files with 0 additions and 35 deletions
Download Patch File Download Diff File Expand all files Collapse all files

35
extensions/slack/src/monitor/exec-approvals.test.ts
View File

@@ -103,41 +103,6 @@ describe("SlackExecApprovalHandler", () => {
);
});
it("accepts approvers inferred from allowFrom without requiring explicit execApprovals.approvers", async () => {
const app = buildApp();
const cfg = {
channels: {
slack: {
botToken: "xoxb-test",
appToken: "xapp-test",
allowFrom: ["U123APPROVER"],
execApprovals: {
enabled: true,
target: "dm",
},
},
},
} as OpenClawConfig;
const handler = new SlackExecApprovalHandler({
app,
accountId: "default",
config: cfg.channels!.slack!.execApprovals!,
cfg,
});
await handler.handleApprovalRequested(buildRequest());
expect(sendMessageSlackMock).toHaveBeenCalledTimes(2);
expect(sendMessageSlackMock).toHaveBeenNthCalledWith(
2,
"user:U123APPROVER",
expect.stringContaining("Exec approval required"),
expect.objectContaining({
accountId: "default",
}),
);
});
it("does not post a redundant DM redirect notice when the origin is already the approver DM", async () => {
const app = buildApp();
const handler = new SlackExecApprovalHandler({