test(common): add more tests for location_spec and integration spec

Add more tests for location_spec to test location.goTo(0), location.goTo(), location.goTo(100) and location.goTo(-100). We also add new tests for integration spec.
angular · Sep 18, 2020 · 365794f · 365794f
1 parent 8e474e7
commit 365794f
Show file tree

Hide file tree

Showing 2 changed files with 51 additions and 0 deletions.
diff --git a/packages/common/test/location/location_spec.ts b/packages/common/test/location/location_spec.ts
@@ -109,6 +109,22 @@ describe('Location Class', () => {
 
       location.goTo(2);
       expect(location.getState()).toEqual({url: 'test3'});
+
+      location.go('/test3', '', {url: 'test4'});
+      location.goTo(0);
+      expect(location.getState()).toEqual({url: 'test4'});
+
+      location.goTo();
+      expect(location.getState()).toEqual({url: 'test4'});
+
+      location.goTo(100);
+      expect(location.getState()).toEqual({url: 'test4'});
+
+      location.goTo(-100);
+      expect(location.getState()).toEqual({url: 'test4'});
+
+      location.back();
+      expect(location.getState()).toEqual({url: 'test3'});
     });
   });
 

diff --git a/packages/router/test/integration.spec.ts b/packages/router/test/integration.spec.ts
@@ -186,6 +186,41 @@ describe('Integration', () => {
          expect(navigation.extras.state).toEqual(state);
        })));
 
+    it('should set history.state when navigation with browser go',
+       fakeAsync(inject([Router, Location], (router: Router, location: SpyLocation) => {
+         router.resetConfig([
+           {path: '', component: SimpleCmp},
+           {path: 'simple', component: SimpleCmp},
+         ]);
+
+         const fixture = createRoot(router, RootCmp);
+         let navigation: Navigation = null!;
+         router.events.subscribe(e => {
+           if (e instanceof NavigationStart) {
+             navigation = <Navigation>router.getCurrentNavigation()!;
+           }
+         });
+
+         const state = {foo: 'bar'};
+         router.navigateByUrl('/simple', {state});
+         tick();
+         location.goTo(-1);
+         tick();
+         location.goTo(1);
+         tick();
+         location.goTo(-100);
+         tick();
+         location.goTo(100);
+         tick();
+         location.goTo(0);
+         tick();
+         location.goTo();
+         tick();
+
+         expect(navigation.extras.state).toBeDefined();
+         expect(navigation.extras.state).toEqual(state);
+       })));
+
     it('should not error if state is not {[key: string]: any}',
        fakeAsync(inject([Router, Location], (router: Router, location: SpyLocation) => {
          router.resetConfig([