Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[TEST] Setup chromatic ignore stories #101

Closed
pspaczek opened this issue Feb 27, 2023 · 0 comments · Fixed by #103
Closed

[TEST] Setup chromatic ignore stories #101

pspaczek opened this issue Feb 27, 2023 · 0 comments · Fixed by #103
Assignees
@pspaczek

Comments

@pspaczek
Copy link
Contributor

pspaczek commented Feb 27, 2023 •
edited
Loading

How this feature will help us
Currently we are testing all stories. This approach is it is inefficient. When we test stories that look the same we are burn snapshots without any value. The most common case of such tests are stories "With slot...". ignoring these stories will save us free snapshots.

Considered solution
Add

 parameters: {
    chromatic: { disableSnapshot: true },
  },

to stories.
https://www.chromatic.com/docs/ignoring-elements
@pspaczek pspaczek self-assigned this Feb 27, 2023
@pspaczek pspaczek mentioned this issue Feb 27, 2023
Merged
@pspaczek pspaczek reopened this Feb 27, 2023
@pspaczek pspaczek changed the title [TEST] Setup ignore stories and viewports for chromatic [TEST] Setup ignore stories Mar 6, 2023
@pspaczek pspaczek closed this as completed Mar 6, 2023
@pspaczek pspaczek changed the title [TEST] Setup ignore stories [TEST] Setup chromatic ignore stories Mar 6, 2023
@pspaczek pspaczek mentioned this issue Mar 6, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pspaczek pspaczek

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

test(chromatic): setup ignore stories pspaczek/component-library
1 participant
@pspaczek