electron / fiddle / 23473127630

89%

push

github

test: migrate Enzyme tests to React Testing Library (#1851)

* test: migrate Enzyme tests to React Testing Library

Replace all 21 Enzyme test files with React Testing Library equivalents.
Remove enzyme, enzyme-adapter-react-16, enzyme-to-json and their @types
packages. Remove Enzyme config from tests/setup.ts and vitest.config.ts.
Delete commands-spec.tsx (RTL equivalents already exist in rtl-spec/).
Delete all Enzyme-generated snapshot files.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix: address PR review feedback

- Revert output.tsx source change; provide MosaicContext.Provider in test
- Remove crypto.subtle polyfill from setup.ts (not needed on React 16)
- Migrate commands-spec.tsx to RTL instead of deleting (unique coverage)
- Fix userEvent import: default → named import in all test files
- Use renderClassComponentWithInstanceRef consistently, remove (as any)
- Remove unnecessary act() wrappers
- Revert changed test titles in dialog-bisect-spec.tsx
- Remove unused imports

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix: address remaining PR review feedback

- Add waitFor/act for pre-existing RTL tests that need them for MobX
  observer re-renders
- Remove 103 unnecessary (instance as any) casts; use typed instance
  from renderClassComponentWithInstanceRef. Remaining 17 casts are
  commented (private methods or Readonly<S> bypass).
- Fix no-op tests: bisect command tools, settings unknown page, dialogs
  settings — all now assert meaningful component output.
- Restore dropped coverage: button disabled states, radio ordering,
  file-selected render path, renderItem text verification.
- Remove 8 unnecessary act() wrappers around async methods that don't
  need synchronous state flush, matching rtl-spec/ convention.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix: address PR review feedback for RTL migration

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: David S... (continued)

1607 of 1746 branches covered (92.04%)

Branch coverage included in aggregate %.

9208 of 11848 relevant lines covered (77.72%)

33.04 hits per line