-
Notifications
You must be signed in to change notification settings - Fork 215
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: Explicitly enable snapshots #478
Conversation
Test summaryRun details
View run in Cypress Dashboard ➡️ This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard |
@@ -171,7 +171,12 @@ const CheckboxStates = () => ( | |||
); | |||
|
|||
storiesOf('Components|Inputs/Checkbox/React/Visual Testing', module) | |||
.addParameters({component: Checkbox}) | |||
.addParameters({ | |||
component: Checkbox, |
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.
Interesting. It seems like Storybook is able to get the component just fine without this, but I'm not sure why.
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.
I think we'll need to add a note in our documentation about this. We've been saying every story is automatically a visual regression - now we have to say it is not. Our documentation will have to say that visual stories are now intentional and we have to opt into capturing.
Summary
We are testing way too many stories that aren't useful for visual regression. This PR tells Chromatic to ignore all stories by default and only include ones that are marked.
Checklist
yarn test
passespackage.json
canvas-kit-react
and/orcanvas-kit-css
universal modules, ifapplicable
Additional References