format: fix linter issues

Author: marcosvega91

.eslintrc

@@ -3,6 +3,8 @@
   "rules": {
     "max-lines-per-function": "off",
     "no-constant-condition": "off",
-    "no-await-in-loop": "off"
+    "no-await-in-loop": "off",
+    "react-hooks/rules-of-hooks": "off",
+    "no-console": "off"
   }
 }

package.json

@@ -27,7 +27,7 @@
     "validate": "kcd-scripts validate",
     "prepare": "npm run build",
     "build": "kcd-scripts build",
-    "format": "prettier-eslint --write \"**/*.{js,ts,json,yml,md,mdx}\"",
+    "lint": "kcd-scripts lint",
     "coverage": "codecov",
     "test": "kcd-scripts test",
     "test:update": "npm test -- --updateSnapshot --coverage",

src/__tests__/asyncHook.js

@@ -17,7 +17,7 @@ describe('async hook tests', () => {
       return () => {
         clearInterval(interval)
       }
-    }, [...values])
+    }, [otherValues])
 
     return value
   }
@@ -83,7 +83,7 @@ describe('async hook tests', () => {
     const { waitFor } = renderHook(() => null)
 
     let actual = 0
-    let expected = 1
+    const expected = 1
 
     setTimeout(() => {
       actual = expected
@@ -165,7 +165,7 @@ describe('async hook tests', () => {
     const { waitFor } = renderHook(() => null)
 
     let actual = 0
-    let expected = 1
+    const expected = 1
 
     setTimeout(() => {
       actual = expected
@@ -207,7 +207,7 @@ describe('async hook tests', () => {
     const { waitForValueToChange } = renderHook(() => null)
 
     let actual = 0
-    let expected = 1
+    const expected = 1
 
     setTimeout(() => {
       actual = expected

src/__tests__/autoCleanup.noAfterEach.js

@@ -7,8 +7,9 @@ describe('skip auto cleanup (no afterEach) tests', () => {
   let renderHook
 
   beforeAll(() => {
+    // eslint-disable-next-line no-global-assign
     afterEach = false
-    renderHook = require('src').renderHook
+    renderHook = require('../').renderHook
   })
 
   test('first', () => {

src/__tests__/cleanup.js

@@ -21,7 +21,7 @@ describe('cleanup tests', () => {
   })
 
   test('should cleanup all rendered hooks', async () => {
-    let cleanupCalled = []
+    const cleanupCalled = []
     const hookWithCleanup = (id) => {
       useEffect(() => {
         return () => {
@@ -40,7 +40,7 @@ describe('cleanup tests', () => {
   })
 
   test('should call cleanups in reverse order', async () => {
-    let callSequence = []
+    const callSequence = []
     addCleanup(() => {
       callSequence.push('cleanup')
     })
@@ -62,7 +62,7 @@ describe('cleanup tests', () => {
   })
 
   test('should wait for async cleanup', async () => {
-    let callSequence = []
+    const callSequence = []
     addCleanup(() => {
       callSequence.push('cleanup')
     })
@@ -85,7 +85,7 @@ describe('cleanup tests', () => {
   })
 
   test('should remove cleanup using removeCleanup', async () => {
-    let callSequence = []
+    const callSequence = []
     addCleanup(() => {
       callSequence.push('cleanup')
     })
@@ -110,7 +110,7 @@ describe('cleanup tests', () => {
   })
 
   test('should remove cleanup using returned handler', async () => {
-    let callSequence = []
+    const callSequence = []
     addCleanup(() => {
       callSequence.push('cleanup')
     })

src/__tests__/errorHook.js

@@ -21,7 +21,7 @@ describe('error hook tests', () => {
   function useEffectError(throwError) {
     useEffect(() => {
       useError(throwError)
-    }, [])
+    }, [throwError])
     return true
   }
 
@@ -115,6 +115,7 @@ describe('error hook tests', () => {
     Refer to https://github.com/testing-library/react-hooks-testing-library/issues/308
     for more details.
   */
+  // eslint-disable-next-line jest/no-disabled-tests
   describe.skip('effect', () => {
     test('should raise effect error', () => {
       const { result } = renderHook(() => useEffectError(true))
@@ -137,12 +138,9 @@ describe('error hook tests', () => {
     })
 
     test('should reset effect error', () => {
-      const { result, waitForNextUpdate, rerender } = renderHook(
-        (throwError) => useEffectError(throwError),
-        {
-          initialProps: true
-        }
-      )
+      const { result, rerender } = renderHook((throwError) => useEffectError(throwError), {
+        initialProps: true
+      })
 
       expect(result.error).not.toBe(undefined)
 

src/__tests__/useEffect.js

@@ -3,7 +3,7 @@ import { renderHook } from '../'
 
 describe('useEffect tests', () => {
   test('should handle useEffect hook', () => {
-    const sideEffect = { [1]: false, [2]: false }
+    const sideEffect = { 1: false, 2: false }
 
     const { rerender, unmount } = renderHook(
       ({ id }) => {
@@ -32,7 +32,7 @@ describe('useEffect tests', () => {
   })
 
   test('should handle useLayoutEffect hook', () => {
-    const sideEffect = { [1]: false, [2]: false }
+    const sideEffect = { 1: false, 2: false }
 
     const { rerender, unmount } = renderHook(
       ({ id }) => {

src/__tests__/useState.js

@@ -13,7 +13,7 @@ describe('useState tests', () => {
   test('should update setState value using setter', () => {
     const { result } = renderHook(() => useState('foo'))
 
-    const [_, setValue] = result.current
+    const [ignoredValue, setValue] = result.current
 
     act(() => setValue('bar'))
 

src/asyncUtils.js

@@ -39,6 +39,7 @@ function asyncUtils(addResolver) {
   }
 
   const waitFor = async (callback, { interval, timeout, suppressErrors = true } = {}) => {
+    // eslint-disable-next-line consistent-return
     const checkResult = () => {
       try {
         const callbackResult = callback()