Unsubscribe after each test to silence warnings

Author: ellbee

test/createTests.js

@@ -7,8 +7,8 @@ function createSyncedHistoryAndStore(createHistory) {
     routing: routeReducer
   }));
   const history = createHistory();
-  syncReduxAndRouter(history, store);
-  return { history, store };
+  const unsubscribe = syncReduxAndRouter(history, store);
+  return { history, store, unsubscribe };
 }
 
 const defaultReset = () => {};
@@ -115,8 +115,20 @@ module.exports = function createTests(createHistory, name, reset = defaultReset)
     });
 
     describe('syncReduxAndRouter', () => {
+      let history, store, unsubscribe;
+
+      beforeEach(() => {
+        let synced = createSyncedHistoryAndStore(createHistory);
+        history = synced.history;
+        store = synced.store;
+        unsubscribe = synced.unsubscribe;
+      });
+
+      afterEach(() => {
+        unsubscribe();
+      });
+
       it('syncs router -> redux', () => {
-        const { history, store } = createSyncedHistoryAndStore(createHistory);
         expect(store.getState().routing.path).toEqual('/');
 
         history.pushState(null, '/foo');
@@ -147,7 +159,6 @@ module.exports = function createTests(createHistory, name, reset = defaultReset)
       });
 
       it('syncs redux -> router', () => {
-        const { history, store } = createSyncedHistoryAndStore(createHistory);
         expect(store.getState().routing).toEqual({
           path: '/',
           changeId: 1,
@@ -205,7 +216,6 @@ module.exports = function createTests(createHistory, name, reset = defaultReset)
       });
 
       it('updates the router even if path is the same', () => {
-        const { history, store } = createSyncedHistoryAndStore(createHistory);
         expect(store.getState().routing).toEqual({
           path: '/',
           changeId: 1,
@@ -239,7 +249,6 @@ module.exports = function createTests(createHistory, name, reset = defaultReset)
       });
 
       it('does not update the router for other state changes', () => {
-        const { history, store } = createSyncedHistoryAndStore(createHistory);
         store.dispatch({
           type: 'RANDOM_ACTION',
           value: 5
@@ -254,7 +263,6 @@ module.exports = function createTests(createHistory, name, reset = defaultReset)
       });
 
       it('only updates the router once when dispatching from `listenBefore`', () => {
-        const { history, store } = createSyncedHistoryAndStore(createHistory);
         expect(store.getState().routing).toEqual({
           path: '/',
           changeId: 1,
@@ -280,10 +288,9 @@ module.exports = function createTests(createHistory, name, reset = defaultReset)
       });
 
       it('does not unnecessarily update the store', () => {
-        const { history, store } = createSyncedHistoryAndStore(createHistory);
         const updates = [];
 
-        const unsubscribe = store.subscribe(() => {
+        const unsubscribeFromStore = store.subscribe(() => {
           updates.push(store.getState())
         });
 
@@ -293,7 +300,7 @@ module.exports = function createTests(createHistory, name, reset = defaultReset)
         store.dispatch(replacePath('/bar'));
         store.dispatch(replacePath('/bar', { bar: 'foo' }));
 
-        unsubscribe();
+        unsubscribeFromStore();
 
         expect(updates.length).toBe(5);
         expect(updates).toEqual([
@@ -341,7 +348,6 @@ module.exports = function createTests(createHistory, name, reset = defaultReset)
       });
 
       it('allows updating the route from within `listenBefore`', () => {
-        const { history, store } = createSyncedHistoryAndStore(createHistory);
         expect(store.getState().routing).toEqual({
           path: '/',
           changeId: 1,