diff --git a/server/src/services/group_permissions.service.ts b/server/src/services/group_permissions.service.ts index f33fd95ba6..99a3751d01 100644 --- a/server/src/services/group_permissions.service.ts +++ b/server/src/services/group_permissions.service.ts @@ -303,8 +303,9 @@ export class GroupPermissionsService { async findUsers(user: User, groupPermissionId: string): Promise { return createQueryBuilder(User, 'users') - .innerJoinAndSelect('users.groupPermissions', 'group_permissions') - .innerJoinAndSelect('users.userGroupPermissions', 'user_group_permissions') + .select(['users.id', 'users.firstName', 'users.lastName', 'users.email']) + .innerJoin('users.groupPermissions', 'group_permissions') + .innerJoin('users.userGroupPermissions', 'user_group_permissions') .where('group_permissions.id = :groupPermissionId', { groupPermissionId, }) diff --git a/server/test/controllers/group_permissions.e2e-spec.ts b/server/test/controllers/group_permissions.e2e-spec.ts index 2b55483991..7627c03bb1 100644 --- a/server/test/controllers/group_permissions.e2e-spec.ts +++ b/server/test/controllers/group_permissions.e2e-spec.ts @@ -628,8 +628,10 @@ describe('group permissions controller', () => { const user = users[0]; expect(users).toHaveLength(1); - expect(user.default_organization_id).toBe(organization.id); + expect(Object.keys(user).sort()).toEqual(['id', 'email', 'first_name', 'last_name'].sort()); expect(user.email).toBe('admin@tooljet.io'); + expect(user.first_name).toBe('test'); + expect(user.last_name).toBe('test'); }); });