chore: Remove unused manual cleanups from tests

codecov.yml

@@ -23,7 +23,7 @@ comment:
  require_changes: false
  require_base: false
  require_head: true
- after_n_builds: 10 # ceil(number of test files / 80). See ci.yml for more details.
+ after_n_builds: 9 # ceil(number of test files / 80). See ci.yml for more details.
 
 ignore:
  - ./src/**/*.stories.js

src/layouts/BaseLayout/BaseLayout.test.tsx

@@ -1,5 +1,5 @@
 import { QueryClient, QueryClientProvider } from '@tanstack/react-query'
-import { cleanup, render, screen, waitFor } from '@testing-library/react'
+import { render, screen, waitFor } from '@testing-library/react'
 import { graphql, http, HttpResponse } from 'msw2'
 import { setupServer } from 'msw2/node'
 import { MemoryRouter, Route, useLocation } from 'react-router-dom'
@@ -193,7 +193,6 @@ afterEach(() => {
  queryClient.clear()
  server.resetHandlers()
  vi.clearAllMocks()
- cleanup()
 })
 
 afterAll(() => {

src/layouts/BaseLayout/InstallationHelpBanner/InstallationHelpBanner.test.jsx

@@ -1,5 +1,5 @@
 import { QueryClient, QueryClientProvider } from '@tanstack/react-query'
-import { cleanup, render, screen, waitFor } from '@testing-library/react'
+import { render, screen, waitFor } from '@testing-library/react'
 import userEvent from '@testing-library/user-event'
 import { graphql, HttpResponse } from 'msw2'
 import { setupServer } from 'msw2/node'
@@ -18,7 +18,6 @@ beforeAll(() => {
 })
 
 beforeEach(() => {
- cleanup()
  queryClient.clear()
  server.resetHandlers()
 })

src/layouts/BaseLayout/hooks/useUserAccessGate.test.tsx

@@ -1,5 +1,5 @@
 import { QueryClient, QueryClientProvider } from '@tanstack/react-query'
-import { cleanup, renderHook, waitFor } from '@testing-library/react'
+import { renderHook, waitFor } from '@testing-library/react'
 import { delay, graphql, http, HttpResponse } from 'msw2'
 import { setupServer } from 'msw2/node'
 import { MemoryRouter, Route } from 'react-router-dom'
@@ -221,7 +221,6 @@ beforeAll(() => {
 })
 
 beforeEach(() => {
- cleanup()
  queryClient.clear()
  server.resetHandlers()
 })

src/layouts/Header/Header.test.tsx

@@ -1,5 +1,5 @@
 import { QueryClient, QueryClientProvider } from '@tanstack/react-query'
-import { cleanup, render, screen } from '@testing-library/react'
+import { render, screen } from '@testing-library/react'
 import { graphql, HttpResponse } from 'msw2'
 import { setupServer } from 'msw2/node'
 import { MemoryRouter, Route } from 'react-router-dom'
@@ -93,7 +93,6 @@ beforeAll(() => {
 })
 
 afterEach(() => {
- cleanup()
  server.resetHandlers()
  queryClient.clear()
 })

src/layouts/Header/components/AdminLink/AdminLink.test.tsx

@@ -1,5 +1,5 @@
 import { QueryClient, QueryClientProvider } from '@tanstack/react-query'
-import { cleanup, render, screen } from '@testing-library/react'
+import { render, screen } from '@testing-library/react'
 import { http, HttpResponse } from 'msw2'
 import { setupServer } from 'msw2/node'
 import { MemoryRouter, Route } from 'react-router-dom'
@@ -32,7 +32,6 @@ beforeAll(() => {
 })
 
 beforeEach(() => {
- cleanup()
  server.resetHandlers()
  queryClient.clear()
 })

src/layouts/Header/components/GuestHeader/GuestHeader.test.tsx

@@ -1,5 +1,5 @@
 import { QueryClient, QueryClientProvider } from '@tanstack/react-query'
-import { cleanup, render, screen } from '@testing-library/react'
+import { render, screen } from '@testing-library/react'
 import React from 'react'
 import { MemoryRouter, Route } from 'react-router-dom'
 import { type Mock } from 'vitest'
@@ -40,7 +40,6 @@ beforeEach(() => {
 })
 
 afterEach(() => {
- cleanup()
  queryClient.clear()
 })
 

src/layouts/Header/components/Navigator/MyContextSwitcher.test.tsx

@@ -1,5 +1,5 @@
 import { QueryClient, QueryClientProvider } from '@tanstack/react-query'
-import { cleanup, render, screen, waitFor } from '@testing-library/react'
+import { render, screen, waitFor } from '@testing-library/react'
 import userEvent from '@testing-library/user-event'
 import { graphql, HttpResponse } from 'msw2'
 import { setupServer } from 'msw2/node'
@@ -59,7 +59,6 @@ beforeAll(() => {
 })
 
 afterEach(() => {
- cleanup()
  queryClient.clear()
  server.restoreHandlers()
 })

src/layouts/Header/components/SeatDetails/SeatDetails.test.tsx

@@ -1,5 +1,5 @@
 import { QueryClient, QueryClientProvider } from '@tanstack/react-query'
-import { cleanup, render, screen } from '@testing-library/react'
+import { render, screen } from '@testing-library/react'
 import { graphql, HttpResponse } from 'msw2'
 import { setupServer } from 'msw2/node'
 import { MemoryRouter, Route } from 'react-router-dom'
@@ -43,7 +43,6 @@ beforeAll(() => {
 })
 
 afterEach(() => {
- cleanup()
  server.resetHandlers()
  queryClient.clear()
 })

src/layouts/Header/components/ThemeToggle/ThemeToggle.test.tsx

@@ -1,11 +1,5 @@
 import * as Sentry from '@sentry/react'
-import {
- cleanup,
- render,
- screen,
- waitFor,
- within,
-} from '@testing-library/react'
+import { render, screen, waitFor, within } from '@testing-library/react'
 import userEvent from '@testing-library/user-event'
 
 import { ThemeContextProvider } from 'shared/ThemeContext'
@@ -33,7 +27,6 @@ describe('ThemeToggle', () => {
  })
 
  afterEach(() => {
- cleanup()
  vi.clearAllMocks()
  })
 

src/layouts/Header/components/UserDropdown/UserDropdown.test.tsx

@@ -1,5 +1,5 @@
 import { QueryClient, QueryClientProvider } from '@tanstack/react-query'
-import { cleanup, render, screen, waitFor } from '@testing-library/react'
+import { render, screen, waitFor } from '@testing-library/react'
 import userEvent from '@testing-library/user-event'
 import { graphql, http, HttpResponse } from 'msw2'
 import { setupServer } from 'msw2/node'
@@ -92,7 +92,6 @@ beforeAll(() => {
 })
 
 afterEach(() => {
- cleanup()
  queryClient.clear()
  server.resetHandlers()
 })

src/layouts/LoginLayout/LoginLayout.test.tsx

@@ -1,5 +1,5 @@
 import { QueryClient, QueryClientProvider } from '@tanstack/react-query'
-import { cleanup, render, screen, waitFor } from '@testing-library/react'
+import { render, screen, waitFor } from '@testing-library/react'
 import { graphql, HttpResponse } from 'msw2'
 import { setupServer } from 'msw2/node'
 import { MemoryRouter, Route } from 'react-router-dom'
@@ -44,7 +44,6 @@ beforeAll(() => {
 afterEach(() => {
  queryClient.clear()
  server.resetHandlers()
- cleanup()
  vi.clearAllMocks()
 })
 

src/ui/TotalsNumber/TotalsNumber.test.tsx

@@ -1,11 +1,7 @@
-import { cleanup, render, screen } from '@testing-library/react'
+import { render, screen } from '@testing-library/react'
 
 import TotalsNumber from './TotalsNumber'
 
-afterEach(() => {
- cleanup()
-})
-
 describe('TotalsNumber', () => {
  describe('when rendered', () => {
  it('renders commit change when there is a valid value', () => {

src/ui/TruncatedMessage/TruncatedMessage.test.tsx

@@ -1,15 +1,11 @@
-import { cleanup, render, screen } from '@testing-library/react'
+import { render, screen } from '@testing-library/react'
 import userEvent from '@testing-library/user-event'
 
 import { useTruncation } from './hooks'
 import TruncatedMessage from './TruncatedMessage'
 
 vi.mock('./hooks')
 
-afterEach(() => {
- cleanup()
-})
-
 describe('TruncatedMessage', () => {
  function setup({ canTruncate = false }) {
  // @ts-expect-error

src/ui/TruncatedMessage/hooks/useTruncation.test.ts

@@ -1,4 +1,4 @@
-import { cleanup, renderHook } from '@testing-library/react'
+import { renderHook } from '@testing-library/react'
 
 import { useTruncation } from './useTruncation'
 
@@ -36,10 +36,6 @@ vi.mock('react', async () => {
  }
 })
 
-afterEach(() => {
- cleanup()
-})
-
 describe('useTruncation', () => {
  function setup({
  clientHeight = 0,