Merge branch 'master' of github.com:davidkpiano/react-redux-form

Author: davidkpiano

src/reducers/forms-reducer.js

@@ -88,7 +88,7 @@ function createFormCombiner(strategy = defaultStrategy) {
       [key]: (state, action) => strategy.formReducer(model, initialFormState, {
         plugins,
         ...formOptions,
-      })(state, { ...action, state: modelCache[cacheId] }),
+      })(state, { ...action, state: Object.assign({}, modelCache[cacheId]) }),
     };
   }
 

test/model-actions-spec.js

@@ -1,11 +1,15 @@
 import { assert } from 'chai';
 import { combineReducers } from 'redux';
 import Immutable from 'immutable';
-import { actions, modelReducer, formReducer, track, combineForms } from '../src';
+import thunk from 'redux-thunk';
+import { actions, modelReducer, formReducer, track, combineForms, createForms } from '../src';
 import {
   actions as immutableActions,
   modelReducer as immutableModelReducer,
 } from '../immutable';
+import createTestStore from 'redux-test-store';
+
+import { testCreateStore } from './utils';
 
 describe('model actions', () => {
   const testItems = [
@@ -112,6 +116,28 @@ describe('model actions', () => {
       assert.equal(actual.fooForm.$form.pristine, true);
       assert.equal(actual.fooForm.$form.touched, false);
     });
+
+    it('after load model actions on non-set field should not fail', () => {
+      const reducer = createForms({
+        foo: {
+          bar: '',
+        },
+      });
+
+      const store = createTestStore(testCreateStore({
+        ...reducer,
+      }, thunk));
+
+      store.dispatch(actions.load('foo', { bar: 'string' }));
+      const actual = store.getState();
+      assert.deepEqual(actual.foo, { bar: 'string' });
+      assert.equal(actual.forms.$form.pristine, true);
+      assert.equal(actual.forms.$form.touched, false);
+
+
+      store.dispatch(actions.focus('foo.notExist', { silent: true }));
+      store.dispatch(actions.blur('foo.notExist'));
+    });
   });
 
   describe('change()', () => {