LineageOS/android_development
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge "Compare results across ABIs"

Browse Source
This commit is contained in:
Treehugger Robot
2018-06-07 17:35:46 +00:00
committed by Gerrit Code Review
parent 7cc827ff88 21d011cc08
commit 02e55c34d8
1 changed files with 24 additions and 26 deletions
Download Patch File Download Diff File Expand all files Collapse all files

50
tools/compare_failed_tests/compare_failed_tests.py
View File

@@ -20,7 +20,9 @@ ATTRS_TO_SHOW = ['Result::Build.build_model',
'Result.suite_name', 'Result.suite_name',
'Result.suite_plan', 'Result.suite_plan',
'Result.suite_build_number', 'Result.suite_build_number',
'Result.start_display'] 'Result.start_display',
'Result::Build.build_abis_32',
'Result::Build.build_abis_64',]
def parse_attrib_path(attrib_path): def parse_attrib_path(attrib_path):
@@ -90,7 +92,7 @@ def get_result(test_result, module_name, testcase_name, test_name):
if test_name not in tests: if test_name not in tests:
return NO_DATA return NO_DATA
return tests[test_name] return ', '.join([x + ': ' + y for x, y in tests[test_name].items()])
def read_test_result_xml(test_result_path): def read_test_result_xml(test_result_path):
@@ -106,37 +108,38 @@ def read_test_result_xml(test_result_path):
test_result['modules'] = modules test_result['modules'] = modules
for module in root.iter('Module'): for module in root.iter('Module'):
module_name = '|'.join([module.attrib['name'], module.attrib['abi']]) abi = module.attrib['abi']
if module_name in modules: module_name = module.attrib['name']
print 'WARNING: Duplicate module: ' + module_name
testcases = collections.OrderedDict() if not module_name in modules:
modules[module_name] = testcases modules[module_name] = collections.OrderedDict()
testcases = modules[module_name]
for testcase in module.iter('TestCase'): for testcase in module.iter('TestCase'):
testcase_name = testcase.attrib['name'] testcase_name = testcase.attrib['name']
if testcase_name in testcases: if not testcase_name in testcases:
print 'WARNING: Duplicate testcase: ' + testcase_name testcases[testcase_name] = collections.OrderedDict()
tests = collections.OrderedDict() tests = testcases[testcase_name]
testcases[testcase_name] = tests
for test in testcase.iter('Test'): for test in testcase.iter('Test'):
test_name = test.attrib['name'] test_name = test.attrib['name']
if test_name in tests: if not test_name in tests:
print 'WARNING: Duplicate test: ' + test_name tests[test_name] = collections.OrderedDict()
result = test.attrib['result'] if abi in tests[test_name]:
tests[test_name] = result print '[WARNING] duplicated test:', test_name
tests[test_name][abi] = test.attrib['result']
return test_result return test_result
def compare_failed_tests(test_result_a, test_result_b, def compare_failed_tests(test_result_a, test_result_b, csvfile):
csvfile, only_failed_both):
"""Do the comparison. """Do the comparison.
Given two test result dicts (A and B), list all failed test in A and display Given two test result dicts (A and B), list all failed test in A and display
@@ -146,7 +149,6 @@ def compare_failed_tests(test_result_a, test_result_b,
test_result_a: the dict returned from read_test_result(test_result_a.xml) test_result_a: the dict returned from read_test_result(test_result_a.xml)
test_result_b: the dict returned from read_test_result(test_result_b.xml) test_result_b: the dict returned from read_test_result(test_result_b.xml)
csvfile: a opened file csvfile: a opened file
only_failed_both: only display tests those failed in both test results
Returns: Returns:
string: diff report, summary string: diff report, summary
@@ -166,13 +168,12 @@ def compare_failed_tests(test_result_a, test_result_b,
testcase_sub_summary = '' testcase_sub_summary = ''
for test_name, result in tests.iteritems(): for test_name, result in tests.iteritems():
if result == FAIL: if FAIL in result.values():
result_b = get_result( result_b = get_result(
test_result_b, module_name, testcase_name, test_name) test_result_b, module_name, testcase_name, test_name)
if not only_failed_both or result_b == FAIL: testcase_sub_summary += ' ' + test_name + ': ' + result_b + '\n'
testcase_sub_summary += ' ' + test_name + ': ' + result_b + '\n' writer.writerow([module_name, testcase_name, test_name, result_b])
writer.writerow([module_name, testcase_name, test_name, result_b])
if testcase_sub_summary: if testcase_sub_summary:
module_sub_summary = ' ' + testcase_name + '\n' + testcase_sub_summary module_sub_summary = ' ' + testcase_name + '\n' + testcase_sub_summary
@@ -189,8 +190,6 @@ def main():
parser.add_argument('test_result_a', help='path to first test_result.xml') parser.add_argument('test_result_a', help='path to first test_result.xml')
parser.add_argument('test_result_b', help='path to second test_result.xml') parser.add_argument('test_result_b', help='path to second test_result.xml')
parser.add_argument('--csv', default='diff.csv', help='path to csv output') parser.add_argument('--csv', default='diff.csv', help='path to csv output')
parser.add_argument('--only-failed-both', action='store_true',
help='only list tests failed in both test_result.xml')
args = parser.parse_args() args = parser.parse_args()
@@ -200,8 +199,7 @@ def main():
print_test_infos(test_result_a, test_result_b) print_test_infos(test_result_a, test_result_b)
with open(args.csv, 'w') as csvfile: with open(args.csv, 'w') as csvfile:
summary = compare_failed_tests(test_result_a, test_result_b, csvfile, summary = compare_failed_tests(test_result_a, test_result_b, csvfile)
args.only_failed_both)
print summary print summary