Skip to content

Commit

Permalink
Rerun fragile tests only with status == 'fail'
Browse files Browse the repository at this point in the history
Sometimes fragile tests can be skipped or disabled. Such tests were
considered as not passed and test-run rerun them which is obviously
redundant. This patch fixes the issue.
  • Loading branch information
ylobankov committed Feb 16, 2022
1 parent ce9f6ae commit 59506dd
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lib/worker.py
Expand Up @@ -352,7 +352,7 @@ def run_loop(self, task_queue, result_queue):
)
retries_left = self.suite.fragile_retries()
# let's run till short_status became 'pass'
while short_status != 'pass' and retries_left >= 0:
while short_status in [None, 'fail'] and retries_left >= 0:
self.restart_server()
# print message only after some fails occurred
if short_status == 'fail':
Expand Down

0 comments on commit 59506dd

Please sign in to comment.