Support Storybook 7.0 play function exceptions

[tmeasday]

See storybookjs/storybook#19143

We need to ensure that stories that do not emit an STORY_THREW_EXCEPTION on play function error are still handled. (SBs with hidePlayFunctionExceptions set)

To test:

1. Run a SB7.0 sandbox with interactions installed and a failing play function
2. ./bin/test-storybook.js --url http://localhost:6006 -i example-button.test.js
3. Do the above with interactions removed from the SB.

@yannbf I'm not how to automate the above testing, perhaps you can give some guidance, if it's possible?

📦 Published PR as canary version: 0.7.2--canary.190.e69606f.0

✨ Test out this PR locally via:

npm install @storybook/test-runner@0.7.2--canary.190.e69606f.0
# or 
yarn add @storybook/test-runner@0.7.2--canary.190.e69606f.0

Version

Published prerelease version: v0.7.2-next.0

Changelog

🐛 Bug Fix

• Support Storybook 7.0 play function exceptions #190 (@tmeasday @yannbf)
• fix snapshots & remove vscode color settings #192 (@ndelangen)

Authors: 3

• Norbert de Langen (@ndelangen)
• Tom Coleman (@tmeasday)
• Yann Braga (@yannbf)

[yannbf]

I'll take a look at this soon, but first, are there more changes required here? Given the new parameter that could/should be set in case it's not? @tmeasday @shilman

[tmeasday]

@yannbf, nope this should do it. There are two things that can happen from the SB side:

1. They have throwPlayFunctionExceptions set (the default): the SB will emit a PLAY_FUNCTION_THREW_EXCEPTION and and a STORY_THREW_EXCEPTION.

2. They have it set to false (using the interactions addon): the SB will emit a PLAY_FUNCTION_EXCEPTION and that's it.

Either way as long as we listen to both events we should be fine. The breaking change here is in case 2. where the STORY_THREW_EXCEPTION is no longer emitted.

[yannbf]

@yannbf, nope this should do it. There are two things that can happen from the SB side:

1. They have throwPlayFunctionExceptions set (the default): the SB will emit a PLAY_FUNCTION_THREW_EXCEPTION and and a STORY_THREW_EXCEPTION.
2. They have it set to false (using the interactions addon): the SB will emit a PLAY_FUNCTION_EXCEPTION and that's it.

Either way as long as we listen to both events we should be fine.

Got it! I'll take a look at it tomorrow and check regarding the tests

[yannbf]

Thanks @tmeasday !!

[yannbf]

@yannbf I'm not how to automate the above testing, perhaps you can give some guidance, if it's possible?

This test is already automated via the nightly check!

[github-actions]

🚀 PR was released in v0.7.2 🚀