fix: check the logger level first

Drassil · Sep 25, 2024 · d269dc8 · d269dc8
1 parent c650c22
commit d269dc8
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 0 deletions.
diff --git 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
@@ -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;
   }