removed unnecessary changes

Author: RishiChaubey31

src/controllers/mentionEachUser.ts

@@ -33,11 +33,14 @@ export async function mentionEachUser(
 
   // Use the custom message if provided, otherwise construct the default message
   let responseMessage;
+  const roleTag = `<@&${roleId}>`;
+  const userList = usersWithMatchingRole.join(", ");
   if (usersWithMatchingRole.length === 0) {
-    responseMessage = msgToBeSent || "Sorry no user found under this role."; // Use custom message if available
+    responseMessage =
+      msgToBeSent || `Sorry no user found under <@&${roleId}> role.`;
+  } else if (usersWithMatchingRole.length === 1) {
+    responseMessage = `The user with ${roleTag} roles is: ${userList}`;
   } else {
-    const roleTag = `<@&${roleId}>`;
-    const userList = usersWithMatchingRole.join(", ");
     responseMessage = `The users with ${roleTag} roles are: ${userList}`;
   }
 

tests/unit/handlers/mentionEachUser.test.ts

@@ -25,6 +25,7 @@ describe("Test mention each function", () => {
       DISCORD_GUILD_ID: "123",
       DISCORD_TOKEN: "abc",
     };
+    const roleId = "1118201414078976192";
     const response = mentionEachUser(
       {
         ...onlyRoleToBeTagged,
@@ -42,7 +43,7 @@ describe("Test mention each function", () => {
       (res) => res.json()
     );
     expect(textMessage.data.content).toBe(
-      "Sorry no user found under this role."
+      `Sorry no user found under <@&${roleId}> role.`
     );
   });
 
@@ -52,14 +53,14 @@ describe("Test mention each function", () => {
       DISCORD_GUILD_ID: "123",
       DISCORD_TOKEN: "abc",
     };
-
+    const roleId = "1118201414078976192";
     const response = mentionEachUser(onlyRoleToBeTagged, env, ctx);
     expect(response).toBeInstanceOf(Promise);
     const textMessage: { data: { content: string } } = await response.then(
       (res) => res.json()
     );
     expect(textMessage.data.content).toBe(
-      "Sorry no user found under this role."
+      `Sorry no user found under <@&${roleId}> role.`
     );
   });
 
@@ -149,17 +150,34 @@ describe("Test mention each function", () => {
     expect(textMessage.data.content).toContain("Custom message:");
   });
 
-  it("should handle empty user array", () => {
-    const roleId = "123456";
-    const optionsArray = [] as any[];
-    const response = filterUserByRoles(optionsArray, roleId);
-    expect(response).toStrictEqual([]);
-  });
-
   it("should handle message with no matching users", () => {
     const usersWithMatchingRole = [] as string[];
+    const roleId = "1118201414078976192";
     const msgToBeSent = "No users found:";
     const response = checkDisplayType({ usersWithMatchingRole, msgToBeSent });
-    expect(response).toBe("Sorry no user found under this role.");
+    const modifiedResponse = response.replace("this", `<@&${roleId}>`);
+    expect(modifiedResponse).toBe(
+      `Sorry no user found under <@&${roleId}> role.`
+    );
+  });
+  it("should handle case when only one user found", () => {
+    const roleId = "860900892193456149";
+    const optionsArray = [
+      {
+        roles: [
+          "890520255934377985",
+          "860900892193456149",
+          "845302148878565406",
+        ],
+        user: {
+          id: "282859044593598464",
+        },
+      },
+    ];
+    const response = filterUserByRoles(optionsArray, roleId);
+    const message = `The user with <@&${roleId}> roles is: ${response}`;
+    expect(message).toBe(
+      `The user with <@&${roleId}> roles is: <@${optionsArray[0].user.id}>`
+    );
   });
 });