From d371a4c8c3e1474e5963e5e41b1035e830791faf Mon Sep 17 00:00:00 2001 From: Sergii Kozak Date: Fri, 23 Jan 2026 01:11:54 -0800 Subject: [PATCH] Discord Actions: Update tests for optional config parameter --- src/agents/tools/discord-actions.test.ts | 203 +++++++++++++---------- 1 file changed, 118 insertions(+), 85 deletions(-) diff --git a/src/agents/tools/discord-actions.test.ts b/src/agents/tools/discord-actions.test.ts index 3eead3f40..cef4bf30c 100644 --- a/src/agents/tools/discord-actions.test.ts +++ b/src/agents/tools/discord-actions.test.ts @@ -78,7 +78,7 @@ describe("handleDiscordMessagingAction", () => { }, enableAllActions, ); - expect(reactMessageDiscord).toHaveBeenCalledWith("C1", "M1", "✅"); + expect(reactMessageDiscord).toHaveBeenCalledWith("C1", "M1", "✅", {}); }); it("removes reactions on empty emoji", async () => { @@ -91,7 +91,7 @@ describe("handleDiscordMessagingAction", () => { }, enableAllActions, ); - expect(removeOwnReactionsDiscord).toHaveBeenCalledWith("C1", "M1"); + expect(removeOwnReactionsDiscord).toHaveBeenCalledWith("C1", "M1", {}); }); it("removes reactions when remove flag set", async () => { @@ -105,7 +105,7 @@ describe("handleDiscordMessagingAction", () => { }, enableAllActions, ); - expect(removeReactionDiscord).toHaveBeenCalledWith("C1", "M1", "✅"); + expect(removeReactionDiscord).toHaveBeenCalledWith("C1", "M1", "✅", {}); }); it("rejects removes without emoji", async () => { @@ -227,15 +227,18 @@ describe("handleDiscordGuildAction - channel management", () => { }, channelsEnabled, ); - expect(createChannelDiscord).toHaveBeenCalledWith({ - guildId: "G1", - name: "test-channel", - type: 0, - parentId: undefined, - topic: "Test topic", - position: undefined, - nsfw: undefined, - }); + expect(createChannelDiscord).toHaveBeenCalledWith( + { + guildId: "G1", + name: "test-channel", + type: 0, + parentId: undefined, + topic: "Test topic", + position: undefined, + nsfw: undefined, + }, + {}, + ); expect(result.details).toMatchObject({ ok: true }); }); @@ -255,15 +258,18 @@ describe("handleDiscordGuildAction - channel management", () => { }, channelsEnabled, ); - expect(editChannelDiscord).toHaveBeenCalledWith({ - channelId: "C1", - name: "new-name", - topic: "new topic", - position: undefined, - parentId: undefined, - nsfw: undefined, - rateLimitPerUser: undefined, - }); + expect(editChannelDiscord).toHaveBeenCalledWith( + { + channelId: "C1", + name: "new-name", + topic: "new topic", + position: undefined, + parentId: undefined, + nsfw: undefined, + rateLimitPerUser: undefined, + }, + {}, + ); }); it("clears the channel parent when parentId is null", async () => { @@ -275,15 +281,18 @@ describe("handleDiscordGuildAction - channel management", () => { }, channelsEnabled, ); - expect(editChannelDiscord).toHaveBeenCalledWith({ - channelId: "C1", - name: undefined, - topic: undefined, - position: undefined, - parentId: null, - nsfw: undefined, - rateLimitPerUser: undefined, - }); + expect(editChannelDiscord).toHaveBeenCalledWith( + { + channelId: "C1", + name: undefined, + topic: undefined, + position: undefined, + parentId: null, + nsfw: undefined, + rateLimitPerUser: undefined, + }, + {}, + ); }); it("clears the channel parent when clearParent is true", async () => { @@ -295,20 +304,23 @@ describe("handleDiscordGuildAction - channel management", () => { }, channelsEnabled, ); - expect(editChannelDiscord).toHaveBeenCalledWith({ - channelId: "C1", - name: undefined, - topic: undefined, - position: undefined, - parentId: null, - nsfw: undefined, - rateLimitPerUser: undefined, - }); + expect(editChannelDiscord).toHaveBeenCalledWith( + { + channelId: "C1", + name: undefined, + topic: undefined, + position: undefined, + parentId: null, + nsfw: undefined, + rateLimitPerUser: undefined, + }, + {}, + ); }); it("deletes a channel", async () => { await handleDiscordGuildAction("channelDelete", { channelId: "C1" }, channelsEnabled); - expect(deleteChannelDiscord).toHaveBeenCalledWith("C1"); + expect(deleteChannelDiscord).toHaveBeenCalledWith("C1", {}); }); it("moves a channel", async () => { @@ -322,12 +334,15 @@ describe("handleDiscordGuildAction - channel management", () => { }, channelsEnabled, ); - expect(moveChannelDiscord).toHaveBeenCalledWith({ - guildId: "G1", - channelId: "C1", - parentId: "P1", - position: 5, - }); + expect(moveChannelDiscord).toHaveBeenCalledWith( + { + guildId: "G1", + channelId: "C1", + parentId: "P1", + position: 5, + }, + {}, + ); }); it("clears the channel parent on move when parentId is null", async () => { @@ -340,12 +355,15 @@ describe("handleDiscordGuildAction - channel management", () => { }, channelsEnabled, ); - expect(moveChannelDiscord).toHaveBeenCalledWith({ - guildId: "G1", - channelId: "C1", - parentId: null, - position: undefined, - }); + expect(moveChannelDiscord).toHaveBeenCalledWith( + { + guildId: "G1", + channelId: "C1", + parentId: null, + position: undefined, + }, + {}, + ); }); it("clears the channel parent on move when clearParent is true", async () => { @@ -358,12 +376,15 @@ describe("handleDiscordGuildAction - channel management", () => { }, channelsEnabled, ); - expect(moveChannelDiscord).toHaveBeenCalledWith({ - guildId: "G1", - channelId: "C1", - parentId: null, - position: undefined, - }); + expect(moveChannelDiscord).toHaveBeenCalledWith( + { + guildId: "G1", + channelId: "C1", + parentId: null, + position: undefined, + }, + {}, + ); }); it("creates a category with type=4", async () => { @@ -372,12 +393,15 @@ describe("handleDiscordGuildAction - channel management", () => { { guildId: "G1", name: "My Category" }, channelsEnabled, ); - expect(createChannelDiscord).toHaveBeenCalledWith({ - guildId: "G1", - name: "My Category", - type: 4, - position: undefined, - }); + expect(createChannelDiscord).toHaveBeenCalledWith( + { + guildId: "G1", + name: "My Category", + type: 4, + position: undefined, + }, + {}, + ); }); it("edits a category", async () => { @@ -386,16 +410,19 @@ describe("handleDiscordGuildAction - channel management", () => { { categoryId: "CAT1", name: "Renamed Category" }, channelsEnabled, ); - expect(editChannelDiscord).toHaveBeenCalledWith({ - channelId: "CAT1", - name: "Renamed Category", - position: undefined, - }); + expect(editChannelDiscord).toHaveBeenCalledWith( + { + channelId: "CAT1", + name: "Renamed Category", + position: undefined, + }, + {}, + ); }); it("deletes a category", async () => { await handleDiscordGuildAction("categoryDelete", { categoryId: "CAT1" }, channelsEnabled); - expect(deleteChannelDiscord).toHaveBeenCalledWith("CAT1"); + expect(deleteChannelDiscord).toHaveBeenCalledWith("CAT1", {}); }); it("sets channel permissions for role", async () => { @@ -410,13 +437,16 @@ describe("handleDiscordGuildAction - channel management", () => { }, channelsEnabled, ); - expect(setChannelPermissionDiscord).toHaveBeenCalledWith({ - channelId: "C1", - targetId: "R1", - targetType: 0, - allow: "1024", - deny: "2048", - }); + expect(setChannelPermissionDiscord).toHaveBeenCalledWith( + { + channelId: "C1", + targetId: "R1", + targetType: 0, + allow: "1024", + deny: "2048", + }, + {}, + ); }); it("sets channel permissions for member", async () => { @@ -430,13 +460,16 @@ describe("handleDiscordGuildAction - channel management", () => { }, channelsEnabled, ); - expect(setChannelPermissionDiscord).toHaveBeenCalledWith({ - channelId: "C1", - targetId: "U1", - targetType: 1, - allow: "1024", - deny: undefined, - }); + expect(setChannelPermissionDiscord).toHaveBeenCalledWith( + { + channelId: "C1", + targetId: "U1", + targetType: 1, + allow: "1024", + deny: undefined, + }, + {}, + ); }); it("removes channel permissions", async () => { @@ -445,6 +478,6 @@ describe("handleDiscordGuildAction - channel management", () => { { channelId: "C1", targetId: "R1" }, channelsEnabled, ); - expect(removeChannelPermissionDiscord).toHaveBeenCalledWith("C1", "R1"); + expect(removeChannelPermissionDiscord).toHaveBeenCalledWith("C1", "R1", {}); }); });