Empêche les erreurs du MattermostNotifier d'interrompre les autres op…

…érations

src/Application/Notification/Command/CreateNotificationCommandHandler.ts

@@ -22,6 +22,17 @@ export class CreateNotificationCommandHandler {
   ) {}
 
   public async execute(command: CreateNotificationCommand): Promise<string> {
+    try {
+      return await this.createNotification(command);
+    } catch (e) {
+      // On avale l'exception pour éviter d'interrompre les autres traitements.
+      // Tant pis si la notification ne part pas, on finira bien par s'en rendre compte.
+      console.error('Failed to create notification:', e);
+    }
+  }
+  private async createNotification(
+    command: CreateNotificationCommand
+  ): Promise<string> {
     const { message, type, leaveReaquest } = command;
 
     if (type === NotificationType.POST) {

src/Domain/Notification/Exception/NotificationFailureException.ts

@@ -0,0 +1 @@
+export class NotificationFailureException extends Error {}

src/Infrastructure/Adapter/MattermostNotifier.ts

@@ -1,7 +1,8 @@
 import { HttpService } from '@nestjs/axios';
-import { Injectable, InternalServerErrorException } from '@nestjs/common';
+import { Injectable } from '@nestjs/common';
 import { ConfigService } from '@nestjs/config';
 import { IMattermostNotifier } from 'src/Application/IMattermostNotifier';
+import { NotificationFailureException } from 'src/Domain/Notification/Exception/NotificationFailureException';
 
 @Injectable()
 export class MattermostNotifier implements IMattermostNotifier {
@@ -29,7 +30,7 @@ export class MattermostNotifier implements IMattermostNotifier {
 
       return response.data;
     } catch (e) {
-      throw new InternalServerErrorException(e);
+      throw new NotificationFailureException(e);
     }
   }
 
@@ -57,8 +58,7 @@ export class MattermostNotifier implements IMattermostNotifier {
 
       return response.data;
     } catch (e) {
-      console.error('--error: ', e);
-      throw new InternalServerErrorException(e);
+      throw new NotificationFailureException(e);
     }
   }
 
@@ -85,7 +85,7 @@ export class MattermostNotifier implements IMattermostNotifier {
 
       return response.data;
     } catch (e) {
-      throw new InternalServerErrorException(e);
+      throw new NotificationFailureException(e);
     }
   }
 }