-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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(fluid-components): increase coverage and fix skeleton components #17823
test(fluid-components): increase coverage and fix skeleton components #17823
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!
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 #17823 +/- ##
==========================================
+ Coverage 80.35% 80.54% +0.19%
==========================================
Files 406 406
Lines 14041 14038 -3
Branches 4395 4395
==========================================
+ Hits 11282 11307 +25
+ Misses 2593 2563 -30
- Partials 166 168 +2 ☔ View full report in Codecov by Sentry. |
✅ Deploy Preview for v11-carbon-react ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
LGTM!
b7c523a
Closes #17478
Closes #17487
Closes #17486
Closes #17514
Closes #17515
Closes #17518
Closes #17511
Closes #17519
Increase test coverage to 100% for FluidTextAreaSkeleton, FluidSelectSkeleton, FluidComboBoxSkeleton, FluidPasswordInput, FluidTextInputSkeleton, FluidDropdownSkeleton, FluidMultiSelect
Changelog
New
-increased test coverages to 100%
Changed
className
is in the outsideTesting / Reviewing
confirm skeleton updates with animation
confirm tests are at 100%