From: Jon Mason Date: Sun, 2 Jun 2019 18:29:13 +0000 (-0400) Subject: resulttool: Prevent multiple results for the same test X-Git-Tag: uninative-2.7~440 X-Git-Url: https://code.ossystems.io/gitweb?a=commitdiff_plain;h=caeaa0648dff64c56a33f52e45e82bfab6719e3e;p=openembedded-core.git resulttool: Prevent multiple results for the same test Currently, if a test occurs multiple times over different series, the code will sum these. This can lead to confusion over the desired results. Change the code to report the redundant tests and skip adding an additional values. Signed-off-by: Jon Mason Signed-off-by: Ross Burton --- diff --git a/scripts/lib/resulttool/report.py b/scripts/lib/resulttool/report.py index a48c59f632..f706280aa7 100644 --- a/scripts/lib/resulttool/report.py +++ b/scripts/lib/resulttool/report.py @@ -203,8 +203,22 @@ class ResultsTextReport(object): testresults = resultutils.load_resultsdata(source_dir) for testsuite in testresults: for resultid in testresults[testsuite]: + skip = False result = testresults[testsuite][resultid] machine = result['configuration']['MACHINE'] + + # Check to see if there is already results for these kinds of tests for the machine + for key in result['result'].keys(): + testtype = str(key).split('.')[0] + if ((machine in self.ptests and testtype == "ptestresult" and self.ptests[machine]) or + (machine in self.ltptests and testtype == "ltpiresult" and self.ltptests[machine]) or + (machine in self.ltpposixtests and testtype == "ltpposixresult" and self.ltpposixtests[machine])): + print("Already have test results for %s on %s, skipping %s" %(str(key).split('.')[0], machine, resultid)) + skip = True + break + if skip: + break + test_count_report = self.get_aggregated_test_result(logger, result, machine) test_count_report['machine'] = machine test_count_report['testseries'] = result['configuration']['TESTSERIES']