diff options
author | Gavin Mak <gavinmak@google.com> | 2023-01-25 21:19:54 +0000 |
---|---|---|
committer | Gavin Mak <gavinmak@google.com> | 2023-02-13 19:19:38 +0000 |
commit | 3ed84466f4e8548a7667e81d99ad7161bd7d9489 (patch) | |
tree | 458457a9d34652792a746f074963afffc40996e9 | |
parent | 48067714ec2aa41edbccfee9b2f3fddcba0fe3d6 (diff) | |
download | git-repo-3ed84466f4e8548a7667e81d99ad7161bd7d9489.tar.gz |
tests: Rework run_tests to use pytest directly and add vpython3 file
Remove logic to handle importing the right version of pytest.
'./run_tests' still works but this allows presubmit builders to test
using 'vpython3 ./run_tests'.
Google-Bug-Id: b/266734831
Change-Id: I6a543c1f4b5b4449e723095b4a70e5228b1ccd34
Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/356717
Tested-by: Gavin Mak <gavinmak@google.com>
Reviewed-by: Josip Sokcevic <sokcevic@google.com>
Reviewed-by: Mike Frysinger <vapier@google.com>
-rwxr-xr-x | run_tests | 42 | ||||
-rw-r--r-- | run_tests.vpython3 | 61 |
2 files changed, 63 insertions, 40 deletions
@@ -15,46 +15,8 @@ | |||
15 | 15 | ||
16 | """Wrapper to run pytest with the right settings.""" | 16 | """Wrapper to run pytest with the right settings.""" |
17 | 17 | ||
18 | import os | ||
19 | import shutil | ||
20 | import subprocess | ||
21 | import sys | 18 | import sys |
22 | 19 | import pytest | |
23 | |||
24 | def find_pytest(): | ||
25 | """Try to locate a good version of pytest.""" | ||
26 | # If we're in a virtualenv, assume that it's provided the right pytest. | ||
27 | if 'VIRTUAL_ENV' in os.environ: | ||
28 | return 'pytest' | ||
29 | |||
30 | # Use the Python 3 version if available. | ||
31 | ret = shutil.which('pytest-3') | ||
32 | if ret: | ||
33 | return ret | ||
34 | |||
35 | # Hopefully this is a Python 3 version. | ||
36 | ret = shutil.which('pytest') | ||
37 | if ret: | ||
38 | return ret | ||
39 | |||
40 | print('%s: unable to find pytest.' % (__file__,), file=sys.stderr) | ||
41 | print('%s: Try installing: sudo apt-get install python-pytest' % (__file__,), | ||
42 | file=sys.stderr) | ||
43 | |||
44 | |||
45 | def main(argv): | ||
46 | """The main entry.""" | ||
47 | # Add the repo tree to PYTHONPATH as the tests expect to be able to import | ||
48 | # modules directly. | ||
49 | pythonpath = os.path.dirname(os.path.realpath(__file__)) | ||
50 | oldpythonpath = os.environ.get('PYTHONPATH', None) | ||
51 | if oldpythonpath is not None: | ||
52 | pythonpath += os.pathsep + oldpythonpath | ||
53 | os.environ['PYTHONPATH'] = pythonpath | ||
54 | |||
55 | pytest = find_pytest() | ||
56 | return subprocess.run([pytest] + argv, check=False).returncode | ||
57 | |||
58 | 20 | ||
59 | if __name__ == '__main__': | 21 | if __name__ == '__main__': |
60 | sys.exit(main(sys.argv[1:])) | 22 | sys.exit(pytest.main(sys.argv[1:])) |
diff --git a/run_tests.vpython3 b/run_tests.vpython3 new file mode 100644 index 00000000..d0e821dd --- /dev/null +++ b/run_tests.vpython3 | |||
@@ -0,0 +1,61 @@ | |||
1 | # This is a vpython "spec" file. | ||
2 | # | ||
3 | # Read more about `vpython` and how to modify this file here: | ||
4 | # https://chromium.googlesource.com/infra/infra/+/main/doc/users/vpython.md | ||
5 | # List of available wheels: | ||
6 | # https://chromium.googlesource.com/infra/infra/+/main/infra/tools/dockerbuild/wheels.md | ||
7 | |||
8 | python_version: "3.8" | ||
9 | |||
10 | wheel: < | ||
11 | name: "infra/python/wheels/pytest-py3" | ||
12 | version: "version:6.2.2" | ||
13 | > | ||
14 | |||
15 | # Required by pytest==6.2.2 | ||
16 | wheel: < | ||
17 | name: "infra/python/wheels/py-py2_py3" | ||
18 | version: "version:1.10.0" | ||
19 | > | ||
20 | |||
21 | # Required by pytest==6.2.2 | ||
22 | wheel: < | ||
23 | name: "infra/python/wheels/iniconfig-py3" | ||
24 | version: "version:1.1.1" | ||
25 | > | ||
26 | |||
27 | # Required by pytest==6.2.2 | ||
28 | wheel: < | ||
29 | name: "infra/python/wheels/packaging-py2_py3" | ||
30 | version: "version:16.8" | ||
31 | > | ||
32 | |||
33 | # Required by pytest==6.2.2 | ||
34 | wheel: < | ||
35 | name: "infra/python/wheels/pluggy-py3" | ||
36 | version: "version:0.13.1" | ||
37 | > | ||
38 | |||
39 | # Required by pytest==6.2.2 | ||
40 | wheel: < | ||
41 | name: "infra/python/wheels/toml-py3" | ||
42 | version: "version:0.10.1" | ||
43 | > | ||
44 | |||
45 | # Required by pytest==6.2.2 | ||
46 | wheel: < | ||
47 | name: "infra/python/wheels/pyparsing-py3" | ||
48 | version: "version:3.0.7" | ||
49 | > | ||
50 | |||
51 | # Required by pytest==6.2.2 | ||
52 | wheel: < | ||
53 | name: "infra/python/wheels/attrs-py2_py3" | ||
54 | version: "version:21.4.0" | ||
55 | > | ||
56 | |||
57 | # Required by packaging==16.8 | ||
58 | wheel: < | ||
59 | name: "infra/python/wheels/six-py2_py3" | ||
60 | version: "version:1.16.0" | ||
61 | > | ||