pass actual redirect when no state (#3285)

lib/core/services/auth-guard-bpm.service.spec.ts

@@ -91,6 +91,19 @@ describe('AuthGuardService BPM', () => {
         expect(authService.getRedirect('BPM')).toEqual(['some-url', { q: '123' }]);
     }));
 
+    it('should set redirect navigation commands with query params', async(() => {
+        spyOn(authService, 'setRedirect').and.callThrough();
+        spyOn(routerService, 'navigate').and.stub();
+        const router: RouterStateSnapshot = <RouterStateSnapshot> { url: '/' };
+
+        authGuard.canActivate(null, router);
+
+        expect(authService.setRedirect).toHaveBeenCalledWith({
+            provider: 'BPM', navigation: ['/']
+        });
+        expect(authService.getRedirect('BPM')).toEqual(['/']);
+    }));
+
     it('should get redirect url from config if there is one configured', async(() => {
         appConfigService.config.loginRoute = 'fakeLoginRoute';
         spyOn(authService, 'setRedirect').and.callThrough();

lib/core/services/auth-guard-bpm.service.ts

@@ -60,6 +60,11 @@ export class AuthGuardBpm implements CanActivate, CanActivateChild {
     private getNavigationCommands(redirectUrl: string): any[] {
         const urlTree: UrlTree = this.router.parseUrl(redirectUrl);
         const urlSegmentGroup: UrlSegmentGroup = urlTree.root.children[PRIMARY_OUTLET];
+
+        if (!urlSegmentGroup) {
+            return [redirectUrl];
+        }
+
         const urlSegments: UrlSegment[] = urlSegmentGroup.segments;
 
         return urlSegments.reduce(function(acc, item) {

lib/core/services/auth-guard-ecm.service.spec.ts

@@ -235,6 +235,28 @@ describe('AuthGuardService ECM', () => {
                  });
             });
         });
+
+        describe('with no route state', () => {
+            beforeEach(async(() => {
+                this.test = new TestConfig({
+                    isLoggedIn: false
+                });
+
+                const { guard, auth, router } = this.test;
+
+                guard.canActivate(null, { url: '/' }).then((activate) => {
+                    this.auth = auth;
+                });
+
+                this.navigateSpy = spyOn(router, 'navigate');
+            }));
+
+            it('should set redirect navigation commands with query params', () => {
+                expect(this.auth.setRedirect).toHaveBeenCalledWith({
+                    provider: 'ECM', navigation: ['/']
+                 });
+            });
+        });
     });
 
     describe('canActivateChild', () => {

lib/core/services/auth-guard-ecm.service.ts

@@ -75,6 +75,11 @@ export class AuthGuardEcm implements CanActivate {
     private getNavigationCommands(redirectUrl: string): any[] {
         const urlTree: UrlTree = this.router.parseUrl(redirectUrl);
         const urlSegmentGroup: UrlSegmentGroup = urlTree.root.children[PRIMARY_OUTLET];
+
+        if (!urlSegmentGroup) {
+            return [redirectUrl];
+        }
+
         const urlSegments: UrlSegment[] = urlSegmentGroup.segments;
 
         return urlSegments.reduce(function(acc, item) {

lib/core/services/auth-guard.service.spec.ts

@@ -101,4 +101,17 @@ describe('AuthGuardService', () => {
         });
         expect(router.navigate).toHaveBeenCalledWith(['/fakeLoginRoute']);
     }));
+
+    it('should pass actual redirect when no state segments exists', async(() => {
+        state.url = '/';
+
+        spyOn(router, 'navigate');
+        spyOn(authService, 'setRedirect');
+
+        service.canActivate(null, state);
+
+        expect(authService.setRedirect).toHaveBeenCalledWith({
+            provider: 'ALL', navigation: ['/']
+        });
+    }));
 });

lib/core/services/auth-guard.service.ts

@@ -63,6 +63,11 @@ export class AuthGuard implements CanActivate, CanActivateChild {
     private getNavigationCommands(redirectUrl: string): any[] {
         const urlTree: UrlTree = this.router.parseUrl(redirectUrl);
         const urlSegmentGroup: UrlSegmentGroup = urlTree.root.children[PRIMARY_OUTLET];
+
+        if (!urlSegmentGroup) {
+            return [redirectUrl];
+        }
+
         const urlSegments: UrlSegment[] = urlSegmentGroup.segments;
 
         return urlSegments.reduce(function(acc, item) {