Skip to content

test: migrate Enzyme tests to React Testing Library#1850

Closed
felixrieseberg wants to merge 1 commit intofelixr/react-18-corefrom
felixr/enzyme-to-rtl
Closed

test: migrate Enzyme tests to React Testing Library#1850
felixrieseberg wants to merge 1 commit intofelixr/react-18-corefrom
felixr/enzyme-to-rtl

Conversation

@felixrieseberg
Copy link
Member

Summary

RTL replacements for all 21 Enzyme test files removed in #1849. This is a pure test-only PR — no source code changes.

Changes

All 21 test files rewritten from Enzyme (shallow/mount) to React Testing Library (render/screen/userEvent):

  • dialog-add-theme-spec, dialog-add-version-spec, dialog-bisect-spec
  • dialog-generic-spec, dialog-token-spec, dialogs-spec
  • header-spec, output-spec
  • settings-credits-spec, settings-electron-spec, settings-execution-spec
  • settings-general-appearance-spec, settings-general-block-accelerators-spec
  • settings-general-console-spec, settings-general-font-spec
  • settings-general-gist-spec, settings-general-github-spec
  • settings-general-mirror-spec, settings-general-spec
  • settings-spec, sidebar-package-manager-spec

Verification

  • tsc --noEmit: 0 errors
  • vitest run: 91 files, 806 passed (full coverage restored)

Add RTL replacements for all 21 Enzyme test files that were removed
in the React 18 upgrade. Tests use render/screen/userEvent from
@testing-library/react instead of shallow/mount from Enzyme.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
@felixrieseberg
Copy link
Member Author

Splitting differently — RTL migration first, React 18 upgrade later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant