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.

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

Release 0.8.0 #199

Merged
merged 12 commits into from Oct 7, 2022
Merged

Release 0.8.0 #199

merged 12 commits into from Oct 7, 2022

Conversation

yannbf
Copy link
Member

@yannbf yannbf commented Oct 7, 2022

This release includes:

#194 Fix StorybookTestRunnerError – length on undefined
#198 Add isTestRunner utility

ericclemmons and others added 12 commits September 21, 2022 09:46
When running `yarn test-storybook` on my story, a 3rd party library throws an uncaught exception, but `StorybookTestRunnerError` _assumes_ `logs` is an array, which throws this error:

```console
UnhandledPromiseRejectionWarning: page.evaluate: TypeError: Cannot read properties of undefined (reading 'length')
    at new StorybookTestRunnerError (<anonymous>:58:34)
    at __throwError (<anonymous>:71:15)
    at eval (eval at evaluate (:192:30), <anonymous>:4:19)
    at UtilityScript.evaluate (<anonymous>:194:17)
    at UtilityScript.<anonymous> (<anonymous>:1:44)
```

This PR simply **defaults `logs` to `[]`**, so I can correctly see that the underlying error is _actually_ coming from a 3rd party library:

```console
 Cannot read properties of null (reading 'focus')
```
Fix StorybookTestRunnerError – length on undefined
@yannbf yannbf added the minor Increment the minor version when merged label Oct 7, 2022
@yannbf yannbf merged commit 479d3a2 into main Oct 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor Increment the minor version when merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants