-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Initial attempt to split storybook stories into separate snapshots #21484
Conversation
Size Change: 0 B Total Size: 889 kB ℹ️ View Unchanged
|
return currentElement; | ||
}, | ||
} ) ), | ||
integrityOptions: { cwd: __dirname }, // it will start searching from the current directory |
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.
This mocking logic is essential to make tests pass when components use createRef
.
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.
Yes, I was waiting to see which direction #21485 went before spending more time getting all the tests running correctly. I didn't want to spend the time if the decision is to drop storyshots altogether.
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.
we can always make those snapshots more useful but adding additional assertions like running a11y checks on the generated content.
Interesting exploration 👍 |
Should it be closed now that we are removing StoryShots? 😃 |
Yes, was just waiting for that to be merged 👍 |
Work in Progress
Description
Updates the storybook add-on story shots to save snapshots in separate stories for each component.
Due to what appears to be an unfixed bug in story shots, each story will also have to export
parameters: { fileName: __filename },
How has this been tested?
Screenshots
Types of changes
Checklist: