-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
test: added unit tests for SwitcherItem component to improve coverage #17876
test: added unit tests for SwitcherItem component to improve coverage #17876
Conversation
✅ Deploy Preview for v11-carbon-web-components ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for carbon-elements ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for v11-carbon-react ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #17876 +/- ##
==========================================
+ Coverage 81.84% 81.92% +0.08%
==========================================
Files 404 404
Lines 14095 14095
Branches 4413 4364 -49
==========================================
+ Hits 11536 11548 +12
+ Misses 2397 2385 -12
Partials 162 162 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good to me!
Hi @kennylam , can you please review this? |
4e5ffb2
Closes #17494
This PR adds unit tests for the SwitcherItem component to increase its coverage by ensuring its proper rendering and functionality. The tests verify that the skeleton displays with the correct classes, supports custom class names etc.
Changelog
Changed
Testing / Reviewing
Additional Notes