commit 0b3d24a7790d436bb611e5e10c4d0c06ea04fd3c
author Simon Glass <sjg@chromium.org> Sun Jul 05 21:41:48 2020 -0600
committer Simon Glass <sjg@chromium.org> Fri Jul 24 19:25:15 2020 -0600
tree 208f3955c96150bd5ccc465e237c7bf08ec44b05
parent ada61f1ee2a4eaa1b29d699b5ba940483171df8a

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>

tools/patman/main.py

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

tools/patman/test_util.py

diff --git a/tools/patman/test_util.py b/tools/patman/test_util.py
index aac58fb..0827488 100644
--- a/tools/patman/test_util.py
+++ b/tools/patman/test_util.py
@@ -123,12 +123,12 @@
     for test, err in result.failures:
         print(err, result.failures)
     if result.skipped:
-        print('%d binman test%s SKIPPED:' %
-              (len(result.skipped), 's' if len(result.skipped) > 1 else ''))
+        print('%d %s test%s SKIPPED:' % (len(result.skipped), toolname,
+            's' if len(result.skipped) > 1 else ''))
         for skip_info in result.skipped:
             print('%s: %s' % (skip_info[0], skip_info[1]))
     if result.errors or result.failures:
-        print('binman tests FAILED')
+        print('%s tests FAILED' % toolname)
         return 1
     return 0