Skip to content
Snippets Groups Projects
Commit 21e7dfc7 authored by Nathaniel Manista's avatar Nathaniel Manista
Browse files

Ignore the number of failed skipped tests

jobset.run was changed on 16 October 2015 in e4c35610
to return the number of failures and the result set. On 20 October 2016
in 5c691c63 jobset.run was changed to
allow jobs to be skipped, but the introduced code path only returned
the result set, giving jobset.run inconsistent return value semantics.
On 24 October 2016 7e9bd6ca introduced
a use of jobset.run's "skip_jobs" return value semantics, and on 8
March 2017 d159af31 resolved
jobset.run's inconsistent return value semantics but failed to correct
run_tests_matrix's use of the inconsistent "skip_jobs" return value
semantics.
parent 0b7bd20d
No related branches found
No related tags found
No related merge requests found
...@@ -415,8 +415,8 @@ if __name__ == "__main__": ...@@ -415,8 +415,8 @@ if __name__ == "__main__":
maxjobs=args.jobs) maxjobs=args.jobs)
# Merge skipped tests into results to show skipped tests on report.xml # Merge skipped tests into results to show skipped tests on report.xml
if skipped_jobs: if skipped_jobs:
skipped_results = jobset.run(skipped_jobs, ignored_num_skipped_failures, skipped_results = jobset.run(
skip_jobs=True) skipped_jobs, skip_jobs=True)
resultset.update(skipped_results) resultset.update(skipped_results)
report_utils.render_junit_xml_report(resultset, 'report_%s' % _REPORT_SUFFIX, report_utils.render_junit_xml_report(resultset, 'report_%s' % _REPORT_SUFFIX,
suite_name='aggregate_tests') suite_name='aggregate_tests')
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment