Fixing a bug in the getRoute method

CHANGELOG.md

@@ -1,3 +1,7 @@
+## 8.3.1
+
+Fixing a bug in the `getRoute` method.
+
 ## 8.3.0
 
 Adding `addBeforeHook`, `addAfterHook`, `addAlreadyHook`, `addLeaveHook` and `getRoute` methods.

package.json

@@ -1,6 +1,6 @@
 {
   "name": "navigo",
-  "version": "8.3.0",
+  "version": "8.3.1",
   "description": "A simple vanilla JavaScript router",
   "main": "lib/navigo.js",
   "browser": "lib/navigo.min.js",

src/__tests__/basic.spec.ts

@@ -27,7 +27,7 @@ describe("Given the Navigo library", () => {
       const router: NavigoRouter = new Navigo("/foo");
       router.on(/^b/, handler);
       expect(router.routes).toStrictEqual([
-        { path: /^b/, handler, hooks: {}, name: "/^b/" },
+        { path: /^b/, handler, hooks: {}, name: "^b" },
       ]);
     });
     it("should accept object with paths and handlers", () => {

src/__tests__/getRoute.spec.ts

@@ -20,7 +20,7 @@ describe("Given the Navigo library", () => {
       expect(r.routes).toHaveLength(3);
       expect(r.getRoute("foo")).toStrictEqual(r.routes[0]);
       expect(r.getRoute("xxx")).toStrictEqual(r.routes[1]);
-      expect(r.getRoute("/foo(.*)/")).toStrictEqual(r.routes[2]);
+      expect(r.getRoute("foo(.*)")).toStrictEqual(r.routes[2]);
       expect(r.getRoute("nope")).toEqual(undefined);
     });
   });

src/index.ts

@@ -76,7 +76,7 @@ export default function Navigo(
   ): Route {
     path = isString(path) ? clean(`${root}/${clean(path as string)}`) : path;
     return {
-      name: name || String(path),
+      name: name || clean(String(path)),
       path,
       handler,
       hooks: accumulateHooks(hooks),
@@ -335,8 +335,8 @@ export default function Navigo(
     }
     return () => {};
   }
-  function getRoute(name): Route | undefined {
-    return routes.find((r) => r.name === name);
+  function getRoute(name: string): Route | undefined {
+    return routes.find((r) => r.name === clean(name));
   }
 
   this.root = root;