search

DevExpress / testcafe-browser-provider-saucelabs

This is the Sauce Labs browser provider plugin for TestCafe.
https://devexpress.github.io/testcafe/
MIT License
31 stars 41 forks source link

Add retry option #47

Closed dimaip closed 1 year ago

dimaip commented 5 years ago

Saucelabs tests often fail with an error like this:

ERROR Error: 18 Jul 10:56:47 - Sauce Connect could not establish a connection.
    at handleError (/home/circleci/app/Packages/Application/Neos.Neos.Ui/node_modules/sauce-connect-launcher/lib/sauce-connect-launcher.js:410:17)

Would be nice to add an auto retry option for such connection failures. Wdyt?

edit: oh, there's MAX_TUNNEL_CONNECT_RETRY_COUNT. Maybe make it configurable in that case?

miherlosev commented 5 years ago

Hi @dimaip

At present, the MAX_TUNNEL_CONNECT_RETRY_COUNT value is not configurable. We will consider the ways how to make it configurable.

Also, I think it will be useful to add a delay between connect retries. @AndreyBelym what do you think about it?

github-actions[bot] commented 1 year ago

This issue has been automatically marked as stale because it has not had any activity for a long period. It will be closed and archived if no further activity occurs. However, we may return to this issue in the future. If it still affects you or you have any additional information regarding it, please leave a comment and we will keep it open.

github-actions[bot] commented 1 year ago

We're closing this issue after a prolonged period of inactivity. If it still affects you, please add a comment to this issue with up-to-date information. Thank you.