Fix double elements in on change callback (#247)

In case setState was resolved synchronously on change had the new value appended twice

src/ChipInput.js

@@ -337,21 +337,15 @@ class ChipInput extends React.Component {
         if (this.props.value && this.props.onAdd) {
           this.props.onAdd(chip)
         } else {
-          this.setState({ chips: [ ...this.state.chips, chip ] })
-          if (this.props.onChange) {
-            this.props.onChange([ ...this.state.chips, chip ])
-          }
+          this.updateChips([ ...this.state.chips, chip ])
         }
       }
     } else if (chip.trim().length > 0) {
       if (this.props.allowDuplicates || chips.indexOf(chip) === -1) {
         if (this.props.value && this.props.onAdd) {
           this.props.onAdd(chip)
         } else {
-          this.setState({ chips: [ ...this.state.chips, chip ] })
-          if (this.props.onChange) {
-            this.props.onChange([ ...this.state.chips, chip ])
-          }
+          this.updateChips([ ...this.state.chips, chip ])
         }
       }
     } else {
@@ -375,14 +369,18 @@ class ChipInput extends React.Component {
         } else if (this.state.focusedChip > i) {
           focusedChip = this.state.focusedChip - 1
         }
-        this.setState({ chips, focusedChip })
-        if (this.props.onChange) {
-          this.props.onChange(chips)
-        }
+        this.updateChips(chips, { focusedChip })
       }
     }
   }
 
+  updateChips (chips, additionalUpdates = {}) {
+    this.setState({ chips, ...additionalUpdates })
+    if (this.props.onChange) {
+      this.props.onChange(chips)
+    }
+  }
+
   /**
    * Clears the text field for adding new chips.
    * @public

src/ChipInput.spec.js

@@ -386,9 +386,10 @@ describe('blurBehavior modes', () => {
     expect(tree.find('input').getDOMNode().value).toBe('')
     expect(setTimeout).toHaveBeenCalledTimes(1)
 
-    setTimeout(_ => {
-      expect(tree.find('Chip').map((chip) => chip.text())).toEqual(['a', 'b', 'blur'])
-    })
+    expect(handleChange.mock.calls[0][0]).toEqual(['a', 'b', 'blur'])
+
+    tree.update()
+    expect(tree.find('Chip').map((chip) => chip.text())).toEqual(['a', 'b', 'blur'])
   })
 })