fix(router): do not add routes with nav=false

routes with nav:false were still being included in router.navigation

routes that contain nav:false will no longer be included

src/router.js

@@ -179,7 +179,7 @@ export class Router {
 
     config.navModel = navModel;
 
-    if (('nav' in config || 'order' in navModel) && this.navigation.indexOf(navModel) === -1) {
+    if ((config.nav || 'order' in navModel) && this.navigation.indexOf(navModel) === -1) {
       navModel.order = navModel.order || config.nav;
       navModel.href = navModel.href || config.href;
       navModel.isActive = false;
@@ -232,4 +232,4 @@ export class Router {
     this.isNavigating = false;
     this.navigation = [];
   }
-}
+}

test/router.spec.js

@@ -2,9 +2,28 @@ import {History} from 'aurelia-history';
 import {Container} from 'aurelia-dependency-injection';
 import {AppRouter, PipelineProvider} from '../src/index';
 
-describe('observer locator', () => {
+describe('the router', () => {
   it('should have some tests', () => {
     var router = new AppRouter(new History(), new PipelineProvider(new Container()));
     expect(router).not.toBe(null);
   });
-});
+
+  describe('addRoute', () => {
+
+    it('should add a route to navigation if it has a nav=true', () => {
+      var router = new AppRouter(new History(), new PipelineProvider(new Container())),
+          testRoute = {};
+
+      router.addRoute({ route: 'test', moduleId: 'test', title: 'Resume', nav: true }, testRoute);
+      expect(router.navigation).toContain(testRoute);
+    });
+
+    it('should not add a route to navigation if it has a nav=false', () => {
+      var router = new AppRouter(new History(), new PipelineProvider(new Container())),
+          testRoute = {};
+
+      router.addRoute({ route: 'test', moduleId: 'test', title: 'Resume', nav: false }, testRoute);
+      expect(router.navigation).not.toContain(testRoute);
+    });
+  });
+});