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

[Cypress] Refactor snapshot upload process #124

Merged
merged 4 commits into from
Sep 10, 2024
Merged

Conversation

omacranger
Copy link
Member

@omacranger omacranger commented Sep 5, 2024

Description

  • Refactors the snapshot upload process in Cypress to handle it immediately after each snapshot instead of at the end of the spec.
  • Fixes / re-exposes the sauceVisualResults feature in a spec to allow asserting on results and tweaks the exponential backoff delays to make it more usable.

Types of Changes

What types of changes does your code introduce? Keep the ones that apply:

  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Refactor/improvements

remove second finishBuild in test
@omacranger omacranger merged commit 8a64e27 into main Sep 10, 2024
2 checks passed
@omacranger omacranger deleted the bugfix/cypress-updates branch September 10, 2024 12:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants