Merge branch 'Real-Dev-Squad:develop' into mentionDisplay

Author: RishiChaubey31

src/controllers/baseHandler.ts

@@ -82,9 +82,11 @@ export async function baseHandler(
     case getCommandName(REMOVE): {
       const data = message.data?.options as Array<messageRequestDataOptions>;
       const transformedArgument = {
+        member: message.member,
         roleToBeRemovedObj: data[0],
         channelId: message.channel_id,
       };
+
       return await kickEachUser(transformedArgument, env, ctx);
     }
 

src/controllers/kickEachUser.ts

@@ -7,15 +7,27 @@ import { getMembersInServer } from "../utils/getMembersInServer";
 import { filterUserByRoles } from "../utils/filterUsersByRole";
 import { discordTextResponse } from "../utils/discordResponse";
 import { removeUsers } from "../utils/removeUsers";
+import { SUPER_USER_ONE, SUPER_USER_TWO } from "../constants/variables";
+import { messageRequestMember } from "../typeDefinitions/discordMessage.types";
 
 export async function kickEachUser(
   transformedArgument: {
+    member: messageRequestMember;
     roleToBeRemovedObj: MentionEachUserOptions;
     channelId: number;
   },
   env: env,
   ctx: ExecutionContext
 ) {
+  const isUserSuperUser = [SUPER_USER_ONE, SUPER_USER_TWO].includes(
+    transformedArgument.member.user.id.toString()
+  );
+
+  if (!isUserSuperUser) {
+    const responseText = `You're not authorized to make this request.`;
+    return discordTextResponse(responseText);
+  }
+
   const getMembersInServerResponse = await getMembersInServer(env);
   const roleId = transformedArgument.roleToBeRemovedObj.value;
 

src/typeDefinitions/discordMessage.types.d.ts

@@ -77,7 +77,7 @@ export interface messageRequestMember {
 }
 
 export interface messageMember {
-  id: number;
+  id: number | bigint;
   username: string;
   avatar: string;
   discriminator: string;

tests/fixtures/fixture.ts

@@ -8,6 +8,7 @@ import { InteractionType } from "discord-interactions";
 import { UserBackend } from "../../src/typeDefinitions/userBackend.types";
 import { TaskOverdueResponse } from "../../src/typeDefinitions/taskOverdue.types";
 import { UserStatus } from "../../src/typeDefinitions/userStatus.type";
+import { SUPER_USER_ONE } from "../../src/constants/variables";
 
 export const dummyHelloMessage: discordMessageRequest = {
   type: InteractionType.APPLICATION_COMMAND,
@@ -292,6 +293,30 @@ export const userFutureStatusMock: UserStatus = {
   message: "User Status found successfully.",
 };
 
+export const messageRequestMemberMockNonSuperUser = {
+  member: {
+    user: {
+      id: 123455,
+      username: "ankush",
+      avatar: "https://cdn.discordapp.com/avatars/1/userAvatarHash.jpg",
+      discriminator: "discriminator",
+    },
+    joined_at: "2024-03-07T18:46:20.327000+00:00",
+  },
+};
+
+export const messageRequestMemberMockSuperUser = {
+  member: {
+    user: {
+      id: 154585730465660929n,
+      username: "ankush",
+      avatar: "https://cdn.discordapp.com/avatars/1/userAvatarHash.jpg",
+      discriminator: "discriminator",
+    },
+    joined_at: "2024-03-07T18:46:20.327000+00:00",
+  },
+};
+
 export const memberGroupRoleList: memberGroupRole[] = [
   { userid: "XXXX", roleid: "XXXX" },
   { userid: "YYYY", roleid: "YYYY" },

tests/unit/handlers/kickEachUser.test.ts

@@ -1,20 +1,53 @@
 import { kickEachUser } from "../../../src/controllers/kickEachUser";
-import { transformedArgument, ctx } from "../../fixtures/fixture";
+import {
+  transformedArgument,
+  ctx,
+  messageRequestMemberMockSuperUser,
+  messageRequestMemberMockNonSuperUser,
+} from "../../fixtures/fixture";
 
 describe("kickEachUser", () => {
-  it("should run when found no users with Matched Role", async () => {
+  it("should fail when a non super_user runs ", async () => {
     const env = {
       BOT_PUBLIC_KEY: "xyz",
       DISCORD_GUILD_ID: "123",
       DISCORD_TOKEN: "abc",
     };
 
     const { roleToBeTaggedObj } = transformedArgument; // Extracting roleToBeTaggedObj
-    const response = kickEachUser(
-      { roleToBeRemovedObj: roleToBeTaggedObj, channelId: 12345 },
-      env,
-      ctx
+    const messageRequestMember = {
+      roleToBeRemovedObj: roleToBeTaggedObj,
+      channelId: 12345,
+      ...messageRequestMemberMockNonSuperUser,
+    };
+    const response = kickEachUser(messageRequestMember, env, ctx);
+
+    const roleID = roleToBeTaggedObj.value;
+
+    expect(response).toBeInstanceOf(Promise);
+
+    const textMessage: { data: { content: string } } = await response.then(
+      (res) => res.json()
+    );
+    expect(textMessage.data.content).toBe(
+      `You're not authorized to make this request.`
     );
+  });
+
+  it("should run when found no users with Matched Role", async () => {
+    const env = {
+      BOT_PUBLIC_KEY: "xyz",
+      DISCORD_GUILD_ID: "123",
+      DISCORD_TOKEN: "abc",
+    };
+
+    const { roleToBeTaggedObj } = transformedArgument; // Extracting roleToBeTaggedObj
+    const messageRequestMember = {
+      roleToBeRemovedObj: roleToBeTaggedObj,
+      channelId: 12345,
+      ...messageRequestMemberMockSuperUser,
+    };
+    const response = kickEachUser(messageRequestMember, env, ctx);
 
     const roleID = roleToBeTaggedObj.value;