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

Add retry-ability #169

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Add retry-ability #169

wants to merge 3 commits into from

Conversation

gbasto
Copy link

@gbasto gbasto commented Feb 22, 2021

Adds retry-ability to snapshot tests, by adding two new configuration options:

retryCount - the number of retries that should be done if the test is failing
retryDelay - the amount of time between each retry

This is useful, for example, for cases when some tests are failing because the image snapshot is done before some images are loaded due to slow network.

@YadrovSergey
Copy link

@meinaart please merge this pull requests.

@edimitchel
Copy link

edimitchel commented Mar 31, 2021

@meinaart could you look into this PR ? We need it !

@edimitchel
Copy link

@gbasto please look at my PR I made on your fork !

@scottrippey
Copy link

@meinaart This PR would be extremely useful ... I hope this project hasn't been abandoned! Please merge this PR. Thank you!

@gbasto
Copy link
Author

gbasto commented Jul 24, 2023

I've been using a patched version of cypress-plugin-snapshots with this change for the last 2 years and it has been working perfectly to me, however this PR still has no feedback from the repo maintainers.

Is this project completely abandoned? @meinaart let me know if you need any help, otherwise I'll just close it as it's tickling my OCD anytime I look into my open PRs and I always find this one stalled there 😅

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.

None yet

5 participants