summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMike Frysinger <vapier@google.com>2021-03-11 00:07:24 -0500
committerMike Frysinger <vapier@google.com>2021-04-01 14:56:57 +0000
commitbfbcfd904592a2f61e11a12eec281102230281d6 (patch)
treefa991f993402fd33985645e63dcb8d3ed31e8790
parent74317d3b019c290e7e9b6a255de528a2568af37f (diff)
downloadgit-repo-bfbcfd904592a2f61e11a12eec281102230281d6.tar.gz
run_tests: fix exit code handling
We need to pass back an int, not a CompletedProcess object. Switch to check=False so we don't throw an exception on failure -- we're already showing pytest's stderr, and will return the non-zero status. Change-Id: Ib0d3862a09a3963f25025f39a8e34419cf2a54df Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/299624 Reviewed-by: Michael Mortensen <mmortensen@google.com> Tested-by: Mike Frysinger <vapier@google.com>
-rwxr-xr-xrun_tests2
1 files changed, 1 insertions, 1 deletions
diff --git a/run_tests b/run_tests
index 9f19e2e2..07cd98db 100755
--- a/run_tests
+++ b/run_tests
@@ -50,7 +50,7 @@ def main(argv):
50 os.environ['PYTHONPATH'] = pythonpath 50 os.environ['PYTHONPATH'] = pythonpath
51 51
52 pytest = find_pytest() 52 pytest = find_pytest()
53 return subprocess.run([pytest] + argv, check=True) 53 return subprocess.run([pytest] + argv, check=False).returncode
54 54
55 55
56if __name__ == '__main__': 56if __name__ == '__main__':