zebrunner / carina

Carina automation framework (TestNG): Web, Mobile, API, DB etc testing...
https://zebrunner.github.io/carina
Apache License 2.0
954 stars 218 forks source link

revert retries in web and mobile command executors #1654

Closed vdelendik closed 2 years ago

vdelendik commented 2 years ago

in 7.3 we introduced retries for web and mobile command executors. it fixed temporary rare case when esg can't return response in time... at the same time set of complains raised that every retry for mobile and web takes up to 5-10m. Especially case when driver exited due to the idleTimeout.

let remove retries and test original issue as some improvements delivered in ESG

vdelendik commented 2 years ago

done: https://github.com/zebrunner/carina/pull/1655

vdelendik commented 2 years ago

after additional testing retries removed on mobile and keep for the web. at the same time such updates for the web testing:

  1. minimized number of attempts from 10 to 2
  2. don't retry for driver quit operation
vdelendik commented 2 years ago

confirmed by customers using snapshot build