-
Notifications
You must be signed in to change notification settings - Fork 63
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
Feat: Adds jest-axe and Card component test #2440
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. |
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.
🎉 Great! 🎉
I've left only a few small comments 😄
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.
Great!
What's the purpose of this pull request?
@testing-library/jest-dom
to help to test our components when checking an element's attributes, its text content, its css classes...Card
component testsCheckboxField
component testTODO: Document
How it works?
Currently, our components library do not have tests, but we want to introduce them for upcoming components. For existing components, we will gradually add tests retroactively.
How to test it?
packages/components
-> runyarn install
yarn test
, you will be able to see the tests running<Label>
component fromCheckboxField
, when running the step 2, you will get:References
https://testing-library.com/docs/ecosystem-jest-dom
testing-library/jest-dom#546
NickColley/jest-axe#79