Storybook: Storyshot should split tests in a way that Jest only tests changed components

Created on 28 Mar 2019  路  3Comments  路  Source: storybookjs/storybook

Is your feature request related to a problem? Please describe.
When running tests using Storyshots and Jest for React with CRA, it will run all tests for Storyshots even if only a single component changed. This makes the test running take a lot more time than required.

Describe the solution you'd like
When running test runner using Storyshots I would want Storyshots to split up the tests as if I've written separate test files. This way Jest will only run tests for changed components in stead of all of them.

Describe alternatives you've considered
The only alternative as I see it is to write the tests manually, which takes away the point of Storyshot.

Are you able to assist bring the feature to reality?
We could probably assist in the development of this feature

storyshots feature request inactive

Most helpful comment

jest's findRelatedTests option would do this. But it doesn't work for storyshot testing unfortunately :/

All 3 comments

Hi everyone! Seems like there hasn't been much going on in this issue lately. If there are still questions, comments, or bugs, please feel free to continue the discussion. Unfortunately, we don't have time to get to every issue. We are always open to contributions so please send us a pull request if you would like to help. Inactive issues will be closed after 30 days. Thanks!

Hey there, it's me again! I am going close this issue to help our maintainers focus on the current development roadmap instead. If the issue mentioned is still a concern, please open a new ticket and mention this old one. Cheers and thanks for using Storybook!

jest's findRelatedTests option would do this. But it doesn't work for storyshot testing unfortunately :/

Was this page helpful?
0 / 5 - 0 ratings

Related issues

zvictor picture zvictor  路  3Comments

tirli picture tirli  路  3Comments

purplecones picture purplecones  路  3Comments

dnlsandiego picture dnlsandiego  路  3Comments

MrOrz picture MrOrz  路  3Comments