Merge pull request #18 from aivins/multinames

Support multiple names for a single route

src/route-recognizer.js

@@ -63,10 +63,13 @@ export class RouteRecognizer {
     let handlers = [{ handler: route.handler, names: names }];
 
     if (routeName) {
-      this.names[routeName] = {
-        segments: segments,
-        handlers: handlers
-      };
+      let routeNames = Array.isArray(routeName) ? routeName : [routeName];
+      for (let i = 0; i < routeNames.length; i++) {
+        this.names[routeNames[i]] = {
+          segments: segments,
+          handlers: handlers
+        };
+      }
     }
 
     currentState.handlers = handlers;

test/route-recognizer.spec.js

@@ -3,6 +3,7 @@ import core from 'core-js';
 
 const staticRoute = {'path': 'static','handler': {'name': 'static'}};
 const dynamicRoute = {'path': 'dynamic/:id','handler': {'name': 'dynamic'}};
+const multiNameRoute = {'path': 'static','handler': {'name': ['static-multiple', 'static-multiple-alias']}};
 
 const routeTestData = [{
     title: 'empty path routes',
@@ -117,4 +118,12 @@ describe('route recognizer', () => {
     expect(recognizer.hasRoute('static')).toBe(true);
     expect(recognizer.handlersFor('static')[0].handler).toEqual(staticRoute.handler);
   });
+
+  it('should find a handler by multiple names', () => {
+    let recognizer = new RouteRecognizer();
+    recognizer.add([multiNameRoute]);
+
+    expect(recognizer.handlersFor('static-multiple')[0].handler)
+      .toEqual(recognizer.handlersFor('static-multiple-alias')[0].handler)
+  });
 });