diff options
| author |   Richard Purdie <richard.purdie@linuxfoundation.org> | 2019-02-27 16:51:56 +0000 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2019-03-25 23:12:07 +0000 |
| commit | 829754c380ca2b614afa23500e53a8c5b850932f (patch) | |
| tree | e1b8ef34eb78cfbcf22d265afcacd6d1339c10f4 | |
| parent | 22a5982dbcb624ffcdbacef92b2a235851214663 (diff) | |
| download | openembedded-core-contrib-829754c380ca2b614afa23500e53a8c5b850932f.tar.gz | |
resulttool/report: Handle missing metadata sections more cleanly
Currently some older results files cause the code to give tracebacks.
Handle these missing sections more cleanly.
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
| -rw-r--r-- | scripts/lib/resulttool/report.py | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/scripts/lib/resulttool/report.py b/scripts/lib/resulttool/report.py index 2f5ea308e2..5ffe262f89 100644 --- a/scripts/lib/resulttool/report.py +++ b/scripts/lib/resulttool/report.py @@ -31,9 +31,12 @@ class ResultsTextReport(object): def handle_ptest_result(self, k, status, result): if k == 'ptestresult.sections': return - _, suite, test = k.split(".", 2) + try: + _, suite, test = k.split(".", 2) + except ValueError: + return # Handle 'glib-2.0' - if suite not in result['ptestresult.sections']: + if 'ptestresult.sections' in result and suite not in result['ptestresult.sections']: try: _, suite, suite1, test = k.split(".", 3) if suite + "." + suite1 in result['ptestresult.sections']: @@ -45,7 +48,7 @@ class ResultsTextReport(object): for tk in self.result_types: if status in self.result_types[tk]: self.ptests[suite][tk] += 1 - if suite in result['ptestresult.sections']: + 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]: |