refactor: delete unused test-utils Importer#21830
refactor: delete unused test-utils Importer#21830adamalston wants to merge 1 commit intocarbon-design-system:mainfrom
Conversation
✅ Deploy Preview for v11-carbon-react ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for v11-carbon-web-components ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #21830 +/- ##
==========================================
+ Coverage 87.75% 94.85% +7.09%
==========================================
Files 536 536
Lines 43637 45084 +1447
Branches 6725 7948 +1223
==========================================
+ Hits 38293 42763 +4470
+ Misses 5193 2181 -3012
+ Partials 151 140 -11
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
I don't plan to mark this pull request as ready for review until some of my earlier pull requests have been reviewed. I want to avoid diverting attention from them. Otherwise, these changes are ready to go. |
No issue.
Deleted the unused test-utils
Importer.Changelog
Removed
Importer.Testing / Reviewing
I saw that the module did not have any test coverage. When I looked into adding tests, I found that the file was not used anywhere, so I deleted it. If anyone thinks it should be kept, let me know and I can look into adding tests for it.
PR Checklist
As the author of this PR, before marking ready for review, confirm you:
Updated documentation and storybook examplesWrote passing tests that cover this changeAddressed any impact on accessibility (a11y)Tested for cross-browser consistencyMore details can be found in the pull request guide