-
Notifications
You must be signed in to change notification settings - Fork 14.6k
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: FormLabel dedicated directory and tests #13270
Conversation
Codecov Report
@@ Coverage Diff @@
## master #13270 +/- ##
===========================================
- Coverage 72.29% 62.20% -10.10%
===========================================
Files 864 567 -297
Lines 44883 20665 -24218
Branches 5403 5421 +18
===========================================
- Hits 32450 12854 -19596
+ Misses 12224 7602 -4622
Partials 209 209
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Thanks for the PR! this project is progressing so nicely, thanks to you all @geido @michael-s-molina @yardz 🙏 |
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.
Just a suggestion to remove the ID and use getByText
and parentNode
.
* Form label directory and tests * Fix linting * Remove unnecessary data-test-id
SUMMARY
This PR only includes tests for the FormLabel component but no storybook. The FormLabel is a React Bootstrap component that we aim to deprecate asap, so the goal of this PR is only to contribute to the stability of the system as it currently stands with no additional investment.
This PR also moves the FormLabel component to a dedicated directory.