Merge pull request react-bootstrap#514 from AlexKVal/fix507

[fixed] Fix for react-bootstrap#507

Author: mtscout6

src/Input.js

@@ -245,7 +245,12 @@ const Input = React.createClass({
       ];
     }
 
-    return <FormGroup {...this.props}>{children}</FormGroup>;
+    if (this.props.type === 'submit') {
+      let {bsStyle, ...other} = this.props; /* eslint no-unused-vars: 0 */
+      return <FormGroup {...other}>{children}</FormGroup>;
+    } else {
+      return <FormGroup {...this.props}>{children}</FormGroup>;
+    }
   }
 });
 

test/InputSpec.js

@@ -40,7 +40,7 @@ describe('Input', function () {
     assert.equal(instance.getValue(), 'v');
   });
 
-  it.skip('renders a submit button element when type=submit', function () {
+  it('renders a submit button element when type=submit', function () {
     let instance = ReactTestUtils.renderIntoDocument(
       <Input type="submit" bsStyle="danger" wrapperClassName='test' />
     );
@@ -50,6 +50,24 @@ describe('Input', function () {
     assert.equal(node.getAttribute('class'), 'btn btn-danger');
   });
 
+  it('must not throw warning when bsStyle=danger and type=submit', function () {
+    ReactTestUtils.renderIntoDocument(
+      <Input type="submit" bsStyle="danger" />
+    );
+
+    console.warn.called.should.be.false;
+  });
+
+  it('throws warning about wrong type for bsStyle=error when type=submit', function () {
+    ReactTestUtils.renderIntoDocument(
+      <Input type="submit" bsStyle="error" />
+    );
+
+    console.warn.called.should.be.true;
+    console.warn.calledWithMatch('propType: Invalid').should.be.true;
+    console.warn.reset(); // reset state for afterEach()
+  });
+
   it('renders a p element when type=static', function () {
     let instance = ReactTestUtils.renderIntoDocument(
       <Input type="static" value="v" />