Added test to (attempt to) demonstrate #249

Author: davidkpiano

.eslintrc

@@ -11,6 +11,12 @@
         "capIsNewExceptions": ["List", "Map"]
       }
     ],
-    "react/prefer-stateless-function": 0
+    "react/prefer-stateless-function": 0,
+    "no-unused-vars": [
+      "error",
+      {
+        "varsIgnorePattern": "^_"
+      }
+    ]
   }
 }

test/field-component-spec.js

@@ -1281,7 +1281,7 @@ describe('<Field /> component', () => {
   });
 
   describe('event handlers on control', () => {
-    const reducer = modelReducer('test', { foo: '' });
+    const reducer = modelReducer('test', { foo: '', bar: '' });
     const store = applyMiddleware(thunk)(createStore)(combineReducers({
       test: reducer,
     }));
@@ -1316,6 +1316,36 @@ describe('<Field /> component', () => {
         'testing');
     });
 
+    it('should not execute custom onChange functions of unchanged controls', () => {
+      const onChangeFn = (val) => val;
+      const onChangeFnSpy = sinon.spy(onChangeFn);
+
+      const field = TestUtils.renderIntoDocument(
+        <Provider store={store}>
+          <div>
+            <Field
+              model="test.foo"
+            >
+              <input type="text" onChange={onChangeFnSpy} />
+            </Field>
+            <Field
+              model="test.bar"
+            >
+              <input type="text" />
+            </Field>
+          </div>
+        </Provider>
+      );
+
+      const [_, controlBar] = TestUtils.scryRenderedDOMComponentsWithTag(field, 'input');
+
+      controlBar.value = 'testing';
+
+      TestUtils.Simulate.change(controlBar);
+
+      assert.isFalse(onChangeFnSpy.called);
+    });
+
     it('should persist and return the event even when not returned', () => {
       const onChangeFn = () => {};
       const onChangeFnSpy = sinon.spy(onChangeFn);