fast-koa-router - npm Package Compare versions

Comparing version 1.3.0 to 1.3.1

.nyc_output/processinfo/index.json

@@ -1,1 +0,1 @@
-{"processes":{"373b3c77-f04a-41e7-90e5-ec602ca50297":{"parent":null,"children":[]}},"files":{"/home/runner/work/fast-koa-router/fast-koa-router/src/middleware.ts":["373b3c77-f04a-41e7-90e5-ec602ca50297"],"/home/runner/work/fast-koa-router/fast-koa-router/src/router.ts":["373b3c77-f04a-41e7-90e5-ec602ca50297"],"/home/runner/work/fast-koa-router/fast-koa-router/src/parse.ts":["373b3c77-f04a-41e7-90e5-ec602ca50297"]},"externalIds":{}}
+{"processes":{"d1d2240f-eb7c-4557-be0d-cb350be1a6e8":{"parent":null,"children":[]}},"files":{"/home/runner/work/fast-koa-router/fast-koa-router/src/middleware.ts":["d1d2240f-eb7c-4557-be0d-cb350be1a6e8"],"/home/runner/work/fast-koa-router/fast-koa-router/src/router.ts":["d1d2240f-eb7c-4557-be0d-cb350be1a6e8"],"/home/runner/work/fast-koa-router/fast-koa-router/src/parse.ts":["d1d2240f-eb7c-4557-be0d-cb350be1a6e8"]},"externalIds":{}}

build/parse.js

@@ -93,6 +93,4 @@ "use strict";
     }
-    const params = {};
-    let _matchedRoute = '';
     routes = routes[method] || {};
-    if (routes[path]) {
+    if (routes[path] && routes[path].middleware) {
         return { middleware: routes[path].middleware, _matchedRoute: path };
@@ -99,0 +97,0 @@ }

package.json

 {
   "name": "fast-koa-router",
-  "version": "1.3.0",
+  "version": "1.3.1",
   "description": "",
@@ -5,0 +5,0 @@ "main": "build/middleware.js",

test/parse.test.ts

@@ -201,2 +201,37 @@ import * as assert from 'assert';
 
+    it('star path should also match with path not ending with /', function () {
+      let routes = handlePathVariables(
+        addPrefixMiddleware(
+          parse({
+            get: {
+              '/path': ['middleware'],
+              '/path/1': ['one'],
+              '/path/1/3/foo': ['foo'],
+              '/foo/*': ['star'],
+              '/*': ['initialStar']
+            }
+          }),
+          { '/path': 'prefix' }
+        )
+      );
+
+      assert.deepStrictEqual(getPathMethod(routes, '/foo/', 'get'), {
+        _matchedRoute: '/foo/*',
+        params: {},
+        middleware: ['star']
+      });
+
+      assert.deepStrictEqual(getPathMethod(routes, '/foo', 'get'), {
+        _matchedRoute: '/foo/*',
+        params: {},
+        middleware: ['star']
+      });
+
+      assert.deepStrictEqual(getPathMethod(routes, '/foobar', 'get'), {
+        _matchedRoute: '/*',
+        params: {},
+        middleware: ['initialStar']
+      });
+    });
+
     it('star path should also work with many params in path', function () {
@@ -203,0 +238,0 @@ let routes = handlePathVariables(

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

113950

increased by0.85%

Lines of code

1148

increased by2.59%

No dependency changes