fix: implement does not related to dca1209

vectier · Apr 30, 2022 · 16a65a3 · 16a65a3
1 parent bbefd08
commit 16a65a3
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/lib/controllers/ControllerRegistry.js b/lib/controllers/ControllerRegistry.js
@@ -28,7 +28,7 @@ class ControllerRegistry {
 
       const registerMiddleware = (middleware) => {
         const canRegisterMiddleware = middleware
-          .getRegisterCondition()(routeMethod, routeMetadata);
+          .getRegisterCondition(routeMethod, routeMetadata);
 
         if (canRegisterMiddleware) {
           return middleware.getHandler(routeMetadata);

diff --git a/test/mock/MockMiddleware.ts b/test/mock/MockMiddleware.ts
@@ -1,4 +1,4 @@
-import { Middleware, MiddlewareCondition, RouteHandler } from '../..';
+import { Methods, Middleware, RouteHandler, RouteMetadata } from '../..';
 
 export class MockMiddleware extends Middleware {
 
@@ -18,10 +18,8 @@ export class MockConditionalMiddleware extends Middleware {
     };
   }
 
-  public getRegisterCondition(): MiddlewareCondition {
-    return (routeMethod, routeMetadata) => {
-      return routeMetadata.path === '/test2';
-    };
+  public getRegisterCondition(routeMethod: Methods, routeMetadata: RouteMetadata): boolean {
+    return routeMetadata.path === '/test2';
   }
 
 }