[BD-46] feat: move Enzyme to React testing library #2440

[khudym]

Description

Include a description of your changes here, along with a link to any relevant Jira tickets and/or Github issues.

Deploy Preview

Include a direct link to your changes in this PR's deploy preview here (e.g., a specific component page).

Merge Checklist

• If your update includes visual changes, have they been reviewed by a designer? Send them a link to the Netlify deploy preview, if applicable.
• Does your change adhere to the documented style conventions?
• Do any prop types have missing descriptions in the Props API tables in the documentation site (check deploy preview)?
• Were your changes tested using all available themes (see theme switcher in the header of the deploy preview, under the "Settings" icon)?
• Were your changes tested in the example app?
• Is there adequate test coverage for your changes?
• Consider whether this change needs to reviewed/QA'ed for accessibility (a11y). If so, please add wittjeff and adamstankiewicz as reviewers on this PR.

Post-merge Checklist

• Verify your changes were released to NPM at the expected version.
• If you'd like, share your contribution in #show-and-tell.
• 🎉 🙌 Celebrate! Thanks for your contribution.

[openedx-webhooks]

Thanks for the pull request, @khudym!

When this pull request is ready, tag your edX technical lead.

[netlify]

✅ Deploy Preview for paragon-openedx ready!

Built without sensitive environment variables

Name	Link
🔨 Latest commit	6fc6386
🔍 Latest deploy log	https://app.netlify.com/sites/paragon-openedx/deploys/652691b96a98800008e41995
😎 Deploy Preview	https://deploy-preview-2581--paragon-openedx.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[viktorrusakov]

@khudym Left some additional comments. I only went through third of the files for now, so expect some more comments later on 🙂

Another general comment is we can probably remove enzyme from package.json since apparently we no longer use it in the code.

[PKulkoRaccoonGang]

@khudym Great job! I think I've been reviewing this forever 😄 Left minor comments, please review them.
Please add a description for the pull request

[PKulkoRaccoonGang]

Good job!

[viktorrusakov]

@khudym Thanks for the updates! Couple of additional feedback points:

1. Please rebase on master branch, I know the UI in this PR is a little bugged for you and might not display the conflicts but I checked it locally and there area actually a bunch of conflicts with master branch
2. You're still using a lot of fireEvents, please consider replacing them with userEvent wherever possible as it more closely resembles user's behaviour. Here's a good read on why that is the case https://testing-library.com/docs/user-event/intro
3. I noticed that in Modal folder we have a bunch of test files for components and we also have a tests subfolder which contains only one ModalDialog.test.jsx file for some reason. Can you please colocate all of the Modal test files in tests folder?
4. Tests for the deprecated Dropdown component still use enzyme and are failing because we removed enzyme from the project. Please replace enzyme with react-testing-library in those tests.

[openedx-webhooks]

@khudym 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future.

[edx-semantic-release]

🎉 This PR is included in version 21.5.1 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

[edx-semantic-release]

🎉 This PR is included in version 22.0.0-alpha.11 🎉

The release is available on:

• npm package (@alpha dist-tag)
• GitHub release

Your semantic-release bot 📦🚀