From d269dc823e7bca4e2b87a7cd8c00ca8a56c06c5d Mon Sep 17 00:00:00 2001
From: Yehonal <joseph@fung.money>
Date: Wed, 25 Sep 2024 19:15:43 +0200
Subject: [PATCH] fix: check the logger level first

---
 event-manager/src/__tests__/event.helper.spec.ts | 7 +++++++
 event-manager/src/event.helper.ts                | 7 +++++++
 2 files changed, 14 insertions(+)

diff --git a/event-manager/src/__tests__/event.helper.spec.ts b/event-manager/src/__tests__/event.helper.spec.ts
index 87ffa9d..478d193 100644
--- a/event-manager/src/__tests__/event.helper.spec.ts
+++ b/event-manager/src/__tests__/event.helper.spec.ts
@@ -2,6 +2,7 @@ import { describe, expect, it, jest } from '@jest/globals';
 import { getLogLevelByStatus, isErrorEvent } from '../event.helper.js';
 import { HttpException, HttpStatus } from '@nestjs/common';
 import { DefaultError } from '@nestjs-yalc/errors/default.error.js';
+import { LogLevelEnum } from '@nestjs-yalc/logger/logger.enum.js';
 
 describe('EventHelper', () => {
   it('should return the correct log level', () => {
@@ -21,5 +22,11 @@ describe('EventHelper', () => {
         errorClass: new HttpException('test', HttpStatus.BAD_REQUEST),
       }),
     ).toBeFalsy();
+    expect(
+      isErrorEvent({
+        errorClass: HttpException,
+        logger: { level: LogLevelEnum.ERROR },
+      }),
+    ).toBeTruthy();
   });
 });
diff --git a/event-manager/src/event.helper.ts b/event-manager/src/event.helper.ts
index 896d25c..583e9fd 100644
--- a/event-manager/src/event.helper.ts
+++ b/event-manager/src/event.helper.ts
@@ -27,6 +27,13 @@ export function getLogLevelByStatus(statusCode: number) {
  * Check if the event will trigger an error log
  */
 export function isErrorEvent(options: IErrorEventOptions) {
+  if (
+    typeof options.logger === 'object' &&
+    options.logger.level === LogLevelEnum.ERROR
+  ) {
+    return true;
+  }
+
   if (!options.errorClass) {
     return false;
   }