lint fixes, run linting as part of main test

package.json

@@ -5,7 +5,7 @@
   "scripts": {
     "start": "react-scripts start",
     "build": "react-scripts build",
-    "test": "react-scripts test",
+    "test": "npm run lint && react-scripts test",
     "eject": "react-scripts eject",
     "format:js": "prettier --parser typescript --write '**/*.{js,jsx,ts,tsx}'",
     "format:css": "prettier-stylelint --write --quiet 'src/**/*.{css,scss}'",

src/App.css

@@ -15,7 +15,7 @@
     align-items: center;
     justify-content: center;
     font-size: calc(10px + 2vmin);
-    color: white;
+    color: #fff;
 }
 
 .App-link {

src/App.test.tsx

@@ -3,7 +3,7 @@ import ReactDOM from 'react-dom'
 import App from './App'
 
 it('renders without crashing', () => {
-  const div = document.createElement('div')
+    const div = document.createElement('div')
-  ReactDOM.render(<App />, div)
+    ReactDOM.render(<App />, div)
-  ReactDOM.unmountComponentAtNode(div)
+    ReactDOM.unmountComponentAtNode(div)
 })

src/App.tsx

@@ -1,16 +1,16 @@
 import React, { Component } from 'react'
+import './App.css'
 import { User, userDefults } from './context/User'
 import Routes from './Routes'
-import './App.css'
 
 class App extends Component {
-  render() {
+    public render() {
-    return (
+        return (
-      <User.Provider value={userDefults}>
+            <User.Provider value={userDefults}>
-        <Routes />
+                <Routes />
-      </User.Provider>
+            </User.Provider>
-    )
+        )
-  }
+    }
 }
 
 export default App

src/Routes.tsx

@@ -4,12 +4,12 @@ import Home from './pages/Home'
 import NotFound from './pages/NotFound'
 
 const Routes = () => (
-  <Router>
+    <Router>
-    <Switch>
+        <Switch>
-      <Route exact component={Home} path="/" />
+            <Route exact component={Home} path="/" />
-      <Route component={NotFound} />
+            <Route component={NotFound} />
-    </Switch>
+        </Switch>
-  </Router>
+    </Router>
 )
 
 export default Routes

src/context/User.ts

@@ -1,7 +1,7 @@
 import React from 'react'
 
 export const userDefults = {
-  logged: false
+    logged: false
 }
 
 export const User = React.createContext(userDefults)

src/index.css

@@ -1,9 +1,9 @@
 body {
     margin: 0;
     padding: 0;
-    font-family: -apple-system, BlinkMacSystemFont, 'Segoe UI', 'Roboto',
+    font-family: -apple-system, BlinkMacSystemFont, 'Segoe UI', Roboto, Oxygen,
-        'Oxygen', 'Ubuntu', 'Cantarell', 'Fira Sans', 'Droid Sans',
+        Ubuntu, Cantarell, 'Fira Sans', 'Droid Sans', 'Helvetica Neue',
-        'Helvetica Neue', sans-serif;
+        sans-serif;
     -webkit-font-smoothing: antialiased;
     -moz-osx-font-smoothing: grayscale;
 }

src/index.tsx

@@ -1,7 +1,7 @@
 import React from 'react'
 import ReactDOM from 'react-dom'
-import './index.css'
 import App from './App'
+import './index.css'
 import * as serviceWorker from './serviceWorker'
 
 ReactDOM.render(<App />, document.getElementById('root'))

src/pages/Home.tsx

@@ -1,9 +1,9 @@
 import React, { Component } from 'react'
 
 class Home extends Component {
-  render() {
+    public render() {
-    return <div>Home</div>
+        return <div>Home</div>
-  }
+    }
 }
 
 export default Home

src/pages/NotFound.tsx

@@ -1,9 +1,9 @@
 import React, { Component } from 'react'
 
 class NotFound extends Component {
-  render() {
+    public render() {
-    return <div>Not found</div>
+        return <div>Not found</div>
-  }
+    }
 }
 
 export default NotFound

src/serviceWorker.ts

@@ -1,3 +1,5 @@
+/* tslint:disable */
+
 // This optional code is used to register a service worker.
 // register() is not called by default.
 
@@ -11,133 +13,134 @@
 // opt-in, read http://bit.ly/CRA-PWA
 
 const isLocalhost = Boolean(
-  window.location.hostname === 'localhost' ||
+    window.location.hostname === 'localhost' ||
-    // [::1] is the IPv6 localhost address.
+        // [::1] is the IPv6 localhost address.
-    window.location.hostname === '[::1]' ||
+        window.location.hostname === '[::1]' ||
-    // 127.0.0.1/8 is considered localhost for IPv4.
+        // 127.0.0.1/8 is considered localhost for IPv4.
-    window.location.hostname.match(
+        window.location.hostname.match(
-      /^127(?:\.(?:25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)){3}$/
+            /^127(?:\.(?:25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)){3}$/
-    )
+        )
 )
 
 type Config = {
-  onSuccess?: (registration: ServiceWorkerRegistration) => void
+    onSuccess?: (registration: ServiceWorkerRegistration) => void
-  onUpdate?: (registration: ServiceWorkerRegistration) => void
+    onUpdate?: (registration: ServiceWorkerRegistration) => void
 }
 
 export function register(config?: Config) {
-  if (process.env.NODE_ENV === 'production' && 'serviceWorker' in navigator) {
+    if (process.env.NODE_ENV === 'production' && 'serviceWorker' in navigator) {
-    // The URL constructor is available in all browsers that support SW.
+        // The URL constructor is available in all browsers that support SW.
-    const publicUrl = new URL(
+        const publicUrl = new URL(
-      (process as { env: { [key: string]: string } }).env.PUBLIC_URL,
+            (process as { env: { [key: string]: string } }).env.PUBLIC_URL,
-      window.location.href
+            window.location.href
-    )
+        )
-    if (publicUrl.origin !== window.location.origin) {
+        if (publicUrl.origin !== window.location.origin) {
-      // Our service worker won't work if PUBLIC_URL is on a different origin
+            // Our service worker won't work if PUBLIC_URL is on a different origin
-      // from what our page is served on. This might happen if a CDN is used to
+            // from what our page is served on. This might happen if a CDN is used to
-      // serve assets; see https://github.com/facebook/create-react-app/issues/2374
+            // serve assets; see https://github.com/facebook/create-react-app/issues/2374
-      return
+            return
-    }
+        }
 
-    window.addEventListener('load', () => {
+        window.addEventListener('load', () => {
-      const swUrl = `${process.env.PUBLIC_URL}/service-worker.js`
+            const swUrl = `${process.env.PUBLIC_URL}/service-worker.js`
 
-      if (isLocalhost) {
+            if (isLocalhost) {
-        // This is running on localhost. Let's check if a service worker still exists or not.
+                // This is running on localhost. Let's check if a service worker still exists or not.
-        checkValidServiceWorker(swUrl, config)
+                checkValidServiceWorker(swUrl, config)
 
-        // Add some additional logging to localhost, pointing developers to the
+                // Add some additional logging to localhost, pointing developers to the
-        // service worker/PWA documentation.
+                // service worker/PWA documentation.
-        navigator.serviceWorker.ready.then(() => {
+                navigator.serviceWorker.ready.then(() => {
-          console.log(
+                    console.log(
-            'This web app is being served cache-first by a service ' +
+                        'This web app is being served cache-first by a service ' +
-              'worker. To learn more, visit http://bit.ly/CRA-PWA'
+                            'worker. To learn more, visit http://bit.ly/CRA-PWA'
-          )
+                    )
+                })
+            } else {
+                // Is not localhost. Just register service worker
+                registerValidSW(swUrl, config)
+            }
         })
-      } else {
+    }
-        // Is not localhost. Just register service worker
-        registerValidSW(swUrl, config)
-      }
-    })
-  }
 }
 
 function registerValidSW(swUrl: string, config?: Config) {
-  navigator.serviceWorker
+    navigator.serviceWorker
-    .register(swUrl)
+        .register(swUrl)
-    .then(registration => {
+        .then(registration => {
-      registration.onupdatefound = () => {
+            registration.onupdatefound = () => {
-        const installingWorker = registration.installing
+                const installingWorker = registration.installing
-        if (installingWorker == null) {
+                if (installingWorker == null) {
-          return
+                    return
-        }
+                }
-        installingWorker.onstatechange = () => {
+                installingWorker.onstatechange = () => {
-          if (installingWorker.state === 'installed') {
+                    if (installingWorker.state === 'installed') {
-            if (navigator.serviceWorker.controller) {
+                        if (navigator.serviceWorker.controller) {
-              // At this point, the updated precached content has been fetched,
+                            // At this point, the updated precached content has been fetched,
-              // but the previous service worker will still serve the older
+                            // but the previous service worker will still serve the older
-              // content until all client tabs are closed.
+                            // content until all client tabs are closed.
-              console.log(
+                            console.log(
-                'New content is available and will be used when all ' +
+                                'New content is available and will be used when all ' +
-                  'tabs for this page are closed. See http://bit.ly/CRA-PWA.'
+                                    'tabs for this page are closed. See http://bit.ly/CRA-PWA.'
-              )
+                            )
 
-              // Execute callback
+                            // Execute callback
-              if (config && config.onUpdate) {
+                            if (config && config.onUpdate) {
-                config.onUpdate(registration)
+                                config.onUpdate(registration)
-              }
+                            }
-            } else {
+                        } else {
-              // At this point, everything has been precached.
+                            // At this point, everything has been precached.
-              // It's the perfect time to display a
+                            // It's the perfect time to display a
-              // "Content is cached for offline use." message.
+                            // "Content is cached for offline use." message.
-              console.log('Content is cached for offline use.')
+                            console.log('Content is cached for offline use.')
 
-              // Execute callback
+                            // Execute callback
-              if (config && config.onSuccess) {
+                            if (config && config.onSuccess) {
-                config.onSuccess(registration)
+                                config.onSuccess(registration)
-              }
+                            }
+                        }
+                    }
+                }
             }
-          }
+        })
-        }
+        .catch(error => {
-      }
+            console.error('Error during service worker registration:', error)
-    })
+        })
-    .catch(error => {
-      console.error('Error during service worker registration:', error)
-    })
 }
 
 function checkValidServiceWorker(swUrl: string, config?: Config) {
-  // Check if the service worker can be found. If it can't reload the page.
+    // Check if the service worker can be found. If it can't reload the page.
-  fetch(swUrl)
+    fetch(swUrl)
-    .then(response => {
+        .then(response => {
-      // Ensure service worker exists, and that we really are getting a JS file.
+            // Ensure service worker exists, and that we really are getting a JS file.
-      const contentType = response.headers.get('content-type')
+            const contentType = response.headers.get('content-type')
-      if (
+            if (
-        response.status === 404 ||
+                response.status === 404 ||
-        (contentType != null && contentType.indexOf('javascript') === -1)
+                (contentType != null &&
-      ) {
+                    contentType.indexOf('javascript') === -1)
-        // No service worker found. Probably a different app. Reload the page.
+            ) {
-        navigator.serviceWorker.ready.then(registration => {
+                // No service worker found. Probably a different app. Reload the page.
-          registration.unregister().then(() => {
+                navigator.serviceWorker.ready.then(registration => {
-            window.location.reload()
+                    registration.unregister().then(() => {
-          })
+                        window.location.reload()
+                    })
+                })
+            } else {
+                // Service worker found. Proceed as normal.
+                registerValidSW(swUrl, config)
+            }
+        })
+        .catch(() => {
+            console.log(
+                'No internet connection found. App is running in offline mode.'
+            )
         })
-      } else {
-        // Service worker found. Proceed as normal.
-        registerValidSW(swUrl, config)
-      }
-    })
-    .catch(() => {
-      console.log(
-        'No internet connection found. App is running in offline mode.'
-      )
-    })
 }
 
 export function unregister() {
-  if ('serviceWorker' in navigator) {
+    if ('serviceWorker' in navigator) {
-    navigator.serviceWorker.ready.then(registration => {
+        navigator.serviceWorker.ready.then(registration => {
-      registration.unregister()
+            registration.unregister()
-    })
+        })
-  }
+    }
 }