Merge pull request #49 from rsuite/fix-RadioGroup

Fix event is not defined in the RadioGroup's onChange

.travis.yml

@@ -5,6 +5,7 @@ node_js:
 
 env:
   - BROWSER=ChromeCi
+  - BROWSER=Firefox
 
 cache:
   directories:

src/RadioGroup.js

@@ -34,7 +34,7 @@ class RadioGroup extends React.Component<Props, State> {
     return _.isUndefined(value) ? this.state.value : value;
   }
 
-  handleChange = (nextValue: any, event: SyntheticInputEvent<*>) => {
+  handleChange = (nextValue: any, itemChecked: boolean, event: SyntheticInputEvent<*>) => {
     const { onChange } = this.props;
     this.setState({ value: nextValue });
     onChange && onChange(nextValue, event);

test/RadioGroupSpec.js

@@ -93,6 +93,27 @@ describe('RadioGroup', () => {
     ReactTestUtils.Simulate.change(radios[2].querySelector('input'));
   });
 
+  it('Should call onChange callback and return correct parameters', done => {
+    const instance = ReactTestUtils.renderIntoDocument(
+      <RadioGroup
+        name="test"
+        onChange={(value, event) => {
+          if (value === 3 && event.target.name === 'test') {
+            done();
+          }
+        }}
+      >
+        <Radio value={1}>Test1</Radio>
+        <Radio value={2}>Test2</Radio>
+        <Radio value={3}>Test2</Radio>
+        <Radio value={4}>Test2</Radio>
+      </RadioGroup>
+    );
+
+    const radios = findDOMNode(instance).querySelectorAll('.rs-radio');
+    ReactTestUtils.Simulate.change(radios[2].querySelector('input'));
+  });
+
   it('Should have a custom className', () => {
     const instance = ReactTestUtils.renderIntoDocument(<RadioGroup className="custom" />);
     assert.ok(findDOMNode(instance).className.match(/\bcustom\b/));