[oe-commits] [openembedded-core] 05/21: resulttool: Prevent multiple results for the same test

[git at git.openembedded.org]

This is an automated email from the git hooks/post-receive script.

rpurdie pushed a commit to branch master
in repository openembedded-core.

commit caeaa0648dff64c56a33f52e45e82bfab6719e3e
Author: Jon Mason <jdmason at kudzu.us>
AuthorDate: Sun Jun 2 14:29:13 2019 -0400

    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 <jdmason at kudzu.us>
    Signed-off-by: Ross Burton <ross.burton at intel.com>
---
 scripts/lib/resulttool/report.py | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/scripts/lib/resulttool/report.py b/scripts/lib/resulttool/report.py
index a48c59f..f706280 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']

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.