patman: Use test_util to show test results

This handles skipped tests correctly, so use it instead of the existing
code.

Signed-off-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
Simon Glass 2020-07-05 21:41:48 -06:00
parent ada61f1ee2
commit 0b3d24a779
2 changed files with 5 additions and 9 deletions

View file

@ -25,6 +25,7 @@ from patman import patchstream
from patman import project from patman import project
from patman import settings from patman import settings
from patman import terminal from patman import terminal
from patman import test_util
from patman import test_checkpatch from patman import test_checkpatch
@ -101,12 +102,7 @@ elif options.test:
suite = doctest.DocTestSuite(module) suite = doctest.DocTestSuite(module)
suite.run(result) suite.run(result)
# TODO: Surely we can just 'print' result? sys.exit(test_util.ReportResult('patman', None, result))
print(result)
for test, err in result.errors:
print(err)
for test, err in result.failures:
print(err)
# Called from git with a patch filename as argument # Called from git with a patch filename as argument
# Printout a list of additional CC recipients for this patch # Printout a list of additional CC recipients for this patch

View file

@ -123,12 +123,12 @@ def ReportResult(toolname:str, test_name: str, result: unittest.TestResult):
for test, err in result.failures: for test, err in result.failures:
print(err, result.failures) print(err, result.failures)
if result.skipped: if result.skipped:
print('%d binman test%s SKIPPED:' % print('%d %s test%s SKIPPED:' % (len(result.skipped), toolname,
(len(result.skipped), 's' if len(result.skipped) > 1 else '')) 's' if len(result.skipped) > 1 else ''))
for skip_info in result.skipped: for skip_info in result.skipped:
print('%s: %s' % (skip_info[0], skip_info[1])) print('%s: %s' % (skip_info[0], skip_info[1]))
if result.errors or result.failures: if result.errors or result.failures:
print('binman tests FAILED') print('%s tests FAILED' % toolname)
return 1 return 1
return 0 return 0