Fix for issue #7

In the test though I’m not sure how to properly catch the failing mock test and test it’s error state
reduxjs · Jan 8, 2016 · 718abd8 · 718abd8
1 parent 4e08215
commit 718abd8
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/src/index.js b/src/index.js
@@ -44,6 +44,9 @@ export default function configureStore(middlewares = []) {
 
             return action;
           } catch (e) {
+            if (done) {
+              done(e);
+            }
             throw e;
           }
         }

diff --git a/test/index.js b/test/index.js
@@ -115,12 +115,12 @@ describe('Redux mockStore', () => {
   it('handles multiple actions', done => {
     const store = mockStore({}, [{ type: 'ADD_ITEM' }, { type: 'REMOVE_ITEM' }], done);
     try {
-      store.dispatch({ type: 'ADD_ITEMS' });
+      store.dispatch({ type: 'UNEXPECTED_ACTION' });
       store.dispatch({ type: 'REMOVE_ITEM' });
     } catch (e) {
-      expect(e.actual.type).toBe('ADD_ITEMS');
+      expect(e.actual.type).toBe('UNEXPECTED_ACTION');
       expect(e.expected.type).toBe('ADD_ITEM');
-      done();
+      // done(e);
     }
   });