Fix tests

Author: danez

.gitignore

@@ -1,7 +1,4 @@
-.idea
-*.iml
 build/
 node_modules/
 npm-debug.log
 lib/
-yarn.lock

package.json

@@ -62,6 +62,7 @@
     "react-addons-test-utils": "^15.0.0",
     "react-dom": "^15.0.0",
     "react-modal": "^1.3.0",
+    "react-test-renderer": "^15.5.4",
     "rimraf": "^2.5.2",
     "webpack": "^1.13.1",
     "webpack-dev-server": "^1.14.1"

src/components/Tab.js

@@ -55,6 +55,7 @@ export default class Tab extends Component {
       id,
       panelId,
       selected,
+      tabRef,
       ...attributes } = this.props;
 
     return (
@@ -68,7 +69,7 @@ export default class Tab extends Component {
             [disabledTabClassName]: disabled,
           }
         )}
-        ref={(node) => { this.node = node; this.props.tabRef(node); }}
+        ref={(node) => { this.node = node; if (tabRef) tabRef(node); }}
         role="tab"
         id={id}
         aria-selected={selected ? 'true' : 'false'}

src/components/__tests__/Tab-test.js

@@ -4,6 +4,12 @@ import { shallow } from 'enzyme';
 import Tab from '../Tab';
 
 describe('<Tab />', () => {
+  beforeAll(() => {
+    console.error = (error) => {
+      throw new Error(error);
+    };
+  });
+
   it('should have sane defaults', () => {
     const wrapper = shallow(<Tab />);
 

src/components/__tests__/TabList-test.js

@@ -3,13 +3,20 @@ import React from 'react';
 import { shallow, mount } from 'enzyme';
 import Tab from '../Tab';
 import TabList from '../TabList';
+import TabPanel from '../TabPanel';
 import Tabs from '../Tabs';
 
 function hasClassAt(wrapper, index, className) {
   return wrapper.childAt(index).find('li').hasClass(className);
 }
 
 describe('<TabList />', () => {
+  beforeAll(() => {
+    console.error = (error) => {
+      throw new Error(error);
+    };
+  });
+
   it('should have sane defaults', () => {
     const wrapper = shallow(<TabList />);
 
@@ -44,6 +51,8 @@ describe('<TabList />', () => {
           <Tab>Foo</Tab>
           <Tab disabled>Bar</Tab>
         </TabList>
+        <TabPanel>Foo</TabPanel>
+        <TabPanel>Bar</TabPanel>
       </Tabs>
     );
 
@@ -59,6 +68,8 @@ describe('<TabList />', () => {
           <Tab>Foo</Tab>
           <Tab disabled>Bar</Tab>
         </TabList>
+        <TabPanel>Foo</TabPanel>
+        <TabPanel>Bar</TabPanel>
       </Tabs>
     );
 

src/components/__tests__/TabPanel-test.js

@@ -3,7 +3,13 @@ import React from 'react';
 import { shallow } from 'enzyme';
 import TabPanel from '../TabPanel';
 
-describe('Tab', () => {
+describe('<TabPanel />', () => {
+  beforeAll(() => {
+    console.error = (error) => {
+      throw new Error(error);
+    };
+  });
+
   it('should have sane defaults', () => {
     const wrapper = shallow(<TabPanel>Hola</TabPanel>);
 

src/components/__tests__/Tabs-test.js

@@ -38,7 +38,13 @@ function assertTabSelected(wrapper, index) {
   expect(panel.prop('selected')).toBe(true);
 }
 
-describe('react-tabs', () => {
+describe('<Tabs />', () => {
+  beforeAll(() => {
+    console.error = (error) => {
+      throw new Error(error);
+    };
+  });
+
   describe('props', () => {
     it('should default to selectedIndex being 0', () => {
       const wrapper = shallow(createTabs());
@@ -211,20 +217,25 @@ describe('react-tabs', () => {
 
   describe('validation', () => {
     it('should result with warning when tabs/panels are imbalanced', () => {
+      const oldConsoleError = console.error; // eslint-disable-line no-console
+      console.error = () => {}; // eslint-disable-line no-console
       const wrapper = shallow(
         <Tabs>
           <TabList>
             <Tab>Foo</Tab>
           </TabList>
         </Tabs>
       );
+      console.error = oldConsoleError; // eslint-disable-line no-console
 
       const result = Tabs.propTypes.children(wrapper.props(), 'children', 'Tabs');
       expect(result instanceof Error).toBe(true);
     });
 
     it(`should result with warning when tabs/panels are imbalanced and
         it should ignore non tab children`, () => {
+      const oldConsoleError = console.error; // eslint-disable-line no-console
+      console.error = () => {}; // eslint-disable-line no-console
       const wrapper = shallow(
         <Tabs>
           <TabList>
@@ -236,6 +247,7 @@ describe('react-tabs', () => {
           <TabPanel>Hello Bar</TabPanel>
         </Tabs>
       );
+      console.error = oldConsoleError; // eslint-disable-line no-console
 
       const result = Tabs.propTypes.children(wrapper.props(), 'children', 'Tabs');
       expect(result instanceof Error).toBe(true);