fix: delete crisp test users (#394)

src/user/user.service.ts

@@ -373,17 +373,20 @@ export class UserService {
  }
 
  public async deleteCypressTestUsers(clean = false): Promise<UserEntity[]> {
+ let deletedUsers: UserEntity[] = [];
  try {
  const queryResult = await this.userRepository
  .createQueryBuilder('user')
  .select()
  .where('user.name LIKE :searchTerm', { searchTerm: `%Cypress test%` })
  .getMany();
 
- const deletedUsers = await Promise.all(
+ deletedUsers = await Promise.all(
  queryResult.map(async (user) => {
  try {
- await deleteCrispProfile(user.email);
+ // TODO: replace me - temporarily disabled due to too many tests accounts to delete, causing 429 errors on crisp API
+ // once crisp test users have been cleared using the clean function, and there are <50 test users in crisp, this can be replaced
+ // await deleteCrispProfile(user.email);
  await this.authService.deleteFirebaseUser(user.firebaseUid);
  await this.userRepository.delete(user);
  return user;
@@ -392,8 +395,6 @@ export class UserService {
  }
  }),
  );
-
- return deletedUsers;
  } catch (error) {
  // If this fails we don't want to break cypress tests but we want to be alerted
  this.logger.error(`deleteCypressTestUsers - Unable to delete all cypress users`, error);
@@ -413,6 +414,9 @@ export class UserService {
  // If this fails we don't want to break cypress tests but we want to be alerted
  this.logger.error(`deleteCypressTestUsers - Unable to clean all cypress users`, error);
  }
+
+ this.logger.log(`deleteCypressTestUsers - Successfully deleted ${deletedUsers.length} users`);
+ return deletedUsers;
  }
 
  public async getUsers(