chore(devdeps): update dependency @testing-library/jest-dom to v6 (#1483)

renovate[bot] · QuiiBz · web-flow · commit 0fa96c187e49 · 2023-08-29T15:04:56.000+02:00
Co-authored-by: renovate[bot] &lt;29139614+renovate[bot]@users.noreply.github.com&gt;
Co-authored-by: QuiiBz &lt;tlienard@scaleway.com&gt;
diff --git a/.eslintrc b/.eslintrc
@@ -1,8 +1,7 @@
 {
   "root": true,
   "env": {
-    "browser": true,
-    "jest": true
+    "browser": true
   },
   "overrides": [
     {
@@ -30,6 +29,15 @@
       "rules": {
         "no-console": "off"
       }
+    },
+    {
+      "files": [
+        "packages/jest-helpers/**/*.ts{x,}",
+        "**/__tests__/**/*.ts{x,}"
+      ],
+      "rules": {
+        "import/no-extraneous-dependencies": "off"
+      }
     }
   ]
 }
diff --git a/package.json b/package.json
@@ -21,14 +21,14 @@
     "@changesets/cli": "2.26.2",
     "@commitlint/cli": "17.7.1",
     "@commitlint/config-conventional": "17.7.0",
+    "@jest/globals": "^29.6.4",
     "@rollup/plugin-babel": "6.0.3",
     "@rollup/plugin-node-resolve": "15.2.1",
     "@scaleway/eslint-config-react": "workspace:*",
     "@scaleway/tsconfig": "workspace:*",
-    "@testing-library/jest-dom": "5.17.0",
+    "@testing-library/jest-dom": "6.0.1",
     "@testing-library/react": "13.4.0",
     "@tsd/typescript": "5.1.6",
-    "@types/jest": "29.5.4",
     "@types/node": "18.17.9",
     "@types/prop-types": "15.7.5",
     "@types/react": "18.2.21",
diff --git a/packages/changesets-renovate/src/__tests__/generate-changeset.ts b/packages/changesets-renovate/src/__tests__/generate-changeset.ts
@@ -1,3 +1,4 @@
+import { afterEach, beforeEach, describe, expect, it, jest } from '@jest/globals'
 import fs from 'node:fs/promises'
 import { simpleGit } from 'simple-git'
 import { run } from '..'
@@ -95,9 +96,9 @@ describe('generate changeset file', () => {
     })
 
     fs.readFile = jest
-      .fn()
+      .fn<any>()
       .mockResolvedValue(`{"name":"packageName","version":"1.0.0"}`)
-    fs.writeFile = jest.fn()
+    fs.writeFile = jest.fn<any>()
 
     await run()
 
@@ -129,9 +130,9 @@ describe('generate changeset file', () => {
     })
 
     fs.readFile = jest
-      .fn()
+      .fn<any>()
       .mockResolvedValue(`{"name":"packageName","workspaces":[]}`)
-    fs.writeFile = jest.fn()
+    fs.writeFile = jest.fn<any>()
 
     await run()
 
@@ -159,8 +160,8 @@ describe('generate changeset file', () => {
 `,
     })
 
-    fs.readFile = jest.fn().mockResolvedValue(`{"name":"packageName"}`)
-    fs.writeFile = jest.fn()
+    fs.readFile = jest.fn<any>().mockResolvedValue(`{"name":"packageName"}`)
+    fs.writeFile = jest.fn<any>()
 
     await run()
 
diff --git a/packages/jest-helpers/src/__tests__/index.test.tsx b/packages/jest-helpers/src/__tests__/index.test.tsx
@@ -1,3 +1,4 @@
+import { beforeAll, describe, expect, it, jest, test } from '@jest/globals'
 import makeHelpers from '..'
 import type { RenderWithThemeFn } from '../helpers/renderWithTheme'
 import type { ShouldMatchEmotionSnapshotFn } from '../helpers/shouldMatchEmotionSnapshot'
@@ -31,7 +32,7 @@ describe('@jest-helpers', () => {
   })
 
   test('should call tranform with shouldMatchEmotionSnapshot', async () => {
-    const transform = jest.fn()
+    const transform = jest.fn<any>()
     await shouldMatchEmotionSnapshot(<div id="test" />, { transform })
 
     expect(transform).toHaveBeenCalledTimes(1)
@@ -42,7 +43,7 @@ describe('@jest-helpers', () => {
   })
 
   it('should call transform with shouldMatchEmotionSnapshot', async () => {
-    const transform = jest.fn()
+    const transform = jest.fn<any>()
     await shouldMatchEmotionSnapshotWithPortal(<div id="test" />, { transform })
 
     expect(transform).toHaveBeenCalledTimes(1)
diff --git a/packages/jest-helpers/src/helpers/shouldMatchEmotionSnapshot.ts b/packages/jest-helpers/src/helpers/shouldMatchEmotionSnapshot.ts
@@ -1,3 +1,4 @@
+import { expect } from '@jest/globals'
 import type { RenderOptions, render } from '@testing-library/react'
 import type { ReactNode } from 'react'
 import type { RenderWithThemeFn } from './renderWithTheme'
diff --git a/packages/jest-helpers/src/helpers/shouldMatchEmotionSnapshotWithPortal.ts b/packages/jest-helpers/src/helpers/shouldMatchEmotionSnapshotWithPortal.ts
@@ -1,3 +1,4 @@
+import { expect, jest } from '@jest/globals'
 import type { RenderOptions, render } from '@testing-library/react'
 import type { ReactNode } from 'react'
 import type { RenderWithThemeFn } from './renderWithTheme'
diff --git a/packages/jest-helpers/src/index.ts b/packages/jest-helpers/src/index.ts
@@ -1,5 +1,6 @@
 import type { CreateSerializerOptions } from '@emotion/jest'
 import { createSerializer } from '@emotion/jest'
+import { expect } from '@jest/globals'
 import type { FC, ReactNode } from 'react'
 import type { RenderWithThemeFn } from './helpers/renderWithTheme'
 import makeRenderWithTheme from './helpers/renderWithTheme'
diff --git a/packages/outdated-browser/src/__tests__/index.ts b/packages/outdated-browser/src/__tests__/index.ts
@@ -1,3 +1,5 @@
+import { afterEach, beforeAll, describe, expect, jest, test } from '@jest/globals'
+
 /**
  * @jest-environment jsdom
  */
diff --git a/packages/random-name/src/__tests__/index.ts b/packages/random-name/src/__tests__/index.ts
@@ -1,3 +1,4 @@
+import { describe, expect, it } from '@jest/globals'
 import randomName from '..'
 
 describe('randomName', () => {
diff --git a/packages/regex/src/__tests__/index.ts b/packages/regex/src/__tests__/index.ts
@@ -1,3 +1,4 @@
+import { describe, expect, test } from '@jest/globals'
 import {
   absoluteLinuxPath,
   accessKeyRegex,
diff --git a/packages/use-dataloader/src/__tests__/DataLoaderProvider.test.tsx b/packages/use-dataloader/src/__tests__/DataLoaderProvider.test.tsx
@@ -1,3 +1,4 @@
+import { describe, expect, jest, test } from '@jest/globals'
 import { render, renderHook, screen, waitFor } from '@testing-library/react'
 import type { ReactNode } from 'react'
 import DataLoaderProvider, { useDataLoaderContext } from '../DataLoaderProvider'
@@ -62,7 +63,7 @@ describe('DataLoaderProvider', () => {
     try {
       // @ts-expect-error Should throw an error
       await result.current.reload(3).catch(undefined)
-      fail('It should throw an error')
+      throw new Error('It should throw an error')
     } catch (error) {
       expect((error as Error).message).toBe(KEY_IS_NOT_STRING_ERROR)
     }
@@ -72,7 +73,7 @@ describe('DataLoaderProvider', () => {
     try {
       // @ts-expect-error Should throw an error
       result.current.clearCachedData(3)
-      fail('It should throw an error')
+      throw new Error('It should throw an error')
     } catch (error) {
       expect((error as Error).message).toBe(KEY_IS_NOT_STRING_ERROR)
       expect(result.current.getCachedData(TEST_KEY)).toBe(true)
@@ -122,7 +123,7 @@ describe('DataLoaderProvider', () => {
     if (testReload) {
       expect(await testReload()).toBeNull()
     } else {
-      fail('It shoulded be defined')
+      throw new Error('It shoulded be defined')
     }
     expect(result.current.getCachedData(TEST_KEY)).toBe(null)
     expect(result.current.getCachedData()).toStrictEqual({ test: null })
diff --git a/packages/use-dataloader/src/__tests__/dataloader.test.ts b/packages/use-dataloader/src/__tests__/dataloader.test.ts
@@ -1,3 +1,4 @@
+import { describe, expect, jest, test } from '@jest/globals'
 import waitForExpect from 'wait-for-expect'
 import { StatusEnum } from '../constants'
 import DataLoader from '../dataloader'
diff --git a/packages/use-dataloader/src/__tests__/helpers.test.ts b/packages/use-dataloader/src/__tests__/helpers.test.ts
@@ -1,3 +1,4 @@
+import { describe, expect, test } from '@jest/globals'
 import { marshalQueryKey } from '../helpers'
 
 describe('marshalQueryKey', () => {
diff --git a/packages/use-dataloader/src/__tests__/useDataLoader.test.tsx b/packages/use-dataloader/src/__tests__/useDataLoader.test.tsx
@@ -1,4 +1,5 @@
 /* eslint-disable no-console */
+import { describe, expect, jest, test } from '@jest/globals'
 import { renderHook, waitFor } from '@testing-library/react'
 import type { ReactNode } from 'react'
 import DataLoaderProvider, { useDataLoaderContext } from '../DataLoaderProvider'
@@ -501,7 +502,7 @@ describe('useDataLoader', () => {
   })
 
   test('should call onSuccess', async () => {
-    const onSuccess = jest.fn()
+    const onSuccess = jest.fn<any>()
     const { result } = renderHook(
       props => useDataLoader(props.key, props.method, props.config),
       {
@@ -523,8 +524,8 @@ describe('useDataLoader', () => {
   })
 
   test('should call onError', async () => {
-    const onSuccess = jest.fn()
-    const onError = jest.fn()
+    const onSuccess = jest.fn<any>()
+    const onError = jest.fn<any>()
     const error = new Error('Test error')
     const { result } = renderHook(
       props => useDataLoader(props.key, props.method, props.config),
@@ -557,8 +558,8 @@ describe('useDataLoader', () => {
   })
 
   test('should override onError from Provider', async () => {
-    const onSuccess = jest.fn()
-    const onError = jest.fn()
+    const onSuccess = jest.fn<any>()
+    const onError = jest.fn<any>()
     const error = new Error('Test error')
     const onErrorProvider = jest.fn()
     const { result } = renderHook(
@@ -593,7 +594,7 @@ describe('useDataLoader', () => {
   })
 
   test('should call onError from Provider', async () => {
-    const onSuccess = jest.fn()
+    const onSuccess = jest.fn<any>()
     const error = new Error('Test error')
     const onErrorProvider = jest.fn()
     const { result } = renderHook(
@@ -629,7 +630,7 @@ describe('useDataLoader', () => {
 
   test('should clear error on new response', async () => {
     let success = false
-    const onSuccess = jest.fn()
+    const onSuccess = jest.fn<any>()
     const onError = jest.fn(() => {
       success = true
     })
diff --git a/packages/use-growthbook/src/__tests__/AbTestProvider.tsx b/packages/use-growthbook/src/__tests__/AbTestProvider.tsx
@@ -1,5 +1,6 @@
 // @ts-expect-error TODO: remove once Growthbook is correctly typed and export
 import { GrowthBook } from '@growthbook/growthbook-react'
+import { beforeEach, describe, expect, it, jest } from '@jest/globals'
 import { render } from '@testing-library/react'
 import type { TrackingCallback } from '../AbTestProvider'
 import { AbTestProvider } from '../AbTestProvider'
diff --git a/packages/use-growthbook/src/__tests__/useAbTestAttributes.ts b/packages/use-growthbook/src/__tests__/useAbTestAttributes.ts
@@ -1,12 +1,11 @@
 // @ts-expect-error TODO: remove once Growthbook is correctly typed and export
 import { useGrowthBook } from '@growthbook/growthbook-react'
+import { beforeEach, describe, expect, it, jest } from '@jest/globals'
 import { renderHook } from '@testing-library/react'
 import type { Attributes } from '../types'
 import { useAbTestAttributes } from '../useAbTestAttributes'
 
-jest.mock('@growthbook/growthbook-react', () => ({
-  useGrowthBook: jest.fn(),
-}))
+jest.mock('@growthbook/growthbook-react')
 
 describe('useAbTestAttributes', () => {
   let getAttributes: () => Attributes
diff --git a/packages/use-gtm/src/__tests__/index.tsx b/packages/use-gtm/src/__tests__/index.tsx
@@ -1,3 +1,4 @@
+import { afterEach, beforeEach, describe, expect, it, jest } from '@jest/globals'
 import { fireEvent, renderHook } from '@testing-library/react'
 import mockdate from 'mockdate'
 import type { ReactNode } from 'react'
diff --git a/packages/use-i18n/src/__tests__/formatDate.ts b/packages/use-i18n/src/__tests__/formatDate.ts
@@ -1,3 +1,4 @@
+import { describe, expect, test } from '@jest/globals'
 import type { FormatDateOptions } from '../formatDate'
 import formatDate, { supportedFormats } from '../formatDate'
 
diff --git a/packages/use-i18n/src/__tests__/formatUnit.ts b/packages/use-i18n/src/__tests__/formatUnit.ts
@@ -1,3 +1,4 @@
+import { describe, expect, test } from '@jest/globals'
 import type { FormatUnitOptions } from '../formatUnit'
 import formatUnit, { supportedUnits } from '../formatUnit'
 
diff --git a/packages/use-i18n/src/__tests__/usei18n.tsx b/packages/use-i18n/src/__tests__/usei18n.tsx
@@ -1,3 +1,4 @@
+import { afterEach, beforeEach, describe, expect, it, jest } from '@jest/globals'
 import { act, renderHook, waitFor } from '@testing-library/react'
 import mockdate from 'mockdate'
 import type { ReactNode } from 'react'
diff --git a/packages/use-media/src/__tests__/useMedia.tsx b/packages/use-media/src/__tests__/useMedia.tsx
@@ -1,3 +1,4 @@
+import { describe, expect, it, jest } from '@jest/globals'
 import { renderHook } from '@testing-library/react'
 import { useMedia } from '..'
 
@@ -15,7 +16,7 @@ describe('useMedia hook', () => {
       callback()
 
     Object.defineProperty(window, 'matchMedia', {
-      value: jest.fn().mockImplementation((query: string) => ({
+      value: jest.fn<any>().mockImplementation((query: string) => ({
         addEventListener: mockAddEventListener,
         addListener: jest.fn(),
         dispatchEvent: jest.fn(),
@@ -43,7 +44,7 @@ describe('useMedia hook', () => {
     }
 
     Object.defineProperty(window, 'matchMedia', {
-      value: jest.fn().mockImplementation((query: string) => ({
+      value: jest.fn<any>().mockImplementation((query: string) => ({
         addEventListener: mockAddEventListener,
         addListener: jest.fn(),
         dispatchEvent: jest.fn(),
diff --git a/packages/use-query-params/src/__tests__/index.tsx b/packages/use-query-params/src/__tests__/index.tsx
@@ -1,3 +1,4 @@
+import { describe, expect, it, test } from '@jest/globals'
 import { act, renderHook } from '@testing-library/react'
 import type { History } from 'history'
 import { createMemoryHistory } from 'history'
diff --git a/packages/use-random-name/src/__tests__/index.ts b/packages/use-random-name/src/__tests__/index.ts
@@ -1,3 +1,4 @@
+import { afterAll, beforeAll, describe, expect, it, jest } from '@jest/globals'
 import { renderHook } from '@testing-library/react'
 import useRandomName from '..'
 
diff --git a/packages/use-segment/src/__tests__/index.tsx b/packages/use-segment/src/__tests__/index.tsx
@@ -1,3 +1,4 @@
+import { beforeEach, describe, expect, it, jest } from '@jest/globals'
 import type { Context } from '@segment/analytics-next'
 import { AnalyticsBrowser } from '@segment/analytics-next'
 import { renderHook, waitFor } from '@testing-library/react'
@@ -218,7 +219,7 @@ describe('segment hook', () => {
       .spyOn(AnalyticsBrowser, 'load')
       .mockResolvedValue([{} as Analytics, {} as Context])
 
-    const onEventError = jest.fn()
+    const onEventError = jest.fn<any>()
     const onError = jest.fn()
 
     const settings = { writeKey: 'pleasethrow' }
diff --git a/packages/use-storage/src/__tests__/index.client.ts b/packages/use-storage/src/__tests__/index.client.ts
@@ -2,6 +2,7 @@
  * @jest-environment jsdom
  */
 
+import { afterEach, describe, expect, it } from '@jest/globals'
 import { act, renderHook } from '@testing-library/react'
 import { useLocalStorage, useSessionStorage } from '..'
 
diff --git a/packages/use-storage/src/__tests__/index.server.ts b/packages/use-storage/src/__tests__/index.server.ts
@@ -2,6 +2,7 @@
  * @jest-environment node
  */
 
+import { describe, expect, it } from '@jest/globals'
 import { act, renderHook } from '@testing-library/react/pure'
 import { useLocalStorage, useSessionStorage } from '..'
 
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
diff --git a/rollup.config.mjs b/rollup.config.mjs

Original file line number	Diff line number	Diff line change
`@@ -1,8 +1,7 @@`
`1`	`1`	`{`
`2`	`2`	`"root": true,`
`3`	`3`	`"env": {`
`4`		`- "browser": true,`
`5`		`- "jest": true`
	`4`	`+ "browser": true`
`6`	`5`	`},`
`7`	`6`	`"overrides": [`
`8`	`7`	`{`
`@@ -30,6 +29,15 @@`
`30`	`29`	`"rules": {`
`31`	`30`	`"no-console": "off"`
`32`	`31`	`}`
	`32`	`+ },`
	`33`	`+ {`
	`34`	`+ "files": [`
	`35`	`+ "packages/jest-helpers/*/.ts{x,}",`
	`36`	`+ "/__tests__//*.ts{x,}"`
	`37`	`+ ],`
	`38`	`+ "rules": {`
	`39`	`+ "import/no-extraneous-dependencies": "off"`
	`40`	`+ }`
`33`	`41`	`}`
`34`	`42`	`]`
`35`	`43`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,4 @@`
	`1`	`+import { expect } from '@jest/globals'`
`1`	`2`	`import type { RenderOptions, render } from '@testing-library/react'`
`2`	`3`	`import type { ReactNode } from 'react'`
`3`	`4`	`import type { RenderWithThemeFn } from './renderWithTheme'`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,4 @@`
	`1`	`+import { expect, jest } from '@jest/globals'`
`1`	`2`	`import type { RenderOptions, render } from '@testing-library/react'`
`2`	`3`	`import type { ReactNode } from 'react'`
`3`	`4`	`import type { RenderWithThemeFn } from './renderWithTheme'`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+import { afterEach, beforeAll, describe, expect, jest, test } from '@jest/globals'`
	`2`	`+`
`1`	`3`	`/**`
`2`	`4`	`* @jest-environment jsdom`
`3`	`5`	`*/`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,4 @@`
	`1`	`+import { describe, expect, it } from '@jest/globals'`
`1`	`2`	`import randomName from '..'`
`2`	`3`
`3`	`4`	`describe('randomName', () => {`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,4 @@`
	`1`	`+import { describe, expect, test } from '@jest/globals'`
`1`	`2`	`import {`
`2`	`3`	`absoluteLinuxPath,`
`3`	`4`	`accessKeyRegex,`