commit a6487cedaf8d49cfc3294ad461452f8fc8a1089c
author Matteo Scandolo <matteo.scandolo@gmail.com> Mon Feb 06 16:42:21 2017 -0800
committer Matteo Scandolo <matteo.scandolo@gmail.com> Mon Feb 06 16:42:21 2017 -0800
tree 5ced01454bbeab6db5aa9336f27248276110b9e3
parent a1f748d3805b364b74cec24f3df279dfef3861b1

CORD-833 Prevent duplication of navigation routes

Change-Id: Idbd797976080a3d7b29912950c687f1b0fcb67e1

src/app/core/services/navigation.spec.ts

diff --git a/src/app/core/services/navigation.spec.ts b/src/app/core/services/navigation.spec.ts
index 62e8d3c..fbfb91c 100644
--- a/src/app/core/services/navigation.spec.ts
+++ b/src/app/core/services/navigation.spec.ts
@@ -4,7 +4,7 @@
 import {xosCore} from '../index';
 import {IXosNavigationService, IXosNavigationRoute} from './navigation';
 
-let service: IXosNavigationService;
+let service: IXosNavigationService, $log: ng.ILogService;
 
 let defaultRoutes: IXosNavigationRoute[];
 
@@ -36,8 +36,11 @@
 
   beforeEach(angular.mock.inject((
     NavigationService: IXosNavigationService,
+    _$log_: ng.ILogService
   ) => {
     service = NavigationService;
+    $log = _$log_;
+    spyOn($log, 'warn');
     defaultRoutes = [
       {
         label: 'Home',
@@ -61,6 +64,7 @@
     ];
     service.add(testRoutes[0]);
     service.add(testRoutes[1]);
+    expect($log.warn).not.toHaveBeenCalled();
     const serviceRoutes = service.query();
     expect(serviceRoutes).toEqual(defaultRoutes.concat(testRoutes));
   });
@@ -84,4 +88,13 @@
     }
     expect(wrapper).toThrowError('[XosNavigation] You can\'t provide both state and url');
   });
+
+  it('should not add route that already exist', () => {
+    const testRoute: IXosNavigationRoute = {label: 'TestState', state: 'xos.test'};
+    service.add(testRoute);
+    service.add(testRoute);
+    expect($log.warn).toHaveBeenCalled();
+    expect($log.warn).toHaveBeenCalledWith(`[XosNavigation] Route with label: ${testRoute.label}, state: ${testRoute.state} and parent: ${testRoute.parent} already exist`);
+    expect(service.query()).toEqual(defaultRoutes.concat([testRoute]));
+  });
 });