inveniosoftware / inveniosoftware/react-invenio-app-ils

Migrate testing library away from enzyme

Open
#667 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
6
Forks
24
Avg merge
2d 46m
Merged PRs (30d)
3

Description

After the project was [moved to react version 18](https://github.com/CERNDocumentServer/cds-ils/issues/1007) we lost official `enzyme` support.
To make the tests pass for now, the unofficial adapter [`@cfaester/enzyme-adapter-react-18`](https://www.npmjs.com/package/@cfaester/enzyme-adapter-react-18) was added.
This should not be a permanent solution and moving to a newer testing library like the [React Testing Library](https://testing-library.com/docs/react-testing-library/intro/).

Contributor guide

No contributing guide indexed for this repository

Research direction

Begin by inventorying the current Enzyme-based tests and the @cfaester/enzyme-adapter-react-18 setup, then compare the migration target with React Testing Library documentation. Replace the temporary adapter and migrate the tests so they run on React 18 without Enzyme; done means the test suite passes with the new library.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript, react
Domain
frontend, testing
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.