From 79e7a1efaa20ca3590a6fd3d661de4e9525dbe52 Mon Sep 17 00:00:00 2001 From: Vasileios Karakasis Date: Wed, 24 Feb 2021 18:49:49 +0100 Subject: [PATCH 1/2] Fix status printout in case of retries --- reframe/frontend/executors/__init__.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/reframe/frontend/executors/__init__.py b/reframe/frontend/executors/__init__.py index d13606087f..7a73b98e52 100644 --- a/reframe/frontend/executors/__init__.py +++ b/reframe/frontend/executors/__init__.py @@ -415,7 +415,7 @@ def runall(self, testcases, restored_cases=None): # Print the summary line num_failures = len(self._stats.failed()) num_completed = len(self._stats.completed()) - if num_failures > 0 or num_completed < len(testcases): + if num_failures > 0 or num_completed < len(self._stats.tasks()): status = 'FAILED' else: status = 'PASSED' From 7b44b4a28584ac1bbedf515477efcfe78f56eda5 Mon Sep 17 00:00:00 2001 From: Vasileios Karakasis Date: Fri, 26 Feb 2021 22:47:59 +0100 Subject: [PATCH 2/2] Address PR comments --- reframe/frontend/executors/__init__.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/reframe/frontend/executors/__init__.py b/reframe/frontend/executors/__init__.py index 7a73b98e52..3173b52c0d 100644 --- a/reframe/frontend/executors/__init__.py +++ b/reframe/frontend/executors/__init__.py @@ -415,14 +415,16 @@ def runall(self, testcases, restored_cases=None): # Print the summary line num_failures = len(self._stats.failed()) num_completed = len(self._stats.completed()) - if num_failures > 0 or num_completed < len(self._stats.tasks()): + num_tasks = len(self._stats.tasks()) + if num_failures > 0 or num_completed < num_tasks: status = 'FAILED' else: status = 'PASSED' + total_run = len(testcases) - num_tasks + num_completed self._printer.status( status, - f'Ran {num_completed}/{len(testcases)}' + f'Ran {num_completed}/{total_run}' f' test case(s) from {num_checks} check(s) ' f'({num_failures} failure(s))', just='center'