mirror of
https://git.yoctoproject.org/poky
synced 2026-07-17 04:07:06 +00:00
Richard Purdie
10b6fbfb8a
This is a combined patch of the various tweaks and improvements I made to resulttool: * Avoid subprocess.run() as its a python 3.6 feature and we have autobuilder workers with 3.5. * Avoid python keywords as variable names * Simplify dict accesses using .get() * Rename resultsutils -> resultutils to match the resultstool -> resulttool rename * Formalised the handling of "file_name" to "TESTSERIES" which the code will now add into the json configuration data if its not present, based on the directory name. * When we don't have failed test cases, print something saying so instead of an empty table * Tweak the table headers in the report to be more readable (reference "Test Series" instead if file_id and ID instead of results_id) * Improve/simplify the max string length handling * Merge the counts and percentage data into one table in the report since printing two reports of the same data confuses the user * Removed the confusing header in the regression report * Show matches, then regressions, then unmatched runs in the regression report, also remove chatting unneeded output * Try harder to "pair" up matching configurations to reduce noise in the regressions report * Abstracted the "mapping" table concept used to pairing in the regression code to general code in resultutils * Created multiple mappings for results analysis, results storage and 'flattening' results data in a merge * Simplify the merge command to take a source and a destination, letting the destination be a directory or a file, removing the need for an output directory parameter * Add the 'IMAGE_PKGTYPE' and 'DISTRO' config options to the regression mappings * Have the store command place the testresults files in a layout from the mapping, making commits into the git repo for results storage more useful for simple comparison purposes * Set the oe-git-archive tag format appropriately for oeqa results storage (and simplify the commit messages closer to their defaults) * Fix oe-git-archive to use the commit/branch data from the results file * Cleaned up the command option help to match other changes * Follow the model of git branch/tag processing used by oe-build-perf-report and use that to read the data using git show to avoid branch change * Add ptest summary to the report command * Update the tests to match the above changes (From OE-Core rev: b4513e75f746a0989b09ee53cb85e489d41e5783) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
95 lines
6.4 KiB
Python
95 lines
6.4 KiB
Python
import os
|
|
import sys
|
|
basepath = os.path.abspath(os.path.dirname(__file__) + '/../../../../../')
|
|
lib_path = basepath + '/scripts/lib'
|
|
sys.path = sys.path + [lib_path]
|
|
from resulttool.report import ResultsTextReport
|
|
from resulttool import regression as regression
|
|
from resulttool import resultutils as resultutils
|
|
from oeqa.selftest.case import OESelftestTestCase
|
|
|
|
class ResultToolTests(OESelftestTestCase):
|
|
base_results_data = {'base_result1': {'configuration': {"TEST_TYPE": "runtime",
|
|
"TESTSERIES": "series1",
|
|
"IMAGE_BASENAME": "image",
|
|
"IMAGE_PKGTYPE": "ipk",
|
|
"DISTRO": "mydistro",
|
|
"MACHINE": "qemux86"},
|
|
'result': {}},
|
|
'base_result2': {'configuration': {"TEST_TYPE": "runtime",
|
|
"TESTSERIES": "series1",
|
|
"IMAGE_BASENAME": "image",
|
|
"IMAGE_PKGTYPE": "ipk",
|
|
"DISTRO": "mydistro",
|
|
"MACHINE": "qemux86-64"},
|
|
'result': {}}}
|
|
target_results_data = {'target_result1': {'configuration': {"TEST_TYPE": "runtime",
|
|
"TESTSERIES": "series1",
|
|
"IMAGE_BASENAME": "image",
|
|
"IMAGE_PKGTYPE": "ipk",
|
|
"DISTRO": "mydistro",
|
|
"MACHINE": "qemux86"},
|
|
'result': {}},
|
|
'target_result2': {'configuration': {"TEST_TYPE": "runtime",
|
|
"TESTSERIES": "series1",
|
|
"IMAGE_BASENAME": "image",
|
|
"IMAGE_PKGTYPE": "ipk",
|
|
"DISTRO": "mydistro",
|
|
"MACHINE": "qemux86"},
|
|
'result': {}},
|
|
'target_result3': {'configuration': {"TEST_TYPE": "runtime",
|
|
"TESTSERIES": "series1",
|
|
"IMAGE_BASENAME": "image",
|
|
"IMAGE_PKGTYPE": "ipk",
|
|
"DISTRO": "mydistro",
|
|
"MACHINE": "qemux86-64"},
|
|
'result': {}}}
|
|
|
|
def test_report_can_aggregate_test_result(self):
|
|
result_data = {'result': {'test1': {'status': 'PASSED'},
|
|
'test2': {'status': 'PASSED'},
|
|
'test3': {'status': 'FAILED'},
|
|
'test4': {'status': 'ERROR'},
|
|
'test5': {'status': 'SKIPPED'}}}
|
|
report = ResultsTextReport()
|
|
result_report = report.get_aggregated_test_result(None, result_data)
|
|
self.assertTrue(result_report['passed'] == 2, msg="Passed count not correct:%s" % result_report['passed'])
|
|
self.assertTrue(result_report['failed'] == 2, msg="Failed count not correct:%s" % result_report['failed'])
|
|
self.assertTrue(result_report['skipped'] == 1, msg="Skipped count not correct:%s" % result_report['skipped'])
|
|
|
|
def test_regression_can_get_regression_base_target_pair(self):
|
|
|
|
results = {}
|
|
resultutils.append_resultsdata(results, ResultToolTests.base_results_data)
|
|
resultutils.append_resultsdata(results, ResultToolTests.target_results_data)
|
|
self.assertTrue('target_result1' in results['runtime/mydistro/qemux86/image'], msg="Pair not correct:%s" % results)
|
|
self.assertTrue('target_result3' in results['runtime/mydistro/qemux86-64/image'], msg="Pair not correct:%s" % results)
|
|
|
|
def test_regrresion_can_get_regression_result(self):
|
|
base_result_data = {'result': {'test1': {'status': 'PASSED'},
|
|
'test2': {'status': 'PASSED'},
|
|
'test3': {'status': 'FAILED'},
|
|
'test4': {'status': 'ERROR'},
|
|
'test5': {'status': 'SKIPPED'}}}
|
|
target_result_data = {'result': {'test1': {'status': 'PASSED'},
|
|
'test2': {'status': 'FAILED'},
|
|
'test3': {'status': 'PASSED'},
|
|
'test4': {'status': 'ERROR'},
|
|
'test5': {'status': 'SKIPPED'}}}
|
|
result, text = regression.compare_result(self.logger, "BaseTestRunName", "TargetTestRunName", base_result_data, target_result_data)
|
|
self.assertTrue(result['test2']['base'] == 'PASSED',
|
|
msg="regression not correct:%s" % result['test2']['base'])
|
|
self.assertTrue(result['test2']['target'] == 'FAILED',
|
|
msg="regression not correct:%s" % result['test2']['target'])
|
|
self.assertTrue(result['test3']['base'] == 'FAILED',
|
|
msg="regression not correct:%s" % result['test3']['base'])
|
|
self.assertTrue(result['test3']['target'] == 'PASSED',
|
|
msg="regression not correct:%s" % result['test3']['target'])
|
|
|
|
def test_merge_can_merged_results(self):
|
|
results = {}
|
|
resultutils.append_resultsdata(results, ResultToolTests.base_results_data, configmap=resultutils.flatten_map)
|
|
resultutils.append_resultsdata(results, ResultToolTests.target_results_data, configmap=resultutils.flatten_map)
|
|
self.assertEqual(len(results[''].keys()), 5, msg="Flattened results not correct %s" % str(results))
|
|
|