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
Headline | Docs | Testing and Documentation for Headline Component #1210
Conversation
Hey @mikemai2awesome - can you review this? Some questions in the PR. |
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.
@TashaBasalt Good work. Lets go through the TESTING.md with a fine tooth comb with an eye for "how to describe this for someone with minimal technical expertise" and for typos.
Just adding here that the quotes should be present.
@joekarasek @TashaBasalt just checking in -- what's the status on this one? |
@sghoweri We're still waiting on confirmation for whether or not to have the TESTING.md describe a successful function that isn't actually functional. |
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.
Issue with the "quoted" variation has been resolved. We added that functionality to the the TESTING.md docs even though it is currently not working.
@sghoweri @mikemai2awesome This is now ready for final review/merge.
|
Thanks guys!! |
Jira
http://vjira2:8080/browse/BDS-986
Summary
Adding Jest Snapshot tests of headline component and a TESTING.md file.
Details
Added tests to cover various combinations of font sizes and weights, but I'm not sure that amount of specificity is required. There is also a 'quoted' example of text in the Bolt Pattern Lab, but the sample text isn't actually quoted in any way. Is that an error?
How to test
yarn test:js