Update dependencies; Fix linting; Add enzyme adapter;

Author: flexdinesh

.eslintrc

@@ -20,6 +20,7 @@
       }
     },
     "rules": {
+      "function-paren-newline": "off",
       "no-param-reassign": "off",
       "arrow-parens": [
         "error",
@@ -51,7 +52,8 @@
       ],
       "jsx-a11y/aria-props": 2,
       "jsx-a11y/heading-has-content": 0,
-      "jsx-a11y/href-no-hash": 2,
+      "jsx-a11y/href-no-hash": "off",
+      "jsx-a11y/anchor-is-valid": "off",
       "jsx-a11y/label-has-for": 2,
       "jsx-a11y/mouse-events-have-key-events": 2,
       "jsx-a11y/role-has-required-aria-props": 2,
@@ -63,6 +65,7 @@
       "no-use-before-define": 0,
       "prefer-template": 2,
       "react/forbid-prop-types": 0,
+      "react/jsx-curly-brace-presence": "off",
       "react/jsx-first-prop-new-line": [
         2,
         "multiline"

app/components/Button/tests/index.test.js

@@ -9,12 +9,13 @@ import Button from '../index';
 
 const handleRoute = () => {};
 const href = 'http://twitter.com/flexdinesh';
-const children = (<h1>Test</h1>);
-const renderComponent = (props = {}) => mount(
-  <Button href={href} {...props}>
-    {children}
-  </Button>
-);
+const children = <h1>Test</h1>;
+const renderComponent = (props = {}) =>
+  mount(
+    <Button href={href} {...props}>
+      {children}
+    </Button>
+  );
 
 describe('<Button />', () => {
   it('should render an <a> tag if no route is specified', () => {

app/components/Footer/tests/index.test.js

@@ -5,12 +5,12 @@ import Footer from '../index';
 
 describe('<Footer />', () => {
   it('should render the copyright notice', () => {
-    const renderedComponent = shallow(
-      <Footer />
-    );
-    expect(renderedComponent.contains(
-      <section>This project is licensed under the MIT license.</section>
-    )).toBe(true);
+    const renderedComponent = shallow(<Footer />);
+    expect(
+      renderedComponent.contains(
+        <section>This project is licensed under the MIT license.</section>
+      )
+    ).toBe(true);
   });
 
   it('should render the credits', () => {

app/components/Header/Header.js

@@ -12,8 +12,12 @@ class Header extends React.Component { // eslint-disable-line react/prefer-state
           <img src={Banner} alt="react-redux-boilerplate - Logo" />
         </a>
         <div className="nav-bar">
-          <Link className="router-link" to="/">Home</Link>
-          <Link className="router-link" to="/features">Features</Link>
+          <Link className="router-link" to="/">
+            Home
+          </Link>
+          <Link className="router-link" to="/features">
+            Features
+          </Link>
         </div>
       </div>
     );

app/components/Header/tests/index.test.js

@@ -5,9 +5,7 @@ import Header from '../index';
 
 describe('<Header />', () => {
   it('should render a div', () => {
-    const renderedComponent = shallow(
-      <Header />
-    );
+    const renderedComponent = shallow(<Header />);
     expect(renderedComponent.length).toEqual(1);
   });
 });

app/components/Icons/IssueIcon/tests/index.test.js

@@ -5,9 +5,7 @@ import IssueIcon from '../index';
 
 describe('<IssueIcon />', () => {
   it('should render a SVG', () => {
-    const renderedComponent = shallow(
-      <IssueIcon />
-    );
+    const renderedComponent = shallow(<IssueIcon />);
     expect(renderedComponent.find('svg').length).toBe(1);
   });
 });

app/components/List/tests/index.test.js

@@ -6,17 +6,12 @@ import List from '../index';
 
 describe('<List />', () => {
   it('should render the component if no items are passed', () => {
-    const renderedComponent = shallow(
-      <List component={ListItem} />
-    );
+    const renderedComponent = shallow(<List component={ListItem} />);
     expect(renderedComponent.find(ListItem)).toBeDefined();
   });
 
   it('should pass all items props to rendered component', () => {
-    const items = [
-      { id: 1, name: 'Hello' },
-      { id: 2, name: 'World' },
-    ];
+    const items = [{ id: 1, name: 'Hello' }, { id: 2, name: 'World' }];
 
     const component = ({ item }) => <ListItem>{item.name}</ListItem>; // eslint-disable-line react/prop-types
 

app/components/ListItem/ListItem.js

@@ -5,14 +5,12 @@ import './style.scss';
 
 const ListItem = (props) => (
   <div className="list-item-wrapper">
-    <li className="list-item">
-      {props.item}
-    </li>
+    <li className="list-item">{props.item}</li>
   </div>
-  );
+);
 
 ListItem.propTypes = {
-  item: PropTypes.any,
+  item: PropTypes.any
 };
 
 export default ListItem;

app/components/ListItem/tests/index.test.js

@@ -10,10 +10,8 @@ describe('<ListItem />', () => {
   });
 
   it('should render the content passed to it', () => {
-    const content = (<div>Hello world!</div>);
-    const renderedComponent = mount(
-      <ListItem item={content} />
-    );
+    const content = <div>Hello world!</div>;
+    const renderedComponent = mount(<ListItem item={content} />);
     expect(renderedComponent.contains(content)).toBe(true);
   });
 });

app/components/LoadingIndicator/tests/index.test.js

@@ -5,9 +5,7 @@ import LoadingIndicator from '../index';
 
 describe('<LoadingIndicator />', () => {
   it('should render 13 divs', () => {
-    const renderedComponent = render(
-      <LoadingIndicator />
-    );
+    const renderedComponent = render(<LoadingIndicator />);
     expect(renderedComponent.length).toBe(1);
   });
 });

app/components/ReposList/ReposList.js

@@ -28,7 +28,7 @@ const ReposList = ({ loading, error, repos }) => {
 ReposList.propTypes = {
   loading: PropTypes.bool,
   error: PropTypes.any,
-  repos: PropTypes.any,
+  repos: PropTypes.any
 };
 
 export default ReposList;

app/components/ReposList/tests/index.test.js

@@ -8,49 +8,45 @@ import ReposList from '../index';
 
 describe('<ReposList />', () => {
   it('should render the loading indicator when its loading', () => {
-    const renderedComponent = shallow(
-      <ReposList loading />
-    );
-    expect(renderedComponent.contains(<List component={LoadingIndicator} />)).toEqual(true);
+    const renderedComponent = shallow(<ReposList loading />);
+    expect(
+      renderedComponent.contains(<List component={LoadingIndicator} />)
+    ).toEqual(true);
   });
 
   it('should render an error if loading failed', () => {
     const renderedComponent = mount(
-      <ReposList
-        loading={false}
-        error={{ message: 'Loading failed!' }}
-      />
+      <ReposList loading={false} error={{ message: 'Loading failed!' }} />
     );
     expect(renderedComponent.text()).toMatch(/Something went wrong/);
   });
 
   it('should render the repositories if loading was successful', () => {
-    const repos = [{
-      owner: {
-        login: 'flexdinesh',
-      },
-      html_url: 'https://github.com/flexdinesh/react-redux-boilerplate',
-      name: 'react-redux-boilerplate',
-      open_issues_count: 20,
-      full_name: 'flexdinesh/react-redux-boilerplate',
-    }];
+    const repos = [
+      {
+        owner: {
+          login: 'flexdinesh'
+        },
+        html_url: 'https://github.com/flexdinesh/react-redux-boilerplate',
+        name: 'react-redux-boilerplate',
+        open_issues_count: 20,
+        full_name: 'flexdinesh/react-redux-boilerplate'
+      }
+    ];
     const renderedComponent = shallow(
-      <ReposList
-        repos={repos}
-        error={false}
-      />
+      <ReposList repos={repos} error={false} />
     );
 
-    expect(renderedComponent.contains(<List items={repos} component={RepoListItem} />)).toEqual(true);
+    expect(
+      renderedComponent.contains(
+        <List items={repos} component={RepoListItem} />
+      )
+    ).toEqual(true);
   });
 
   it('should not render anything if nothing interesting is provided', () => {
     const renderedComponent = shallow(
-      <ReposList
-        repos={false}
-        error={false}
-        loading={false}
-      />
+      <ReposList repos={false} error={false} loading={false} />
     );
 
     expect(renderedComponent.html()).toEqual(null);

app/containers/App/tests/index.test.js

@@ -8,23 +8,17 @@ import App from '../index';
 
 describe('<App />', () => {
   it('should render the header', () => {
-    const renderedComponent = shallow(
-      <App />
-    );
+    const renderedComponent = shallow(<App />);
     expect(renderedComponent.find(Header).length).toBe(1);
   });
 
   it('should render some routes', () => {
-    const renderedComponent = shallow(
-      <App />
-    );
+    const renderedComponent = shallow(<App />);
     expect(renderedComponent.find(Route).length).not.toBe(0);
   });
 
   it('should render the footer', () => {
-    const renderedComponent = shallow(
-      <App />
-    );
+    const renderedComponent = shallow(<App />);
     expect(renderedComponent.find(Footer).length).toBe(1);
   });
 });

app/containers/FeaturePage/tests/index.test.js

@@ -5,18 +5,12 @@ import FeaturePage from '../index';
 
 describe('<FeaturePage />', () => {
   it('should render its heading', () => {
-    const renderedComponent = shallow(
-      <FeaturePage />
-    );
-    expect(renderedComponent.contains(
-      <h1>Features</h1>
-    )).toBe(true);
+    const renderedComponent = shallow(<FeaturePage />);
+    expect(renderedComponent.contains(<h1>Features</h1>)).toBe(true);
   });
 
   it('should never re-render the component', () => {
-    const renderedComponent = shallow(
-      <FeaturePage />
-    );
+    const renderedComponent = shallow(<FeaturePage />);
     const inst = renderedComponent.instance();
     expect(inst.shouldComponentUpdate()).toBe(false);
   });

app/containers/HomePage/actions.js

@@ -15,9 +15,7 @@
  *    }
  */
 
-import {
-  CHANGE_USERNAME,
-} from './constants';
+import { CHANGE_USERNAME } from './constants';
 
 /**
  * Changes the input field of the form
@@ -29,6 +27,6 @@ import {
 export function changeUsername(name) {
   return {
     type: CHANGE_USERNAME,
-    name,
+    name
   };
 }

app/containers/HomePage/reducer.js

@@ -11,22 +11,18 @@
  */
 import { fromJS } from 'immutable';
 
-import {
-  CHANGE_USERNAME,
-} from './constants';
+import { CHANGE_USERNAME } from './constants';
 
 // The initial state of the App
 const initialState = fromJS({
-  username: '',
+  username: ''
 });
 
 function homeReducer(state = initialState, action) {
   switch (action.type) {
     case CHANGE_USERNAME:
-
       // Delete prefixed '@' from the github username
-      return state
-        .set('username', action.name.replace(/@/gi, ''));
+      return state.set('username', action.name.replace(/@/gi, ''));
     default:
       return state;
   }

app/containers/HomePage/tests/actions.test.js

@@ -1,18 +1,14 @@
-import {
-  CHANGE_USERNAME,
-} from '../constants';
+import { CHANGE_USERNAME } from '../constants';
 
-import {
-  changeUsername,
-} from '../actions';
+import { changeUsername } from '../actions';
 
 describe('Home Actions', () => {
   describe('changeUsername', () => {
     it('should return the correct type and the passed name', () => {
       const fixture = 'Max';
       const expectedResult = {
         type: CHANGE_USERNAME,
-        name: fixture,
+        name: fixture
       };
 
       expect(changeUsername(fixture)).toEqual(expectedResult);

app/containers/HomePage/tests/index.test.js

@@ -16,7 +16,9 @@ describe('<HomePage />', () => {
     const renderedComponent = shallow(
       <HomePage loading error={false} repos={[]} />
     );
-    expect(renderedComponent.contains(<ReposList loading error={false} repos={[]} />)).toEqual(true);
+    expect(
+      renderedComponent.contains(<ReposList loading error={false} repos={[]} />)
+    ).toEqual(true);
   });
 
   it('should render fetch the repos on mount if a username exists', () => {
@@ -33,12 +35,7 @@ describe('<HomePage />', () => {
 
   it('should not call onSubmitForm if username is an empty string', () => {
     const submitSpy = jest.fn();
-    mount(
-      <HomePage
-        onChangeUsername={() => {}}
-        onSubmitForm={submitSpy}
-      />
-    );
+    mount(<HomePage onChangeUsername={() => {}} onSubmitForm={submitSpy} />);
     expect(submitSpy).not.toHaveBeenCalled();
   });