feat(planet): add stop feature to planet service (#117)

Author: mario-mui

angular.json

@@ -382,5 +382,8 @@
       }
     }
   },
-  "defaultProject": "portal"
+  "defaultProject": "portal",
+  "cli": {
+    "analytics": false
+  }
 }

packages/planet/src/planet.spec.ts

@@ -183,4 +183,38 @@ describe('Planet', () => {
         expect(rerouteSpy).toHaveBeenCalledTimes(2);
         expect(planetRouterEvent.url).toEqual('/users');
     }));
+
+    it('should reroute not be call when planet stop', fakeAsync(() => {
+        const router: Router = TestBed.inject(Router);
+        const ngZone: NgZone = TestBed.inject(NgZone);
+        const rerouteSpy = spyOn(planetApplicationLoader, 'reroute');
+        expect(rerouteSpy).not.toHaveBeenCalled();
+        planet.start();
+        expect(rerouteSpy).toHaveBeenCalledTimes(1);
+
+        ngZone.run(() => {
+            router.navigateByUrl('/app1/dashboard');
+        });
+
+        tick();
+
+        expect(rerouteSpy).toHaveBeenCalledTimes(2);
+        expect(rerouteSpy).toHaveBeenCalledWith({
+            url: '/app1/dashboard'
+        });
+
+        planet.stop();
+
+        ngZone.run(() => {
+            router.navigateByUrl('/app1/users');
+        });
+        tick();
+
+        // rerouteSpy should not be call
+        // url should not change
+        expect(rerouteSpy).toHaveBeenCalledTimes(2);
+        expect(rerouteSpy).toHaveBeenCalledWith({
+            url: '/app1/dashboard'
+        });
+    }));
 });

packages/planet/src/planet.ts

@@ -7,7 +7,7 @@ import {
     AppsLoadingStartEvent,
     AppStatusChangeEvent
 } from './application/planet-application-loader';
-import { Observable } from 'rxjs';
+import { Observable, Subscription } from 'rxjs';
 import { filter, startWith, distinctUntilChanged, map } from 'rxjs/operators';
 import {
     setPortalApplicationData,
@@ -41,6 +41,8 @@ export class Planet {
         return this.planetApplicationLoader.appsLoadingStart;
     }
 
+    private subscription: Subscription;
+
     constructor(
         private injector: Injector,
         private router: Router,
@@ -83,7 +85,7 @@ export class Planet {
     }
 
     start() {
-        this.router.events
+        this.subscription = this.router.events
             .pipe(
                 filter(event => {
                     return event instanceof NavigationEnd;
@@ -100,4 +102,8 @@ export class Planet {
                 });
             });
     }
+
+    stop() {
+        this.subscription.unsubscribe();
+    }
 }