diff options
| author | Mike Frysinger <vapier@google.com> | 2021-03-11 00:07:24 -0500 |
|---|---|---|
| committer | Mike Frysinger <vapier@google.com> | 2021-04-01 14:56:57 +0000 |
| commit | bfbcfd904592a2f61e11a12eec281102230281d6 (patch) | |
| tree | fa991f993402fd33985645e63dcb8d3ed31e8790 | |
| parent | 74317d3b019c290e7e9b6a255de528a2568af37f (diff) | |
| download | git-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-x | run_tests | 2 |
1 files changed, 1 insertions, 1 deletions
| @@ -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 | ||
| 56 | if __name__ == '__main__': | 56 | if __name__ == '__main__': |
