Fix for issue reduxjs#7

In the test though I’m not sure how to properly catch the failing mock test and test it’s error state
subblue · Jan 5, 2016 · 77fc9a0 · 77fc9a0
1 parent 461f7ca
commit 77fc9a0
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
@@ -36,6 +36,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
@@ -86,12 +86,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);
     }
   });
 });