Add reducer tests. Ensure actions are FSA.

Author: timdorr

src/reducer.js

@@ -14,9 +14,9 @@ const initialState = {
  * if you have asynchronously-loaded routes, so reading from and relying on
  * this state it is discouraged.
  */
-export function routerReducer(state = initialState, { type, locationBeforeTransitions }) {
+export function routerReducer(state = initialState, { type, payload }) {
   if (type === LOCATION_CHANGE) {
-    return { ...state, locationBeforeTransitions }
+    return { ...state, locationBeforeTransitions: payload }
   }
 
   return state

src/sync.js

@@ -89,7 +89,7 @@ export default function syncHistoryWithStore(history, store, {
     // Tell the store to update by dispatching an action
     store.dispatch({
       type: LOCATION_CHANGE,
-      locationBeforeTransitions: location
+      payload: location
     })
   }
   unsubscribeFromHistory = history.listen(handleLocationChange)

test/node/reducer.js

@@ -0,0 +1,60 @@
+/* eslint-env mocha */
+
+import expect from 'expect'
+
+import { LOCATION_CHANGE, routerReducer } from '../../src/reducer'
+
+describe('routerReducer', () => {
+  const state = {
+    locationBeforeTransitions: {
+      pathname: '/foo',
+      action: 'POP'
+    }
+  }
+
+  it('updates the path', () => {
+    expect(routerReducer(state, {
+      type: LOCATION_CHANGE,
+      payload: {
+        path: '/bar',
+        action: 'PUSH'
+      }
+    })).toEqual({
+      locationBeforeTransitions: {
+        path: '/bar',
+        action: 'PUSH'
+      }
+    })
+  })
+
+  it('works with initialState', () => {
+    expect(routerReducer(undefined, {
+      type: LOCATION_CHANGE,
+      payload: {
+        path: '/bar',
+        action: 'PUSH'
+      }
+    })).toEqual({
+      locationBeforeTransitions: {
+        path: '/bar',
+        action: 'PUSH'
+      }
+    })
+  })
+
+
+  it('respects replace', () => {
+    expect(routerReducer(state, {
+      type: LOCATION_CHANGE,
+      payload: {
+        path: '/bar',
+        action: 'REPLACE'
+      }
+    })).toEqual({
+      locationBeforeTransitions: {
+        path: '/bar',
+        action: 'REPLACE'
+      }
+    })
+  })
+})