fix: run lint --fix

Author: okendoken

.eslintrc.js

@@ -65,6 +65,9 @@ module.exports = {
     // https://github.com/kriasoft/react-starter-kit/pull/961
     'react/prefer-stateless-function': 'off',
 
+    'react/jsx-no-bind': 'off',
+    'react/forbid-prop-types': 'off',
+
     // ESLint plugin for prettier formatting
     // https://github.com/prettier/eslint-plugin-prettier
     'prettier/prettier': [

.stylelintrc.js

@@ -10,7 +10,6 @@
 // stylelint configuration
 // https://stylelint.io/user-guide/configuration/
 module.exports = {
-
   // The standard config based on a handful of CSS style guides
   // https://github.com/stylelint/stylelint-config-standard
   extends: 'stylelint-config-standard',
@@ -22,21 +21,27 @@ module.exports = {
   ],
 
   rules: {
-    'property-no-unknown': [true, {
-      ignoreProperties: [
-        // CSS Modules composition
-        // https://github.com/css-modules/css-modules#composition
-        'composes',
-      ],
-    }],
+    'property-no-unknown': [
+      true,
+      {
+        ignoreProperties: [
+          // CSS Modules composition
+          // https://github.com/css-modules/css-modules#composition
+          'composes',
+        ],
+      },
+    ],
 
-    'selector-pseudo-class-no-unknown': [true, {
-      ignorePseudoClasses: [
-        // CSS Modules :global scope
-        // https://github.com/css-modules/css-modules#exceptions
-        'global',
-      ],
-    }],
+    'selector-pseudo-class-no-unknown': [
+      true,
+      {
+        ignorePseudoClasses: [
+          // CSS Modules :global scope
+          // https://github.com/css-modules/css-modules#exceptions
+          'global',
+        ],
+      },
+    ],
 
     // Opinionated rule, you can disable it if you want
     'string-quotes': 'single',

jest.config.js

@@ -22,7 +22,11 @@ module.exports = {
   // collectCoverage: false, // [boolean]
 
   // https://facebook.github.io/jest/docs/en/configuration.html#collectcoveragefrom-array
-  collectCoverageFrom: ['src/**/*.{js,jsx}', '!**/node_modules/**', '!**/vendor/**'],
+  collectCoverageFrom: [
+    'src/**/*.{js,jsx}',
+    '!**/node_modules/**',
+    '!**/vendor/**',
+  ],
 
   // https://facebook.github.io/jest/docs/en/configuration.html#coveragedirectory-string
   coverageDirectory: '<rootDir>/coverage', // [string]
@@ -48,7 +52,8 @@ module.exports = {
   // like images or styles with a single module.
   moduleNameMapper: {
     '\\.(css|less|scss|sss)$': 'identity-obj-proxy',
-    '\\.(jpg|jpeg|png|gif|eot|otf|webp|svg|ttf|woff|woff2|mp4|webm|wav|mp3|m4a|aac|oga)$': 'GlobalImageStub',
+    '\\.(jpg|jpeg|png|gif|eot|otf|webp|svg|ttf|woff|woff2|mp4|webm|wav|mp3|m4a|aac|oga)$':
+      'GlobalImageStub',
   },
 
   // modulePathIgnorePatterns: // [array<string>]
@@ -77,7 +82,8 @@ module.exports = {
 
   transform: {
     '\\.jsx?$': 'babel-jest',
-    '\\.(jpe?g|png|gif|eot|otf|webp|svg|ttf|woff|woff2|mp4|webm|wav|mp3|m4a|aac|oga)$': '<rootDir>/jest/fileTransformer.js',
+    '\\.(jpe?g|png|gif|eot|otf|webp|svg|ttf|woff|woff2|mp4|webm|wav|mp3|m4a|aac|oga)$':
+      '<rootDir>/jest/fileTransformer.js',
   },
 
   // transformIgnorePatterns: // [array<string>]

src/DOMUtils.js

@@ -8,7 +8,9 @@
  */
 
 export function updateTag(tagName, keyName, keyValue, attrName, attrValue) {
-  const node = document.head.querySelector(`${tagName}[${keyName}="${keyValue}"]`);
+  const node = document.head.querySelector(
+    `${tagName}[${keyName}="${keyValue}"]`,
+  );
   if (node && node.getAttribute(attrName) === attrValue) return;
 
   // Remove and create a new tag in order to make it work with bookmarks in Safari

src/actions/posts.js

@@ -6,7 +6,6 @@ export const FETCH_POSTS_REQUEST = 'FETCH_POSTS_REQUEST';
 export const FETCH_POSTS_SUCCESS = 'FETCH_POSTS_SUCCESS';
 export const FETCH_POSTS_FAILURE = 'FETCH_POSTS_FAILURE';
 
-
 function createPostInitial() {
   return {
     type: CREATE_POST_INITIAL,
@@ -41,7 +40,7 @@ function createPostError(message) {
 function requestFetchPosts() {
   return {
     type: FETCH_POSTS_REQUEST,
-    isFetching: true
+    isFetching: true,
   };
 }
 
@@ -61,7 +60,7 @@ function fetchPostsError(message) {
   };
 }
 
-export function createPost(post) {
+export function createPost(postData) {
   const config = {
     method: 'post',
     headers: {
@@ -70,7 +69,7 @@ export function createPost(post) {
     },
     body: JSON.stringify({
       query: `mutation {
-                addPost(title: "${post.title}", content: "${post.content}"){
+                addPost(title: "${postData.title}", content: "${postData.content}"){
                   id,
                   title,
                   content
@@ -80,26 +79,27 @@ export function createPost(post) {
     credentials: 'include',
   };
 
-  return (dispatch) => {
+  return dispatch => {
     // We dispatch requestCreatePost to kickoff the call to the API
-    dispatch(requestCreatePost(post));
+    dispatch(requestCreatePost(postData));
 
     return fetch('/graphql', config)
-      .then(response =>
-        response.json().then(post => ({ post, response })),
-      ).then(({ post, response }) => {
+      .then(response => response.json().then(post => ({ post, response })))
+      .then(({ post, response }) => {
         if (!response.ok) {
           // If there was a problem, we want to
           // dispatch the error condition
           dispatch(createPostError(post.message));
           return Promise.reject(post);
         }
-          // Dispatch the success action
+        // Dispatch the success action
         dispatch(createPostSuccess(post));
         setTimeout(() => {
           dispatch(createPostInitial());
         }, 5000);
-      }).catch(err => console.log('Error: ', err));
+        return Promise.resolve(post);
+      })
+      .catch(err => console.error('Error: ', err));
   };
 }
 
@@ -116,21 +116,27 @@ export function fetchPosts() {
     credentials: 'include',
   };
 
-  return (dispatch) => {
+  return dispatch => {
     dispatch(requestFetchPosts());
 
     return fetch('/graphql', config)
       .then(response =>
-        response.json().then(response => ({ posts: response.data.posts, response })),
-      ).then(({ posts, response }) => {
-        if (!response.data.posts) {
+        response.json().then(responseJson => ({
+          posts: responseJson.data.posts,
+          responseJson,
+        })),
+      )
+      .then(({ posts, responseJson }) => {
+        if (!responseJson.data.posts) {
           // If there was a problem, we want to
           // dispatch the error condition
           dispatch(fetchPostsError(posts.message));
           return Promise.reject(posts);
         }
         // Dispatch the success action
         dispatch(fetchPostsSuccess(posts));
-      }).catch(err => console.log('Error: ', err));
+        return Promise.resolve(posts);
+      })
+      .catch(err => console.error('Error: ', err));
   };
 }

src/actions/user.js

@@ -5,7 +5,6 @@ export const LOGOUT_REQUEST = 'LOGOUT_REQUEST';
 export const LOGOUT_SUCCESS = 'LOGOUT_SUCCESS';
 export const LOGOUT_FAILURE = 'LOGOUT_FAILURE';
 
-
 function requestLogin(creds) {
   return {
     type: LOGIN_REQUEST,
@@ -51,7 +50,7 @@ export function receiveLogout() {
 
 // Logs the user out
 export function logoutUser() {
-  return (dispatch) => {
+  return dispatch => {
     dispatch(requestLogout());
     localStorage.removeItem('id_token');
     document.cookie = 'id_token=;expires=Thu, 01 Jan 1970 00:00:01 GMT;';
@@ -67,14 +66,13 @@ export function loginUser(creds) {
     body: `login=${creds.login}&password=${creds.password}`,
   };
 
-  return (dispatch) => {
+  return dispatch => {
     // We dispatch requestLogin to kickoff the call to the API
     dispatch(requestLogin(creds));
 
     return fetch('/login', config)
-      .then(response =>
-        response.json().then(user => ({ user, response })),
-      ).then(({ user, response }) => {
+      .then(response => response.json().then(user => ({ user, response })))
+      .then(({ user, response }) => {
         if (!response.ok) {
           // If there was a problem, we want to
           // dispatch the error condition
@@ -84,8 +82,10 @@ export function loginUser(creds) {
         // in posts create new action and check http status, if malign logout
         // If login was successful, set the token in local storage
         localStorage.setItem('id_token', user.id_token);
-          // Dispatch the success action
+        // Dispatch the success action
         dispatch(receiveLogin(user));
-      }).catch(err => console.log('Error: ', err));
+        return Promise.resolve(user);
+      })
+      .catch(err => console.error('Error: ', err));
   };
 }

src/client.js

@@ -12,7 +12,6 @@ import React from 'react';
 import ReactDOM from 'react-dom';
 import { Router } from 'react-router';
 import deepForceUpdate from 'react-deep-force-update';
-import queryString from 'query-string';
 import { createPath } from 'history/PathUtils';
 import history from './history';
 import App from './components/App';
@@ -31,7 +30,9 @@ const context = {
   insertCss: (...styles) => {
     // eslint-disable-next-line no-underscore-dangle
     const removeCss = styles.map(x => x._insertCss());
-    return () => { removeCss.forEach(f => f()); };
+    return () => {
+      removeCss.forEach(f => f());
+    };
   },
   // Universal HTTP client
   fetch: createFetch(self.fetch, {
@@ -110,9 +111,7 @@ async function onLocationChange(location, action) {
     }
 
     appInstance = ReactDOM.render(
-      <Router
-        history={history}
-      >
+      <Router history={history}>
         <App store={context.store} context={context} />
       </Router>,
       container,

src/components/App.js

@@ -22,13 +22,11 @@ import loadNotFound from 'bundle-loader?lazy!../pages/notFound/NotFound';
 import LayoutComponent from '../components/Layout/Layout';
 import LoginComponent from '../pages/login/Login';
 
-
 // import { auth } from '../config';
 
 const RegisterBundle = Bundle.generateBundle(loadRegister);
 const NotFoundBundle = Bundle.generateBundle(loadNotFound);
 
-
 const ContextType = {
   // Enables critical path CSS rendering
   // https://github.com/kriasoft/isomorphic-style-loader
@@ -40,44 +38,33 @@ const ContextType = {
   ...ReduxProvider.childContextTypes,
 };
 
-// let isAuthenticated = function() {
-//   let t = jwt.verify(cookie.load('id_token'), auth.jwt.secret);
-//
-//   console.log(t);
-//
-//   return true;
-// };
-
-
-const PrivateRoute = ({ component, isAuthenticated, ...rest }) => (
+/* eslint-disable */
+const PrivateRoute = ({ component, isAuthenticated, ...rest }) =>
   <Route
-    {...rest} render={props => (
-    isAuthenticated ? (
-      React.createElement(component, props)
-    ) : (
-      <Redirect
-        to={{
-          pathname: '/login',
-          state: { from: props.location },
-        }}
-      />
-    )
-  )}
-  />
-);
+    {...rest}
+    render={props =>
+      isAuthenticated
+        ? React.createElement(component, props)
+        : <Redirect
+            to={{
+              pathname: '/login',
+              state: { from: props.location },
+            }}
+          />}
+  />;
+/* eslint-enable */
 
 class App extends React.PureComponent {
-
   static propTypes = {
     context: PropTypes.shape(ContextType),
-    store: PropTypes.any,
+    isAuthenticated: PropTypes.bool,
   };
 
   static defaultProps = {
     context: null,
+    isAuthenticated: false,
   };
 
-
   static contextTypes = {
     router: PropTypes.any,
     store: PropTypes.any,
@@ -94,7 +81,11 @@ class App extends React.PureComponent {
     return (
       <Switch>
         <Route path="/" exact render={() => <Redirect to="/app" />} />
-        <PrivateRoute isAuthenticated={this.props.isAuthenticated} path="/app" component={LayoutComponent} />
+        <PrivateRoute
+          isAuthenticated={this.props.isAuthenticated}
+          path="/app"
+          component={LayoutComponent}
+        />
         <Route path="/login" exact component={LoginComponent} />
         <Route path="/register" exact component={RegisterBundle} />
         <Route component={NotFoundBundle} />