Add tests

supasate · Mar 9, 2020 · a78a799 · a78a799
1 parent dbb87d0
commit a78a799
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/src/ConnectedRouter.js b/src/ConnectedRouter.js
@@ -72,7 +72,7 @@ const createConnectedRouter = (structure) => {
       // Dispatch a location change action for the initial location.
       // This makes it backward-compatible with react-router-redux.
       // But, we add `isFirstRendering` to `true` to prevent double-rendering.
-      if(!props.noInitialPop) {
+      if (!props.noInitialPop) {
         handleLocationChange(history.location, history.action, true)
       }
     }

diff --git a/test/ConnectedRouter.test.js b/test/ConnectedRouter.test.js
@@ -191,6 +191,18 @@ describe('ConnectedRouter', () => {
       history.push('/new-location')
       expect(renderCount).toBe(2)
     })
+
+    it('does not call `props.onLocationChanged()` on intial location when `noInitialPop` prop is passed ', () => {
+      mount(
+        <Provider store={store}>
+          <ConnectedRouter {...props} noInitialPop>
+            <Route path="/" render={() => <div>Home</div>} />
+          </ConnectedRouter>
+        </Provider>
+      )
+
+      expect(onLocationChangedSpy.mock.calls).toHaveLength(0)
+    })
   })
 
   describe('with immutable structure', () => {