From 58268151704246a81ec8dae46c26346023057554 Mon Sep 17 00:00:00 2001 From: Richard Purdie Date: Wed, 27 Feb 2019 17:05:29 +0000 Subject: resulttool/report: Ensure test suites with no results show up on the report ptest suites with no results don't show up on the reports even though we have a duration for them. Fix this so the fact they report no tests is visible. Signed-off-by: Richard Purdie --- scripts/lib/resulttool/report.py | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) (limited to 'scripts') diff --git a/scripts/lib/resulttool/report.py b/scripts/lib/resulttool/report.py index 5ffe262f89..ff1b32c770 100644 --- a/scripts/lib/resulttool/report.py +++ b/scripts/lib/resulttool/report.py @@ -30,6 +30,14 @@ class ResultsTextReport(object): def handle_ptest_result(self, k, status, result): if k == 'ptestresult.sections': + # Ensure tests without any test results still show up on the report + for suite in result['ptestresult.sections']: + if suite not in self.ptests: + self.ptests[suite] = {'passed': 0, 'failed': 0, 'skipped': 0, 'duration' : '-', 'failed_testcases': []} + if 'duration' in result['ptestresult.sections'][suite]: + self.ptests[suite]['duration'] = result['ptestresult.sections'][suite]['duration'] + if 'timeout' in result['ptestresult.sections'][suite]: + self.ptests[suite]['duration'] += " T" return try: _, suite, test = k.split(".", 2) @@ -48,11 +56,6 @@ class ResultsTextReport(object): for tk in self.result_types: if status in self.result_types[tk]: self.ptests[suite][tk] += 1 - if 'ptestresult.sections' in result and suite in result['ptestresult.sections']: - if 'duration' in result['ptestresult.sections'][suite]: - self.ptests[suite]['duration'] = result['ptestresult.sections'][suite]['duration'] - if 'timeout' in result['ptestresult.sections'][suite]: - self.ptests[suite]['duration'] += " T" def get_aggregated_test_result(self, logger, testresult): test_count_report = {'passed': 0, 'failed': 0, 'skipped': 0, 'failed_testcases': []} -- cgit 1.2.3-korg