gergely/taskwarrior-2.x
1
0
Fork 0
Code Issues Actions Packages Projects Releases 1 Wiki Activity

problems: Return non-zero code when some tests fail

Browse Source
This commit is contained in:
Tomas Babej
2015-09-11 04:47:49 +02:00
committed by Paul Beckingham
parent cd736a0d70
commit f22322bc18
2 changed files with 11 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
test/problems
View File

@@ -127,3 +127,7 @@ if __name__ == "__main__":
print() print()
print(color(expected_str, "yellow")) print(color(expected_str, "yellow"))
print_category(expected) print_category(expected)
# If we encoutered any failures, return non-zero code
if sum(errors.values()):
sys.exit(1)

10
test/run_all
View File

@@ -177,7 +177,7 @@ class TestRunner(object):
self.tap.flush() self.tap.flush()
log.debug("Calling 'problems --summary' for report") log.debug("Calling 'problems --summary' for report")
call([os.path.abspath("problems"), "--summary", cmd_args.tapfile]) return call([os.path.abspath("problems"), "--summary", cmd_args.tapfile])
def parse_args(): def parse_args():
@@ -197,8 +197,12 @@ def parse_args():
def main(): def main():
runner = TestRunner() runner = TestRunner()
runner.start() runner.start()
# If we're producing summary report, propagate the return code
if not cmd_args.verbose: if not cmd_args.verbose:
runner.show_report() return runner.show_report()
else:
return 0
if __name__ == "__main__": if __name__ == "__main__":
@@ -222,7 +226,7 @@ if __name__ == "__main__":
log.debug("Parsed commandline arguments: %s", cmd_args) log.debug("Parsed commandline arguments: %s", cmd_args)
try: try:
main() sys.exit(main())
except Exception as e: except Exception as e:
log.exception(e) log.exception(e)
sys.exit(1) sys.exit(1)