Merge pull request #103 from edx/fix-node-6-linting-errors

fix(eslint): fix Node 6 eslint errors

src/Modal/Modal.stories.jsx

@@ -51,10 +51,6 @@ ModalWrapper.propTypes = {
   body: PropTypes.oneOfType([PropTypes.string, PropTypes.element]).isRequired,
 };
 
-ModalWrapper.defaultProps = {
-  open: false,
-};
-
 storiesOf('Modal', module)
   .add('basic usage', () => (
     <Modal

src/Modal/index.jsx

@@ -89,13 +89,12 @@ class Modal extends React.Component {
   }
 
   renderBody() {
-    let body;
+    let { body } = this.props;
 
-    if (typeof this.props.body === 'string') {
-      body = <p>{this.props.body}</p>;
-    } else {
-      body = this.props.body;
+    if (typeof body === 'string') {
+      body = <p>{body}</p>;
     }
+
     return body;
   }
 

src/StatusAlert/StatusAlert.test.jsx

@@ -117,10 +117,7 @@ describe('<StatusAlert />', () => {
   });
   describe('focus functions properly', () => {
     it('focus function changes focus', () => {
-      wrapper = mount(<div>
-        <Button label="test" />
-        <StatusAlert {...defaultProps} />
-      </div>);
+      wrapper = mount(<div><Button label="test" /><StatusAlert {...defaultProps} /></div>);
 
       const buttons = wrapper.find('button');
 

src/Table/Table.test.jsx

@@ -12,9 +12,15 @@ const props = {
     { key: 'i', label: 'Imaginary Number' },
   ],
   data: [
-    { sq: 1, num: 1, x2: 2, i: 'i' },
-    { sq: 4, num: 2, x2: 4, i: '2i' },
-    { sq: 9, num: 3, x2: 6, i: '3i' },
+    {
+      sq: 1, num: 1, x2: 2, i: 'i',
+    },
+    {
+      sq: 4, num: 2, x2: 4, i: '2i',
+    },
+    {
+      sq: 9, num: 3, x2: 6, i: '3i',
+    },
   ],
 };
 
@@ -62,7 +68,7 @@ describe('<Table />', () => {
     it('with data in the same order as the columns', () => {
       wrapper.find('tr').at(1).find('td').forEach((td, i) => {
         let parsed = Number(td.text());
-        if (isNaN(parsed)) { parsed = td.text(); }
+        if (Number.isNaN(parsed)) { parsed = td.text(); }
         expect(parsed).toEqual(props.data[0][props.columns[i].key]);
       });
     });

src/Tabs/index.jsx

+// TODO: @jaebradley fix these eslint errors
+
+/* eslint-disable jsx-a11y/click-events-have-key-events */
+/* eslint-disable jsx-a11y/anchor-is-valid */
+
 import React from 'react';
 import classNames from 'classnames';
 import PropTypes from 'prop-types';