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

Exit code is 0 when snapshot limit is exceeded #743

Open
bernardobelchior opened this issue May 10, 2023 · 2 comments
Open

Exit code is 0 when snapshot limit is exceeded #743

bernardobelchior opened this issue May 10, 2023 · 2 comments
Labels
✨ enhancement New feature or request

Comments

@bernardobelchior
Copy link

The problem

When running PERCY_TOKEN=<token> npx percy storybook ./storybook-static and the snapshot limit has been reached, the CLI does not return a non-zero exit code. When using in CI, it means CI won't fail and will instead pass even though tests could be failing.

This happens both in my local mac as well as on CI (GH Actions running on ubuntu-latest).

Environment

  • Node version: v18.16.0 (or v16 for GH Actions)
  • @percy/cli version: 1.24.0
  • Version of Percy SDK you’re using: 4.3.6
  • If needed, a build or snapshot ID: N/A
  • OS version: macOS 13.3.1; Ubuntu (ubuntu-latest from GH Actions)
  • Type of shell command-line [interface]: zsh on macOS; Our CI is GitHub Actions running on ubuntu-latest.

Details

$ PERCY_TOKEN=<token> npx percy storybook ./storybook-static
[percy] Percy has started!
  <truncated for brevity>
[percy] Failed to create build
[percy] Error: This organization has exceeded the limits of the Percy BrowserStack plan plan. Administrators can upgrade here: https://percy.io/organizations/<censored>/billing
[percy] Build not created

$ echo $?
0

Debug logs

See above

Code to reproduce issue

N/A

@samarsault
Copy link
Contributor

samarsault commented May 11, 2023

Thanks for reaching out. We already have this on our roadmap and will be adding this soon

@samarsault samarsault added the ✨ enhancement New feature or request label May 11, 2023
@bernardobelchior
Copy link
Author

Thank you, @samarsault 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants