Merge branch 'feature/v3' of github.com:w3tecch/express-typescript-boilerplate into feature/v3

Author: hirsch88

src/api/middlewares/ErrorHandlerMiddleware.ts

@@ -1,15 +1,18 @@
 import * as express from 'express';
 import { Middleware, ExpressErrorMiddlewareInterface, HttpError } from 'routing-controllers';
 import { env } from '../../core/env';
-import { Logger } from '../../core/Logger';
+import { Logger, LoggerInterface } from '../../decorators/Logger';
 
 
 @Middleware({ type: 'after' })
 export class ErrorHandlerMiddleware implements ExpressErrorMiddlewareInterface {
 
-    public log = new Logger(__filename);
     public isProduction = env.isProduction;
 
+    constructor(
+        @Logger(__filename) private log: LoggerInterface
+    ) { }
+
     public error(error: HttpError, req: express.Request, res: express.Response, next: express.NextFunction): void {
         res.status(error.httpCode || 500);
 

test/unit/middlewares/ErrorHandlerMiddleware.test.ts

@@ -7,8 +7,7 @@ import { LogMock } from '../lib/LogMock';
 describe('ErrorHandlerMiddleware', () => {
 
     test('Should not print stack out in production', () => {
-        const middleware = new ErrorHandlerMiddleware();
-        middleware.log = new LogMock();
+        const middleware = new ErrorHandlerMiddleware(new LogMock());
         middleware.isProduction = true;
 
         const res = new MockExpressResponse();
@@ -21,8 +20,7 @@ describe('ErrorHandlerMiddleware', () => {
     });
 
     test('Should print stack out in production', () => {
-        const middleware = new ErrorHandlerMiddleware();
-        middleware.log = new LogMock();
+        const middleware = new ErrorHandlerMiddleware(new LogMock());
         middleware.isProduction = false;
 
         const res = new MockExpressResponse();