Rerun fragile tests only with status == 'fail'

ylobankov · ylobankov · commit 5eadc1661906 · 2022-02-16T20:41:05.000+03:00
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.
diff --git a/lib/worker.py b/lib/worker.py
@@ -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':

Original file line number	Diff line number	Diff line change
`@@ -352,7 +352,7 @@ def run_loop(self, task_queue, result_queue):`
`352`	`352`	`)`
`353`	`353`	`retries_left = self.suite.fragile_retries()`
`354`	`354`	`# let's run till short_status became 'pass'`
`355`		`- while short_status != 'pass' and retries_left >= 0:`
	`355`	`+ while short_status in (None, 'fail') and retries_left >= 0:`
`356`	`356`	`self.restart_server()`
`357`	`357`	`# print message only after some fails occurred`
`358`	`358`	`if short_status == 'fail':`