facebook/sapling
1
1
Fork 0
mirror of https://github.com/facebook/sapling.git synced 2024-10-07 15:27:13 +03:00
Code Issues Projects Releases Wiki Activity

run-tests: use return values instead of sys.exit

Browse Source
This commit is contained in:
Gregory Szorc 2014-03-24 21:41:56 -07:00
parent 2ae9bd8d6c
commit f33d6c4e65
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
tests/run-tests.py
View File

@ -1228,7 +1228,8 @@ def main(args):
# Meaning of tmpdir has changed since 1.3: we used to create # Meaning of tmpdir has changed since 1.3: we used to create
# HGTMP inside tmpdir; now HGTMP is tmpdir. So fail if # HGTMP inside tmpdir; now HGTMP is tmpdir. So fail if
# tmpdir already exists. # tmpdir already exists.
sys.exit("error: temp dir %r already exists" % tmpdir) print "error: temp dir %r already exists" % tmpdir
return 1
# Automatically removing tmpdir sounds convenient, but could # Automatically removing tmpdir sounds convenient, but could
# really annoy anyone in the habit of using "--tmpdir=/tmp" # really annoy anyone in the habit of using "--tmpdir=/tmp"
@ -1293,10 +1294,10 @@ def main(args):
vlog("# Using", IMPL_PATH, os.environ[IMPL_PATH]) vlog("# Using", IMPL_PATH, os.environ[IMPL_PATH])
try: try:
sys.exit(runtests(options, tests) or 0) return runtests(options, tests) or 0
finally: finally:
time.sleep(.1) time.sleep(.1)
cleanup(options) cleanup(options)
if __name__ == '__main__': if __name__ == '__main__':
main(sys.argv[1:]) sys.exit(main(sys.argv[1:]))