admin/Moltbot
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

test: remove low-value dm-fallback slash access edge case

Browse Source
This commit is contained in:
Peter Steinberger
2026-02-16 09:21:39 +00:00
parent e30900f93e
commit 78976d3f6f
1 changed files with 0 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
src/slack/monitor/slash.test.ts
View File

@@ -352,28 +352,6 @@ describe("slack slash commands access groups", () => {
resetPolicyHarness(harness);
});
it("still treats D-prefixed channel ids as DMs when lookup fails", async () => {
harness.ctx.allowFrom = [];
harness.ctx.resolveChannelName = async () => ({});
const { respond } = await runSlashHandler({
commands: harness.commands,
command: {
channel_id: "D123",
channel_name: "notdirectmessage",
},
});
expect(dispatchMock).toHaveBeenCalledTimes(1);
expect(respond).not.toHaveBeenCalledWith(
expect.objectContaining({ text: "You are not authorized to use this command." }),
);
const dispatchArg = dispatchMock.mock.calls[0]?.[0] as {
ctx?: { CommandAuthorized?: boolean };
};
expect(dispatchArg?.ctx?.CommandAuthorized).toBe(false);
});
it("enforces access-group gating when lookup fails for private channels", async () => {
harness.ctx.allowFrom = [];
harness.ctx.resolveChannelName = async () => ({});