run_tests: log each command run
This should make it clear to devs what commands are run and which fail
in the CI.
Change-Id: Ie863540cba6de7da933b4f32947ad09edee4aa45
Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/519361
Tested-by: Mike Frysinger <vapier@google.com>
Reviewed-by: Gavin Mak <gavinmak@google.com>
diff --git a/run_tests b/run_tests
index 04f2deb..2e1cddb 100755
--- a/run_tests
+++ b/run_tests
@@ -17,6 +17,7 @@
import functools
import os
+import shlex
import shutil
import subprocess
import sys
@@ -26,6 +27,11 @@
ROOT_DIR = os.path.dirname(os.path.realpath(__file__))
+def log_cmd(cmd: str, argv: list[str]) -> None:
+ """Log a debug message to make history easier to track."""
+ print("+", cmd, shlex.join(argv), file=sys.stderr)
+
+
@functools.lru_cache()
def is_ci() -> bool:
"""Whether we're running in our CI system."""
@@ -37,6 +43,7 @@
if is_ci():
argv = ["-m", "not skip_cq"] + argv
+ log_cmd("pytest", argv)
return subprocess.run(
[sys.executable, "-m", "pytest"] + argv,
check=False,
@@ -49,6 +56,7 @@
if is_ci():
argv = ["-m", "not skip_cq"] + argv
+ log_cmd("[vpython 3.8] pytest", argv)
try:
return subprocess.run(
[
@@ -77,8 +85,10 @@
"release/update-hooks",
"release/update-manpages",
]
+ argv = ["--diff", "--check", ROOT_DIR] + extra_programs
+ log_cmd("black", argv)
return subprocess.run(
- [sys.executable, "-m", "black", "--check", ROOT_DIR] + extra_programs,
+ [sys.executable, "-m", "black"] + argv,
check=False,
cwd=ROOT_DIR,
).returncode
@@ -86,8 +96,10 @@
def run_flake8():
"""Returns the exit code from flake8."""
+ argv = [ROOT_DIR]
+ log_cmd("flake8", argv)
return subprocess.run(
- [sys.executable, "-m", "flake8", ROOT_DIR],
+ [sys.executable, "-m", "flake8"] + argv,
check=False,
cwd=ROOT_DIR,
).returncode
@@ -95,8 +107,10 @@
def run_isort():
"""Returns the exit code from isort."""
+ argv = ["--check", ROOT_DIR]
+ log_cmd("isort", argv)
return subprocess.run(
- [sys.executable, "-m", "isort", "--check", ROOT_DIR],
+ [sys.executable, "-m", "isort"] + argv,
check=False,
cwd=ROOT_DIR,
).returncode
@@ -104,8 +118,10 @@
def run_check_metadata():
"""Returns the exit code from check-metadata."""
+ argv = []
+ log_cmd("release/check-metadata.py", argv)
return subprocess.run(
- [sys.executable, "release/check-metadata.py"],
+ [sys.executable, "release/check-metadata.py"] + argv,
check=False,
cwd=ROOT_DIR,
).returncode
@@ -118,8 +134,10 @@
print("update-manpages: help2man not found; skipping test")
return 0
+ argv = ["--check"]
+ log_cmd("release/update-manpages", argv)
return subprocess.run(
- [sys.executable, "release/update-manpages", "--check"],
+ [sys.executable, "release/update-manpages"] + argv,
check=False,
cwd=ROOT_DIR,
).returncode
