Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace enzyme with React Testing Library #440

Closed
Tracked by #195
feanil opened this issue Aug 2, 2024 · 2 comments · Fixed by #445
Closed
Tracked by #195

Replace enzyme with React Testing Library #440

feanil opened this issue Aug 2, 2024 · 2 comments · Fixed by #445
Assignees
Labels
maintenance Routine upkeep necessary for the health of the platform

Comments

@feanil
Copy link
Contributor

feanil commented Aug 2, 2024

This is part of openedx/public-engineering#195 (Enzyme Replacement)

The above ticket explains why but given that we want this to continue to work with the latest version of the openedx platform. We should make this migration in this repository as well.

@feanil
Copy link
Contributor Author

feanil commented Aug 2, 2024

@openedx/committers-cookie-policy-banner can one of you take on this work or otherwise schedule it to be done before the Sumac cut?

@feanil
Copy link
Contributor Author

feanil commented Aug 2, 2024

Looking at the repo though, it looks like this repo is already running tests with Node 18 so I wonder if the tests have all moved but we just haven't gotten around to removing the old dependency.

@feanil feanil added the maintenance Routine upkeep necessary for the health of the platform label Aug 7, 2024
@GlugovGrGlib GlugovGrGlib self-assigned this Aug 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Routine upkeep necessary for the health of the platform
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants