Skip to content

Commit 87b3140

Browse files
committed
Add test for super user check
1 parent dbea094 commit 87b3140

File tree

3 files changed

+60
-11
lines changed

3 files changed

+60
-11
lines changed

src/typeDefinitions/discordMessage.types.d.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ export interface messageRequestMember {
7777
}
7878

7979
export interface messageMember {
80-
id: number;
80+
id: number | bigint;
8181
username: string;
8282
avatar: string;
8383
discriminator: string;

tests/fixtures/fixture.ts

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ import { InteractionType } from "discord-interactions";
88
import { UserBackend } from "../../src/typeDefinitions/userBackend.types";
99
import { TaskOverdueResponse } from "../../src/typeDefinitions/taskOverdue.types";
1010
import { UserStatus } from "../../src/typeDefinitions/userStatus.type";
11+
import { SUPER_USER_ONE } from "../../src/constants/variables";
1112

1213
export const dummyHelloMessage: discordMessageRequest = {
1314
type: InteractionType.APPLICATION_COMMAND,
@@ -292,6 +293,30 @@ export const userFutureStatusMock: UserStatus = {
292293
message: "User Status found successfully.",
293294
};
294295

296+
export const messageRequestMemberMockNonSuperUser = {
297+
member: {
298+
user: {
299+
id: 123455,
300+
username: "ankush",
301+
avatar: "https://cdn.discordapp.com/avatars/1/userAvatarHash.jpg",
302+
discriminator: "discriminator",
303+
},
304+
joined_at: "2024-03-07T18:46:20.327000+00:00",
305+
},
306+
};
307+
308+
export const messageRequestMemberMockSuperUser = {
309+
member: {
310+
user: {
311+
id: 154585730465660929n,
312+
username: "ankush",
313+
avatar: "https://cdn.discordapp.com/avatars/1/userAvatarHash.jpg",
314+
discriminator: "discriminator",
315+
},
316+
joined_at: "2024-03-07T18:46:20.327000+00:00",
317+
},
318+
};
319+
295320
export const memberGroupRoleList: memberGroupRole[] = [
296321
{ userid: "XXXX", roleid: "XXXX" },
297322
{ userid: "YYYY", roleid: "YYYY" },

tests/unit/handlers/kickEachUser.test.ts

Lines changed: 34 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,39 @@
11
import { kickEachUser } from "../../../src/controllers/kickEachUser";
2-
import { transformedArgument, ctx } from "../../fixtures/fixture";
2+
import {
3+
transformedArgument,
4+
ctx,
5+
messageRequestMemberMockSuperUser,
6+
messageRequestMemberMockNonSuperUser,
7+
} from "../../fixtures/fixture";
38

49
describe("kickEachUser", () => {
10+
it("should fail when a non super_user runs ", async () => {
11+
const env = {
12+
BOT_PUBLIC_KEY: "xyz",
13+
DISCORD_GUILD_ID: "123",
14+
DISCORD_TOKEN: "abc",
15+
};
16+
17+
const { roleToBeTaggedObj } = transformedArgument; // Extracting roleToBeTaggedObj
18+
const messageRequestMember = {
19+
roleToBeRemovedObj: roleToBeTaggedObj,
20+
channelId: 12345,
21+
...messageRequestMemberMockNonSuperUser,
22+
};
23+
const response = kickEachUser(messageRequestMember, env, ctx);
24+
25+
const roleID = roleToBeTaggedObj.value;
26+
27+
expect(response).toBeInstanceOf(Promise);
28+
29+
const textMessage: { data: { content: string } } = await response.then(
30+
(res) => res.json()
31+
);
32+
expect(textMessage.data.content).toBe(
33+
`You're not authorized to make this request.`
34+
);
35+
});
36+
537
it("should run when found no users with Matched Role", async () => {
638
const env = {
739
BOT_PUBLIC_KEY: "xyz",
@@ -13,15 +45,7 @@ describe("kickEachUser", () => {
1345
const messageRequestMember = {
1446
roleToBeRemovedObj: roleToBeTaggedObj,
1547
channelId: 12345,
16-
member: {
17-
user: {
18-
id: 123455,
19-
username: "ankush",
20-
avatar: "",
21-
discriminator: "",
22-
},
23-
joined_at: "12345",
24-
},
48+
...messageRequestMemberMockSuperUser,
2549
};
2650
const response = kickEachUser(messageRequestMember, env, ctx);
2751

0 commit comments

Comments
 (0)