remove logger

Author: hersveit

src/controllers/auth/__tests__/auth.e2e.spec.ts

@@ -18,7 +18,6 @@ import { JwtRefreshTokenStrategy } from '../../../services/auth/strategies/jwt-r
 import { getUserStub } from '../../../__mocks__/user.stub';
 import { DatabaseServiceProvider } from '../../../services/database/database.service';
 import { ModuleRef } from '@nestjs/core';
-import { LoggerProvider } from '../../../core/logger.core';
 
 describe('AuthController', () => {
     const appWrap = {} as AppWrap;
@@ -79,7 +78,6 @@ describe('AuthController', () => {
             new HttpInterceptor(
                 createMock<ModuleRef>(),
                 db as unknown as DatabaseServiceProvider,
-                createMock<LoggerProvider>(),
             ),
         );
         appWrap.app.use(cookieParser());

src/controllers/user/__tests__/user.e2e.spec.ts

@@ -15,7 +15,6 @@ import { JwtRefreshTokenStrategy } from '../../../services/auth/strategies/jwt-r
 import { getUserStub } from '../../../__mocks__/user.stub';
 import { DatabaseServiceProvider } from '../../../services/database/database.service';
 import { ModuleRef } from '@nestjs/core';
-import { LoggerProvider } from '../../../core/logger.core';
 import { UserControllerProvider } from '../user.controller';
 import { EmailConfirm, RefreshToken, User } from '@prisma/client';
 import {
@@ -70,7 +69,6 @@ describe('UserController', () => {
             new HttpInterceptor(
                 createMock<ModuleRef>(),
                 db as unknown as DatabaseServiceProvider,
-                createMock<LoggerProvider>(),
             ),
         );
         appWrap.app.use(cookieParser());

src/core/__tests__/interceptor.spec.ts

@@ -2,7 +2,6 @@ import { BaseError } from '../errors.core';
 import { createMock, DeepMocked } from '@golevelup/ts-jest';
 import { DatabaseServiceProvider } from '../../services/database/database.service';
 import { HttpInterceptor } from '../interceptor.core';
-import { Logger, LoggerProvider } from '../logger.core';
 import { ModuleRef } from '@nestjs/core';
 import { Controller, Get } from '@nestjs/common';
 import { ErrorHandleMiddlewareProvider } from '../errorHandleMiddleware.core';
@@ -28,7 +27,6 @@ describe('Interceptor', () => {
     const appWrap = {} as AppWrap;
 
     let interceptor: HttpInterceptor;
-    let logger: DeepMocked<LoggerProvider>;
     let moduleRef: DeepMocked<ModuleRef>;
     let db: {
         Prisma: DeepMocked<DatabaseServiceProvider['Prisma']>;
@@ -52,23 +50,17 @@ describe('Interceptor', () => {
                         },
                     },
                 },
-                {
-                    provide: LoggerProvider,
-                    useValue: createMock<Logger>(),
-                },
             ],
             controllers: [TestController],
         }).compile();
 
         moduleRef = createMock<ModuleRef>();
 
         db = module.get(DatabaseServiceProvider);
-        logger = module.get(LoggerProvider);
 
         interceptor = new HttpInterceptor(
             moduleRef,
             db as unknown as DatabaseServiceProvider,
-            logger,
         );
         appWrap.app = module.createNestApplication();
         appWrap.app.useGlobalInterceptors(interceptor);
@@ -80,7 +72,6 @@ describe('Interceptor', () => {
 
     afterEach(() => {
         moduleRef.resolve.mockClear();
-        logger.log.mockClear();
         testResponse.response.mockClear();
     });
 
@@ -203,29 +194,14 @@ describe('Interceptor', () => {
     describe('onModuleInit and errorHandleMiddleware', () => {
         const errorHandleMiddleware = createMock<ErrorHandleMiddlewareProvider>();
 
-        it('should log a message if error handler not provided', async () => {
-            moduleRef.resolve.mockResolvedValueOnce(null);
-
-            await interceptor.onModuleInit();
-
-            expect(moduleRef.resolve).toBeCalledTimes(1);
-            expect(logger.log).toBeCalledTimes(1);
-            expect(logger.log).toBeCalledWith('No error handler where provided');
-        });
-
-        it('no logs on successful resolve ErrorHandleMiddlewareProvider', async () => {
-            moduleRef.resolve.mockResolvedValueOnce(errorHandleMiddleware);
-
-            await interceptor.onModuleInit();
-
-            expect(moduleRef.resolve).toBeCalledTimes(1);
-            expect(logger.log).toBeCalledTimes(0);
-        });
 
         it('errorHandleMiddleware should be called', async () => {
+            moduleRef.resolve.mockResolvedValueOnce(errorHandleMiddleware);
             testResponse.response.mockImplementationOnce(() => {
                 throw 'Test error';
             });
+
+            await interceptor.onModuleInit();
             await request(appWrap.app.getHttpServer()).get('/test/request').send();
 
             expect(errorHandleMiddleware.handleError).toBeCalledTimes(1);

src/core/interceptor.core.ts

@@ -5,6 +5,7 @@ import {
     ExecutionContext,
     HttpStatus,
     Injectable,
+    Logger,
     NestInterceptor,
     OnModuleInit,
 } from '@nestjs/common';
@@ -20,16 +21,15 @@ import {
     ErrorHandleMiddlewareProvider,
 } from './errorHandleMiddleware.core';
 import { ModuleRef } from '@nestjs/core';
-import { LoggerProvider } from './logger.core';
 
 @Injectable()
 export class HttpInterceptor implements NestInterceptor, OnModuleInit {
     private errorHandleMiddleware: ErrorHandleMiddlewareProvider | null = null;
+    private logger = new Logger(HttpInterceptor.name);
 
     constructor(
         private moduleRef: ModuleRef,
         private db: DatabaseServiceProvider,
-        private logger: LoggerProvider,
     ) {}
 
     async onModuleInit() {

src/core/logger.core.ts

@@ -1,12 +1,12 @@
-import { Module } from '@nestjs/common';
+import { Logger, Module } from '@nestjs/common';
 import { ConfigService, ConfigServiceProvider } from '../config/config.service';
 import { Injectable } from '@nestjs/common';
 import { PrismaClient } from '@prisma/client';
-import { Logger, LoggerProvider } from '../../core/logger.core';
 
 @Injectable()
 export class DatabaseServiceProvider {
     private readonly prisma: PrismaClient;
+    private readonly logger = new Logger(DatabaseServiceProvider.name);
 
     /**
      * !!! NO DB TRANSACTION INSTANCE !!!
@@ -18,7 +18,6 @@ export class DatabaseServiceProvider {
     constructor(
         // import the config to make sure it is initialized first
         private cfg: ConfigServiceProvider,
-        private logger: LoggerProvider,
     ) {
         this.prisma = new PrismaClient();
         this.prisma