[fixed] Define toggleNavKey in the propTypes

react-bootstrap · Feb 27, 2015 · 25b4143 · 25b4143
1 parent 1a4ae1d
commit 25b4143
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 0 deletions.
diff --git a/src/Navbar.jsx b/src/Navbar.jsx
@@ -22,6 +22,10 @@ var Navbar = React.createClass({
  componentClass: React.PropTypes.node.isRequired,
  brand: React.PropTypes.node,
  toggleButton: React.PropTypes.node,
+ toggleNavKey: React.PropTypes.oneOfType([
+ React.PropTypes.string,
+ React.PropTypes.number
+ ]),
  onToggle: React.PropTypes.func,
  navExpanded: React.PropTypes.bool,
  defaultNavExpanded: React.PropTypes.bool

diff --git a/test/NavbarSpec.jsx b/test/NavbarSpec.jsx
@@ -150,4 +150,17 @@ describe('Nav', function () {
 
  assert.ok(header);
  });
+
+ it('Should add header when toggleNavKey is string', function () {
+ var instance = ReactTestUtils.renderIntoDocument(
+ <Navbar toggleNavKey={'string'}>
+ <Nav eventKey={'string'}>
+ </Nav>
+ </Navbar>
+ );
+
+ var header = ReactTestUtils.findRenderedDOMComponentWithClass(instance, 'navbar-header');
+
+ assert.ok(header);
+ });
 });