Fix empty text onChange call count

package.json

@@ -1,6 +1,6 @@
 {
   "name": "rc-input-number",
-  "version": "3.1.1",
+  "version": "3.1.2",
   "description": "input-number ui component for react(web & react-native)",
   "keywords": [
     "react",

src/mixin.js

@@ -44,10 +44,9 @@ export default {
 
   componentWillReceiveProps(nextProps) {
     if ('value' in nextProps) {
-      const value = this.toNumber(nextProps.value);
       this.setState({
-        inputValue: nextProps.value ? nextProps.value.toString() : nextProps.value,
-        value,
+        inputValue: nextProps.value,
+        value: nextProps.value,
       });
     }
   },
@@ -101,7 +100,7 @@ export default {
   setValue(v, callback) {
     // trigger onChange
     const newValue = this.isNotCompleteNumber(parseFloat(v, 10)) ? undefined : parseFloat(v, 10);
-    const changed = `${newValue}` !== this.state.inputValue;
+    const changed = newValue !== this.state.value;
     if (!('value' in this.props)) {
       this.setState({
         value: v,

tests/index.js

@@ -15,6 +15,7 @@ describe('inputNumber', () => {
   const container = document.createElement('div');
   document.body.appendChild(container);
   let onChangeFirstArgument;
+  let onChangeCallCount = 0;
 
   const Component = React.createClass({
     getInitialState() {
@@ -31,6 +32,7 @@ describe('inputNumber', () => {
     },
     onChange(value) {
       onChangeFirstArgument = value;
+      onChangeCallCount += 1;
       this.setState({ value });
     },
     triggerBoolean(propName) {
@@ -65,10 +67,12 @@ describe('inputNumber', () => {
     example = ReactDOM.render(<Component />, container);
     inputNumber = example.refs.inputNum;
     inputElement = ReactDOM.findDOMNode(inputNumber.refs.input);
+    onChangeCallCount = 0;
   });
 
   afterEach(() => {
     ReactDOM.unmountComponentAtNode(container);
+    onChangeCallCount = 0;
   });
 
   describe('keyboard works', () => {
@@ -567,5 +571,24 @@ describe('inputNumber', () => {
       expect(inputElement.value).to.be('6');
       expect(num).to.be(6);
     });
+
+    it('onChange should not be called when input is not changed', () => {
+      Simulate.focus(inputElement);
+      Simulate.change(inputElement, { target: { value: '1' } });
+      expect(onChangeCallCount).to.be(1);
+      expect(onChangeFirstArgument).to.be(1);
+      Simulate.blur(inputElement);
+      expect(onChangeCallCount).to.be(1);
+      Simulate.focus(inputElement);
+      Simulate.change(inputElement, { target: { value: '' } });
+      expect(onChangeCallCount).to.be(2);
+      expect(onChangeFirstArgument).to.be('');
+      Simulate.blur(inputElement);
+      expect(onChangeCallCount).to.be(3);
+      expect(onChangeFirstArgument).to.be(undefined);
+      Simulate.focus(inputElement);
+      Simulate.blur(inputElement);
+      expect(onChangeCallCount).to.be(3);
+    });
   });
 });