Skip to content

Conversation

@heath-freenome
Copy link
Member

Reasons for making this change

In order to prep for React 19, began conversion from react-test-renderer to @testing-library/react for the arrayTests.tsx

  • Updated snapshot-tests to convert the arrayTests.tsx to use @testing-library/react
  • Updated all snapshots accordingly

Checklist

  • I'm updating documentation
  • I'm adding or updating code
    • I've added and/or updated tests. I've run npx nx run-many --target=build --exclude=@rjsf/docs && npm run test:update to update snapshots, if needed.
    • I've updated docs if needed
    • I've updated the changelog with a description of the PR
  • I'm adding a new feature
    • I've updated the playground with an example use of the feature

@heath-freenome heath-freenome marked this pull request as draft November 3, 2025 07:09
@heath-freenome heath-freenome changed the title Chore update array snapshot test to use @testing-library/react Chore update array snapshot tests to use @testing-library/react Nov 3, 2025
@heath-freenome heath-freenome force-pushed the testing-library-array-snapshot-tests branch from dfd7448 to e945094 Compare November 3, 2025 16:53
@heath-freenome heath-freenome marked this pull request as ready for review November 3, 2025 16:54
In order to prep for React 19, began conversion from `react-test-renderer` to `@testing-library/react` for the `arrayTests.tsx`
- Updated `snapshot-tests` to convert the `arrayTests.tsx` to use `@testing-library/react`
- Updated all snapshots accordingly
@heath-freenome heath-freenome force-pushed the testing-library-array-snapshot-tests branch from e945094 to fa06339 Compare November 4, 2025 21:44
@heath-freenome heath-freenome merged commit d451e90 into rjsf-team:main Nov 4, 2025
4 checks passed
@heath-freenome heath-freenome deleted the testing-library-array-snapshot-tests branch November 4, 2025 21:52
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.

2 participants