Better test coverage

Author: kimjoar

test/index.js

@@ -71,6 +71,20 @@ describe('routeReducer', () => {
     });
   });
 
+  it('respects replace', () => {
+    expect(routeReducer(state, {
+      type: UPDATE_PATH,
+      path: '/bar',
+      replace: true,
+      avoidRouterUpdate: false
+    })).toEqual({
+      path: '/bar',
+      replace: true,
+      state: undefined,
+      changeId: 2
+    });
+  });
+
   it('respects `avoidRouterUpdate` flag', () => {
     expect(routeReducer(state, {
       type: UPDATE_PATH,
@@ -99,13 +113,21 @@ describe('syncReduxAndRouter', () => {
     expect(store.getState().routing.path).toEqual('/foo');
     expect(store.getState().routing.state).toEqual({ bar: 'baz' });
 
+    history.replaceState(null, '/bar');
+    expect(store.getState().routing.path).toEqual('/bar');
+    expect(store.getState().routing.state).toBe(null);
+
     history.pushState(null, '/bar');
     expect(store.getState().routing.path).toEqual('/bar');
     expect(store.getState().routing.state).toBe(null);
 
     history.pushState(null, '/bar?query=1');
     expect(store.getState().routing.path).toEqual('/bar?query=1');
 
+    history.replaceState({ bar: 'baz' }, '/bar?query=1');
+    expect(store.getState().routing.path).toEqual('/bar?query=1');
+    expect(store.getState().routing.state).toEqual({ bar: 'baz' });
+
     history.pushState(null, '/bar?query=1#hash=2');
     expect(store.getState().routing.path).toEqual('/bar?query=1#hash=2');
   });
@@ -135,26 +157,34 @@ describe('syncReduxAndRouter', () => {
       state: { bar: 'baz' }
     });
 
-    store.dispatch(pushPath('/bar'));
+    store.dispatch(replacePath('/bar', { bar: 'foo' }));
     expect(store.getState().routing).toEqual({
       path: '/bar',
       changeId: 4,
+      replace: true,
+      state: { bar: 'foo' }
+    });
+
+    store.dispatch(pushPath('/bar'));
+    expect(store.getState().routing).toEqual({
+      path: '/bar',
+      changeId: 5,
       replace: false,
       state: undefined
     });
 
     store.dispatch(pushPath('/bar?query=1'));
     expect(store.getState().routing).toEqual({
       path: '/bar?query=1',
-      changeId: 5,
+      changeId: 6,
       replace: false,
       state: undefined
     });
 
     store.dispatch(pushPath('/bar?query=1#hash=2'));
     expect(store.getState().routing).toEqual({
       path: '/bar?query=1#hash=2',
-      changeId: 6,
+      changeId: 7,
       replace: false,
       state: undefined
     });
@@ -184,6 +214,14 @@ describe('syncReduxAndRouter', () => {
       replace: false,
       state: undefined
     });
+
+    store.dispatch(replacePath('/foo'));
+    expect(store.getState().routing).toEqual({
+      path: '/foo',
+      changeId: 4,
+      replace: true,
+      state: undefined
+    });
   });
 
   it('does not update the router for other state changes', () => {
@@ -246,6 +284,15 @@ describe('syncReduxAndRouter', () => {
         });
         store.dispatch(pushPath('/bar'));
       }
+      else if(location.pathname === '/replace') {
+        expect(store.getState().routing).toEqual({
+          path: '/replace',
+          changeId: 4,
+          replace: false,
+          state: { bar: 'baz' }
+        });
+        store.dispatch(replacePath('/baz', { foo: 'bar' }));
+      }
     });
 
     store.dispatch(pushPath('/foo'));
@@ -255,6 +302,14 @@ describe('syncReduxAndRouter', () => {
       replace: false,
       state: undefined
     });
+
+    store.dispatch(pushPath('/replace', { bar: 'baz' }));
+    expect(store.getState().routing).toEqual({
+      path: '/baz',
+      changeId: 5,
+      replace: true,
+      state: { foo: 'bar' }
+    });
   })
 
   it('throws if "routing" key is missing with default selectRouteState', () => {