Handle thrown errors correctly. Fixes #877

Author: davidkpiano

src/actions/field-actions.js

@@ -214,7 +214,11 @@ function createFieldActions(s = defaultStrategies) {
           setSubmitted(model, true),
           setValidity(model, response),
         ]));
-      }).catch(error => {
+      }).catch(rejection => {
+        const error = rejection instanceof Error
+          ? rejection.message
+          : rejection;
+
         dispatch(batch(model, [
           setSubmitFailed(model),
           errorsAction(model, error, { async: true }),

test/field-actions-spec.js

@@ -1510,6 +1510,25 @@ Object.keys(testContexts).forEach((testKey) => {
 
         store.dispatch(action);
       });
+
+      it('should capture thrown errors', (done) => {
+        const store = createTestStore(testCreateStore({
+          testForm: formReducer('test'),
+        }), done);
+
+        store.when(actionTypes.SET_ERRORS, (_, action) => {
+          assert.containSubset(action, {
+            model: 'test',
+            errors: 'Async error',
+          });
+        });
+
+        const action = actions.submit('test', new Promise(() => {
+          throw new Error('Async error');
+        }));
+
+        store.dispatch(action);
+      });
     });
 
     describe('validate() (thunk)', () => {