refactor: move tests to specifically related folders

* refactor: move tests to specifically related folders

* chore(config): update outdated configs after test changes

* refactor: remove index from import paths in tests

Co-authored-by: Michael Peyper <[email protected]>

Author: joshuaellis

Diff for: .eslintrc

@@ -11,6 +11,6 @@
     "@typescript-eslint/no-invalid-void-type": "off"
   },
   "parserOptions": {
-    "project": ["./tsconfig.json", "./test/tsconfig.json", "./scripts/tsconfig.json"]
+    "project": ["./tsconfig.json", "./scripts/tsconfig.json"]
   }
 }

Diff for: .gitignore

@@ -1,10 +1,20 @@
+# node
 node_modules
-coverage
+
+# build generated
 lib
 dom
 native
 server
 pure
 .docz
 site
+
+# tests (need the !files because of the build generated above)
+coverage
+!src/dom
+!src/native
+!src/server
+
+# settings from IDE
 .vscode

Diff for: jest.config.js

@@ -1,5 +1,5 @@
 import { useState, useRef, useEffect } from 'react'
-import { renderHook } from '../../src/dom'
+import { renderHook } from '..'
 
 describe('async hook tests', () => {
   const useSequence = (...values: string[]) => {

Diff for: test/dom/asyncHook.ts renamed to src/dom/__tests__/asyncHook.test.ts

@@ -1,6 +1,6 @@
 import { useEffect } from 'react'
 
-import { ReactHooksRenderer } from 'types'
+import { ReactHooksRenderer } from '../../types/react'
 
 // This verifies that if RHTL_SKIP_AUTO_CLEANUP is set
 // then we DON'T auto-wire up the afterEach for folks
@@ -11,7 +11,7 @@ describe('skip auto cleanup (disabled) tests', () => {
   beforeAll(() => {
     process.env.RHTL_SKIP_AUTO_CLEANUP = 'true'
     // eslint-disable-next-line @typescript-eslint/no-var-requires
-    renderHook = (require('../../src/native') as ReactHooksRenderer).renderHook
+    renderHook = (require('..') as ReactHooksRenderer).renderHook
   })
 
   test('first', () => {

Diff for: test/native/autoCleanup.disabled.ts renamed to src/dom/__tests__/autoCleanup.disabled.test.ts

@@ -1,6 +1,6 @@
 import { useEffect } from 'react'
 
-import { ReactHooksRenderer } from 'types'
+import { ReactHooksRenderer } from '../../types/react'
 
 // This verifies that if RHTL_SKIP_AUTO_CLEANUP is set
 // then we DON'T auto-wire up the afterEach for folks
@@ -13,7 +13,7 @@ describe('skip auto cleanup (no afterEach) tests', () => {
     // eslint-disable-next-line no-global-assign
     afterEach = false
     // eslint-disable-next-line @typescript-eslint/no-var-requires
-    renderHook = (require('../../src/native') as ReactHooksRenderer).renderHook
+    renderHook = (require('..') as ReactHooksRenderer).renderHook
   })
 
   test('first', () => {

Diff for: test/native/autoCleanup.noAfterEach.ts renamed to src/dom/__tests__/autoCleanup.noAfterEach.test.ts

@@ -1,5 +1,5 @@
 import { useEffect } from 'react'
-import { renderHook } from '../../src/dom'
+import { renderHook } from '..'
 
 // This verifies that by importing RHTL in an
 // environment which supports afterEach (like Jest)

Diff for: test/dom/autoCleanup.ts renamed to src/dom/__tests__/autoCleanup.test.ts

@@ -1,5 +1,5 @@
 import { useEffect } from 'react'
-import { renderHook, cleanup, addCleanup, removeCleanup } from '../../src/dom/pure'
+import { renderHook, cleanup, addCleanup, removeCleanup } from '../pure'
 
 describe('cleanup tests', () => {
   test('should flush effects on cleanup', async () => {

Diff for: test/dom/cleanup.ts renamed to src/dom/__tests__/cleanup.test.ts

@@ -1,5 +1,5 @@
 import { useState, useCallback } from 'react'
-import { renderHook, act } from '../../src/dom'
+import { renderHook, act } from '..'
 
 describe('custom hook tests', () => {
   function useCounter() {

Diff for: test/dom/customHook.ts renamed to src/dom/__tests__/customHook.test.ts

@@ -1,5 +1,5 @@
 import { useState, useEffect } from 'react'
-import { renderHook } from '../../src/dom'
+import { renderHook } from '..'
 
 describe('error hook tests', () => {
   function useError(throwError?: boolean) {

Diff for: test/dom/errorHook.ts renamed to src/dom/__tests__/errorHook.test.ts

@@ -1,4 +1,4 @@
-import { renderHook } from '../../src/dom'
+import { renderHook } from '..'
 
 describe('result history tests', () => {
   let count = 0

Diff for: test/dom/resultHistory.ts renamed to src/dom/__tests__/resultHistory.test.ts

@@ -1,4 +1,4 @@
-import { renderHook } from '../../src/dom'
+import { renderHook } from '..'
 
 describe('suspense hook tests', () => {
   const cache: { value?: Promise<string | Error> | string | Error } = {}

Diff for: test/dom/suspenseHook.ts renamed to src/dom/__tests__/suspenseHook.test.ts

@@ -1,5 +1,5 @@
 import React, { createContext, useContext } from 'react'
-import { renderHook } from '../../src/dom'
+import { renderHook } from '..'
 
 describe('useContext tests', () => {
   test('should get default value from context', () => {

Diff for: test/dom/useContext.tsx renamed to src/dom/__tests__/useContext.test.tsx

@@ -1,5 +1,5 @@
 import { useEffect, useLayoutEffect } from 'react'
-import { renderHook } from '../../src/dom'
+import { renderHook } from '..'
 
 describe('useEffect tests', () => {
   test('should handle useEffect hook', () => {

Diff for: test/dom/useEffect.ts renamed to src/dom/__tests__/useEffect.test.ts

@@ -1,5 +1,5 @@
 import { useMemo, useCallback } from 'react'
-import { renderHook } from '../../src/dom'
+import { renderHook } from '..'
 
 describe('useCallback tests', () => {
   test('should handle useMemo hook', () => {

Diff for: test/dom/useMemo.ts renamed to src/dom/__tests__/useMemo.test.ts

@@ -1,5 +1,5 @@
 import { useReducer } from 'react'
-import { renderHook, act } from '../../src/dom'
+import { renderHook, act } from '..'
 
 describe('useReducer tests', () => {
   test('should handle useReducer hook', () => {

Diff for: test/dom/useReducer.ts renamed to src/dom/__tests__/useReducer.test.ts

@@ -1,5 +1,5 @@
 import { useRef, useImperativeHandle } from 'react'
-import { renderHook } from '../../src/dom'
+import { renderHook } from '..'
 
 describe('useHook tests', () => {
   test('should handle useRef hook', () => {

Diff for: test/dom/useRef.ts renamed to src/dom/__tests__/useRef.test.ts

@@ -1,5 +1,5 @@
 import { useState } from 'react'
-import { renderHook, act } from '../../src/dom'
+import { renderHook, act } from '..'
 
 describe('useState tests', () => {
   test('should use setState value', () => {

Diff for: test/dom/useState.ts renamed to src/dom/__tests__/useState.test.ts

@@ -1,5 +1,5 @@
 import { useState, useRef, useEffect } from 'react'
-import { renderHook } from '../../src/native'
+import { renderHook } from '..'
 
 describe('async hook tests', () => {
   const useSequence = (...values: string[]) => {

Diff for: test/native/asyncHook.ts renamed to src/native/__tests__/asyncHook.test.ts

@@ -1,6 +1,6 @@
 import { useEffect } from 'react'
 
-import { ReactHooksRenderer } from 'types'
+import { ReactHooksRenderer } from '../../types/react'
 
 // This verifies that if RHTL_SKIP_AUTO_CLEANUP is set
 // then we DON'T auto-wire up the afterEach for folks
@@ -11,7 +11,7 @@ describe('skip auto cleanup (disabled) tests', () => {
   beforeAll(() => {
     process.env.RHTL_SKIP_AUTO_CLEANUP = 'true'
     // eslint-disable-next-line @typescript-eslint/no-var-requires
-    renderHook = (require('../../src/server') as ReactHooksRenderer).renderHook
+    renderHook = (require('..') as ReactHooksRenderer).renderHook
   })
 
   test('first', () => {

Diff for: test/server/autoCleanup.disabled.ts renamed to src/native/__tests__/autoCleanup.disabled.test.ts

@@ -1,6 +1,6 @@
 import { useEffect } from 'react'
 
-import { ReactHooksRenderer } from 'types'
+import { ReactHooksRenderer } from '../../types/react'
 
 // This verifies that if RHTL_SKIP_AUTO_CLEANUP is set
 // then we DON'T auto-wire up the afterEach for folks
@@ -13,7 +13,7 @@ describe('skip auto cleanup (no afterEach) tests', () => {
     // eslint-disable-next-line no-global-assign
     afterEach = false
     // eslint-disable-next-line @typescript-eslint/no-var-requires
-    renderHook = (require('../../src/server') as ReactHooksRenderer).renderHook
+    renderHook = (require('..') as ReactHooksRenderer).renderHook
   })
 
   test('first', () => {

Diff for: test/server/autoCleanup.noAfterEach.ts renamed to src/native/__tests__/autoCleanup.noAfterEach.test.ts

@@ -1,5 +1,5 @@
 import { useEffect } from 'react'
-import { renderHook } from '../../src/native'
+import { renderHook } from '..'
 
 // This verifies that by importing RHTL in an
 // environment which supports afterEach (like Jest)

Diff for: test/native/autoCleanup.ts renamed to src/native/__tests__/autoCleanup.test.ts

@@ -1,5 +1,5 @@
 import { useEffect } from 'react'
-import { renderHook, cleanup, addCleanup, removeCleanup } from '../../src/native/pure'
+import { renderHook, cleanup, addCleanup, removeCleanup } from '../pure'
 
 describe('cleanup tests', () => {
   test('should flush effects on cleanup', async () => {

Diff for: test/native/cleanup.ts renamed to src/native/__tests__/cleanup.test.ts

@@ -1,5 +1,5 @@
 import { useState, useCallback } from 'react'
-import { renderHook, act } from '../../src/native'
+import { renderHook, act } from '..'
 
 describe('custom hook tests', () => {
   function useCounter() {

Diff for: test/native/customHook.ts renamed to src/native/__tests__/customHook.test.ts

@@ -1,5 +1,5 @@
 import { useState, useEffect } from 'react'
-import { renderHook } from '../../src/native'
+import { renderHook } from '..'
 
 describe('error hook tests', () => {
   function useError(throwError?: boolean) {

Diff for: test/native/errorHook.ts renamed to src/native/__tests__/errorHook.test.ts

@@ -1,4 +1,4 @@
-import { renderHook } from '../../src/native'
+import { renderHook } from '..'
 
 describe('result history tests', () => {
   let count = 0

Diff for: test/native/resultHistory.ts renamed to src/native/__tests__/resultHistory.test.ts

@@ -1,4 +1,4 @@
-import { renderHook } from '../../src/native'
+import { renderHook } from '..'
 
 describe('suspense hook tests', () => {
   const cache: { value?: Promise<string | Error> | string | Error } = {}

Diff for: test/native/suspenseHook.ts renamed to src/native/__tests__/suspenseHook.test.ts

@@ -1,5 +1,5 @@
 import React, { createContext, useContext } from 'react'
-import { renderHook } from '../../src/native'
+import { renderHook } from '..'
 
 describe('useContext tests', () => {
   test('should get default value from context', () => {

Diff for: test/native/useContext.tsx renamed to src/native/__tests__/useContext.test.tsx

@@ -1,5 +1,5 @@
 import { useEffect, useLayoutEffect } from 'react'
-import { renderHook } from '../../src/native'
+import { renderHook } from '..'
 
 describe('useEffect tests', () => {
   test('should handle useEffect hook', () => {

Diff for: test/native/useEffect.ts renamed to src/native/__tests__/useEffect.test.ts

@@ -1,5 +1,5 @@
 import { useMemo, useCallback } from 'react'
-import { renderHook } from '../../src/native'
+import { renderHook } from '..'
 
 describe('useCallback tests', () => {
   test('should handle useMemo hook', () => {

Diff for: test/native/useMemo.ts renamed to src/native/__tests__/useMemo.test.ts

@@ -1,5 +1,5 @@
 import { useReducer } from 'react'
-import { renderHook, act } from '../../src/native'
+import { renderHook, act } from '..'
 
 describe('useReducer tests', () => {
   test('should handle useReducer hook', () => {

Diff for: test/native/useReducer.ts renamed to src/native/__tests__/useReducer.test.ts

@@ -1,5 +1,5 @@
 import { useRef, useImperativeHandle } from 'react'
-import { renderHook } from '../../src/native'
+import { renderHook } from '..'
 
 describe('useHook tests', () => {
   test('should handle useRef hook', () => {

Diff for: test/native/useRef.ts renamed to src/native/__tests__/useRef.test.ts

@@ -1,5 +1,5 @@
 import { useState } from 'react'
-import { renderHook, act } from '../../src/native'
+import { renderHook, act } from '..'
 
 describe('useState tests', () => {
   test('should use setState value', () => {

Diff for: test/native/useState.ts renamed to src/native/__tests__/useState.test.ts

@@ -1,6 +1,6 @@
 import { useState, useRef, useEffect } from 'react'
 
-import { renderHook } from '../../src/server'
+import { renderHook } from '..'
 
 describe('async hook tests', () => {
   const useSequence = (...values: string[]) => {

Diff for: test/server/asyncHook.ts renamed to src/server/__tests__/asyncHook.test.ts

@@ -1,6 +1,6 @@
 import { useEffect } from 'react'
 
-import { ReactHooksRenderer } from 'types/react'
+import { ReactHooksRenderer } from '../../types/react'
 
 // This verifies that if RHTL_SKIP_AUTO_CLEANUP is set
 // then we DON'T auto-wire up the afterEach for folks
@@ -11,7 +11,7 @@ describe('skip auto cleanup (disabled) tests', () => {
   beforeAll(() => {
     process.env.RHTL_SKIP_AUTO_CLEANUP = 'true'
     // eslint-disable-next-line @typescript-eslint/no-var-requires
-    renderHook = (require('../../src/dom') as ReactHooksRenderer).renderHook
+    renderHook = (require('..') as ReactHooksRenderer).renderHook
   })
 
   test('first', () => {

Diff for: test/dom/autoCleanup.disabled.ts renamed to src/server/__tests__/autoCleanup.disabled.test.ts

@@ -1,6 +1,6 @@
 import { useEffect } from 'react'
 
-import { ReactHooksRenderer } from 'types/react'
+import { ReactHooksRenderer } from '../../types/react'
 
 // This verifies that if RHTL_SKIP_AUTO_CLEANUP is set
 // then we DON'T auto-wire up the afterEach for folks
@@ -13,7 +13,7 @@ describe('skip auto cleanup (no afterEach) tests', () => {
     // eslint-disable-next-line no-global-assign
     afterEach = false
     // eslint-disable-next-line @typescript-eslint/no-var-requires
-    renderHook = (require('../../src/dom') as ReactHooksRenderer).renderHook
+    renderHook = (require('..') as ReactHooksRenderer).renderHook
   })
 
   test('first', () => {

Diff for: test/dom/autoCleanup.noAfterEach.ts renamed to src/server/__tests__/autoCleanup.noAfterEach.test.ts

@@ -1,5 +1,5 @@
 import { useEffect } from 'react'
-import { renderHook } from '../../src/server'
+import { renderHook } from '..'
 
 // This verifies that by importing RHTL in an
 // environment which supports afterEach (like Jest)

Diff for: test/server/autoCleanup.ts renamed to src/server/__tests__/autoCleanup.test.ts

@@ -1,5 +1,5 @@
 import { useEffect } from 'react'
-import { renderHook, cleanup } from '../../src/server'
+import { renderHook, cleanup } from '..'
 
 describe('cleanup tests', () => {
   test('should flush effects on cleanup', async () => {

Diff for: test/server/cleanup.ts renamed to src/server/__tests__/cleanup.test.ts

@@ -1,5 +1,5 @@
 import { useState, useCallback } from 'react'
-import { renderHook, act } from '../../src/server'
+import { renderHook, act } from '..'
 
 describe('custom hook tests', () => {
   function useCounter() {

Diff for: test/server/customHook.ts renamed to src/server/__tests__/customHook.test.ts

@@ -1,6 +1,6 @@
 import { useState, useEffect } from 'react'
 
-import { renderHook } from '../../src/server'
+import { renderHook } from '..'
 
 describe('error hook tests', () => {
   function useError(throwError?: boolean) {

Diff for: test/server/errorHook.ts renamed to src/server/__tests__/errorHook.test.ts

@@ -1,5 +1,5 @@
 import { useState, useCallback } from 'react'
-import { renderHook } from '../../src/server'
+import { renderHook } from '..'
 
 describe('hydration errors tests', () => {
   function useCounter() {

Diff for: test/server/hydrationErrors.ts renamed to src/server/__tests__/hydrationErrors.test.ts

@@ -1,5 +1,5 @@
 import React, { createContext, useContext } from 'react'
-import { renderHook } from '../../src/server'
+import { renderHook } from '..'
 
 describe('useContext tests', () => {
   test('should get default value from context', () => {

Diff for: test/server/useContext.tsx renamed to src/server/__tests__/useContext.test.tsx

@@ -1,5 +1,5 @@
 import { useEffect } from 'react'
-import { renderHook } from '../../src/server'
+import { renderHook } from '..'
 
 describe('useEffect tests', () => {
   test('should handle useEffect hook', () => {

Diff for: test/server/useEffect.ts renamed to src/server/__tests__/useEffect.test.ts

@@ -1,5 +1,5 @@
 import { useMemo, useCallback } from 'react'
-import { renderHook } from '../../src/server'
+import { renderHook } from '..'
 
 describe('useCallback tests', () => {
   test('should handle useMemo hook', () => {

Diff for: test/server/useMemo.ts renamed to src/server/__tests__/useMemo.test.ts

@@ -1,5 +1,5 @@
 import { useReducer } from 'react'
-import { renderHook, act } from '../../src/server'
+import { renderHook, act } from '..'
 
 describe('useReducer tests', () => {
   test('should handle useReducer hook', () => {

Diff for: test/server/useReducer.ts renamed to src/server/__tests__/useReducer.test.ts

@@ -1,5 +1,5 @@
 import { useRef, useImperativeHandle } from 'react'
-import { renderHook } from '../../src/server'
+import { renderHook } from '..'
 
 describe('useHook tests', () => {
   test('should handle useRef hook', () => {