Message ID | 20171113045231.15911-26-sjg@chromium.org |
---|---|
State | Accepted |
Commit | 9677faa34ee81c7abb4c08b0dc4ce4aace5473fc |
Delegated to: | Simon Glass |
Headers | show |
Series | test: Include Python tools in test coverage | expand |
Return exit code 1 when test fail so that callers can detect this.
Signed-off-by: Simon Glass <sjg@chromium.org>
---
tools/binman/binman.py | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
Applied to u-boot-dm thanks!
diff --git a/tools/binman/binman.py b/tools/binman/binman.py index 7ad4d3030b9..3ccf25f1f88 100755 --- a/tools/binman/binman.py +++ b/tools/binman/binman.py @@ -58,7 +58,11 @@ def RunTests(): for test, err in result.errors: print test.id(), err for test, err in result.failures: - print err + print err, result.failures + if result.errors or result.failures: + print 'binman tests FAILED' + return 1 + return 0 def RunTestCoverage(): """Run the tests and check that we get 100% coverage""" @@ -106,7 +110,7 @@ def RunBinman(options, args): sys.tracebacklimit = 0 if options.test: - RunTests() + ret_code = RunTests() elif options.test_coverage: RunTestCoverage()
Return exit code 1 when test fail so that callers can detect this. Signed-off-by: Simon Glass <sjg@chromium.org> --- tools/binman/binman.py | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-)