Project

General

Profile

action #59043

Updated by okurz 7 months ago

## Motivation

https://github.com/os-autoinst/openQA/pull/2455 introduced a `sleep 8` to stabilize a test. Having sleeps in code is better than unstable tests but even better would be to have reliable code that explicitly synchronizes on necessary conditions rather than blind sleeps. Also we have more flaky/unstable tests which we maintain in `.circleci/unstable_tests.txt`.

## Acceptance criteria
* **AC1:** DONE: ~~`sleep 8` of https://github.com/os-autoinst/openQA/pull/2455 is removed~~
~~* * **AC2:** `.circleci/unstable_tests.txt` is empty again
~~


## Suggestions
* Remove explicit and implicit sleep time, e.g. as in https://github.com/os-autoinst/openQA/pull/2626 for the 2s selenium test method time
* Run tests from `.circleci/unstable_tests.txt` locally, reproduce problems and fix them
* Ensure tests also run stable within CI, not only locally

Back