From c1578cb9bf57b57e5fb94d69a2bb0c93bbee6f29 Mon Sep 17 00:00:00 2001 From: Tom Hromatka Date: Thu, 24 Dec 2020 21:13:40 +0000 Subject: [PATCH] ftests: Fix failure cause when multiple tests don't pass When multiple tests don't pass, the last failure_cause recorded was the one being displayed. Append the failure_cause to the failed_tests[] and skipped_tests[] arrays, so that each failure is properly saved and reported. Signed-off-by: Tom Hromatka --- ftests/ftests.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/ftests/ftests.py b/ftests/ftests.py index fad462d4..59717890 100755 --- a/ftests/ftests.py +++ b/ftests/ftests.py @@ -215,9 +215,9 @@ def run_tests(config): if ret == consts.TEST_PASSED: passed_tests.append([filename, run_time]) elif ret == consts.TEST_FAILED: - failed_tests.append([filename, run_time]) + failed_tests.append([filename, run_time, failure_cause]) elif ret == consts.TEST_SKIPPED: - skipped_tests.append([filename, run_time]) + skipped_tests.append([filename, run_time, failure_cause]) else: raise ValueException('Unexpected ret: {}'.format(ret)) @@ -240,7 +240,7 @@ def run_tests(config): print('\t{}{}'.format('{0: <35}'.format("Failed:"), '{0: >15}'.format(test_str))) for test in failed_tests: - print("\t\tTest:\t\t\t\t{} - {}".format(test[0], str(failure_cause))) + print("\t\tTest:\t\t\t\t{} - {}".format(test[0], str(test[2]))) print("-----------------------------------------------------------------") global setup_time -- 2.47.2