Skip to content

Commit

Permalink
feat(errorhandler): add errorhandler middleware
Browse files Browse the repository at this point in the history
  • Loading branch information
wbhob committed Nov 23, 2017
1 parent 651b356 commit 40e6cb4
Show file tree
Hide file tree
Showing 4 changed files with 88 additions and 0 deletions.
1 change: 1 addition & 0 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,7 @@
"@types/cookie-parser": "^1.4.1",
"@types/cors": "^2.8.3",
"@types/csurf": "^1.9.35",
"@types/errorhandler": "0.0.32",
"@types/helmet": "0.0.37",
"@types/mocha": "^2.2.44",
"@types/node": "^8.0.53",
Expand Down
36 changes: 36 additions & 0 deletions src/errorhandler/index.spec.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,36 @@
import { ErrorHandlerMiddleware } from './index';
import { expect } from 'chai';

describe('ErrorHandlerMiddleware', () => {
let middleware: ErrorHandlerMiddleware;
describe('properly configured', () => {
beforeEach(() => {
ErrorHandlerMiddleware.configure({
log: true,
});
middleware = new ErrorHandlerMiddleware();
});

it('should be defined', () => {
expect(middleware).to.not.be.undefined;
});

it('should have a function called resolve', () => {
expect(middleware.resolve).to.be.instanceof(Function);
});

it('should should return a middleware from calling resolve', () => {
expect(middleware.resolve()).to.be.an.instanceof(Function);
});
afterEach(() => {
ErrorHandlerMiddleware.configure(undefined);
});
});

describe('not configured', () => {
middleware = new ErrorHandlerMiddleware();
it('should should return a middleware from calling resolve', () => {
expect(middleware.resolve()).to.be.an.instanceof(Function);
});
});
});
23 changes: 23 additions & 0 deletions src/errorhandler/index.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
import * as errorhandler from 'errorhandler';

import { ErrorRequestHandler, RequestHandler } from 'express';
import { Middleware, NestMiddleware } from '@nestjs/common';

@Middleware()
export class ErrorHandlerMiddleware implements NestMiddleware {

// DELETE THESE LINES IF MIDDLEWARE DOES NOT TAKE OPTIONS
public static configure(opts: errorhandler.Options) {
this.options = opts;
}

private static options: errorhandler.Options;

public resolve(...args: any[]): RequestHandler {
if (ErrorHandlerMiddleware.options) {
return errorhandler(ErrorHandlerMiddleware.options) as any as RequestHandler;
} else {
return errorhandler() as any as RequestHandler;
}
}
}
28 changes: 28 additions & 0 deletions src/errorhandler/package.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,28 @@
{
"name": "@nest-middlewares/errorhandler",
"version": "1.0.0",
"description": "NestJS middleware for Error Handler",
"main": "index",
"repository": {
"type": "git",
"url": "git+https://github.com/wbhob/nest-middlewares.git"
},
"keywords": [
"nest",
"middlewares",
"express",
"node"
],
"author": "Wilson Hobbs <wilsonhobbs1@gmail.com>",
"license": "MIT",
"bugs": {
"url": "https://github.com/wbhob/nest-middlewares/issues"
},
"homepage": "https://github.com/wbhob/nest-middlewares#readme",
"peerDependencies": {
"@nestjs/common": "^4.0.0"
},
"dependencies": {
"errorhandler": "^1.5.0"
}
}

0 comments on commit 40e6cb4

Please sign in to comment.