amirho/nest
1
0
Fork 0
mirror of https://github.com/nestjs/nest.git synced 2026-02-21 23:11:44 +00:00
Code Issues Packages Projects Releases Wiki Activity

RC.8 fixes

Browse Source
This commit is contained in:
kamil.mysliwiec
2017-04-11 13:10:26 +02:00
parent d516d909d0
commit b0e8d5f125
2 changed files with 5 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
src/core/middlewares/builder.ts
View File

@@ -16,13 +16,10 @@ export class MiddlewareBuilder {
resolveParams = data;
return configProxy as MiddlewareConfigProxy;
},
forRoutes: (routes) => {
if (isUndefined(routes)) {
throw new InvalidMiddlewareConfigurationException();
}
forRoutes: (...routes) => {
const configuration = {
middlewares: this.bindValuesToResolve(metatypes, resolveParams),
forRoutes: [].concat(routes)
forRoutes: routes
};
this.middlewaresCollection.add(<MiddlewareConfiguration>configuration);
return this;
@@ -62,5 +59,5 @@ export class MiddlewareBuilder {
export interface MiddlewareConfigProxy {
with: (...data) => MiddlewareConfigProxy;
forRoutes: (routes) => MiddlewareBuilder;
forRoutes: (...routes) => MiddlewareBuilder;
}

9
src/core/test/middlewares/builder.spec.ts
View File

@@ -22,17 +22,12 @@ describe('MiddlewareBuilder', () => {
});
describe('when "forRoutes()" called', () => {
class Test {}
it('should throws "InvalidMiddlewareConfigurationException" when passed argument is undefined', () => {
expect(() => configProxy.forRoutes()).to.throws(
InvalidMiddlewareConfigurationException
);
});
it('should store configuration passed as argument', () => {
configProxy.forRoutes(Test);
configProxy.forRoutes(Test, 'test');
expect(builder.build()).to.deep.equal([{
middlewares: [],
forRoutes: [ Test ]
forRoutes: [ Test, 'test' ]
}]);
});
});