test fix

Author: walker-sean

src/backend/src/controllers/users.controllers.ts

@@ -15,7 +15,7 @@ export default class UsersController {
 
   static async getAllOrgUsers(req: Request, res: Response, next: NextFunction) {
     try {
-      const users = await UsersService.getAllOrgUsers(req.organization);
+      const users = await UsersService.getAllOrgUsers(req.organization.organizationId);
       res.status(200).json(users);
     } catch (error: unknown) {
       next(error);

src/backend/src/services/users.services.ts

@@ -56,10 +56,10 @@ export default class UsersService {
    * @param organization the organization to get the users from
    * @returns a list of all the users in the current organization
    */
-  static async getAllOrgUsers(organization: Organization): Promise<User[]> {
+  static async getAllOrgUsers(organizationId: string): Promise<User[]> {
     const users = await prisma.user.findMany({
-      where: { organizations: { some: { organizationId: organization.organizationId } } },
-      ...getUserQueryArgs(organization.organizationId)
+      where: { organizations: { some: { organizationId } } },
+      ...getUserQueryArgs(organizationId)
     });
 
     users.sort((a, b) => a.firstName.localeCompare(b.firstName));

src/backend/src/utils/slack.utils.ts

@@ -550,14 +550,14 @@ export const blockToString = async (block: SlackRichTextBlock) => {
  */
 export const blockToMentionedUsers = async (
   block: SlackRichTextBlock,
-  _organizationId: string,
+  organizationId: string,
   channelId: string
 ): Promise<string[]> => {
   switch (block.type) {
     case 'broadcast':
       switch (block.range) {
         case 'everyone':
-          const usersInOrg = await UsersService.getAllUsers();
+          const usersInOrg = await UsersService.getAllOrgUsers(organizationId);
           return usersInOrg.map((user) => user.userId);
         case 'channel':
         case 'here':