Ability for custom storyshots testFunctions to utilise "snapshot per story file"

[digitalmaster]

Issue:

Can't use multiSnapshotWithOptions with custom test function because getSnapshotFileName is not exported.

What I did

This is a small tweak to the new multiSnapshotWithOptions test function. It basically just exports the getSnapshotFileName function so it can be used by those of use who have implemented our own custom test function. I also did a minor refactor and moved the getSnapshotFileName function to the utils.js file. Do let me know if this doesn't make any sense.

How to test

Go to storybook\addons\storyshots
and run yarn example

Is this testable with jest or storyshots?
Storyshots

Does this need a new example in the kitchen sink apps?
It should not.

Does this need an update to the documentation?
[Updated Docs] Yes.. We could mention this export in the docs so other users could become aware of it.

If your answer is yes to any of these, please make sure to include it in your PR.

[codecov]

Codecov Report

Merging #1841 into release/3.3 will increase coverage by 0.01%.
The diff coverage is 75%.

@@               Coverage Diff               @@
##           release/3.3    #1841      +/-   ##
===============================================
+ Coverage        22.31%   22.32%   +0.01%     
===============================================
  Files              324      324              
  Lines             6322     6319       -3     
  Branches           797      795       -2     
===============================================
  Hits              1411     1411              
+ Misses            4323     4316       -7     
- Partials           588      592       +4

Impacted Files	Coverage Δ
addons/storyshots/src/test-bodies.js	19.14% <ø> (-3.08%)	⬇️
addons/storyshots/src/index.js	80.85% <ø> (ø)	⬆️
addons/storyshots/src/utils.js	87.5% <75%> (-12.5%)	⬇️
...codemod/src/transforms/update-organisation-name.js	40.62% <0%> (ø)	⬆️
...react-native/src/manager/components/PreviewHelp.js	0% <0%> (ø)	⬆️
app/react-native/src/bin/storybook-build.js	0% <0%> (ø)	⬆️
.../ui/src/modules/ui/components/layout/dimensions.js	15.62% <0%> (ø)	⬆️
lib/codemod/src/transforms/update-addon-info.js	51.16% <0%> (ø)	⬆️
addons/info/src/components/Node.js	39.43% <0%> (ø)	⬆️
addons/knobs/src/components/types/Color.js	8.1% <0%> (ø)	⬆️
... and 31 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7a34c63...5c1d8ff. Read the comment docs.

[ndelangen]

Fantastic first PR @digitalmaster !

I'd like @igor-dv to have a look before merging 👍

[igor-dv]

Cool. Can you please add an example to the readme of how to use it ?

[digitalmaster]

@ndelangen Thanks a lot! Glad others also find this useful :)

@igor-dv Added an example.. Let me know if it's unclear or has any bugs. 😬

[igor-dv]

@digitalmaster, thanks, looks great !

[igor-dv]

@digitalmaster, wanna merge ?

[digitalmaster]

Sure.. on to this 3.3 release right? storybooks:release/3.3? Just wanna confirm... Also don't think i have push access.

[igor-dv]

yeah to the 3.3

[digitalmaster]

@igor-dv I can't merge this PR myself without having push access to this repo right? Or am I missing something?

[igor-dv]

Ah, so let me push it then 😃

[digitalmaster]

Awesome thanks.

[digitalmaster]

Do you guys have any idea when another release will be created?