]> git.ipfire.org Git - thirdparty/bind9.git/commitdiff
Remove prereq.sh support from the system test runner 12335/head
authorNicki Křížek <nicki@isc.org>
Wed, 1 Jul 2026 13:40:07 +0000 (13:40 +0000)
committerNicki Křížek <nicki@isc.org>
Fri, 10 Jul 2026 11:17:41 +0000 (13:17 +0200)
With every prereq.sh converted to pytest markers, the conftest
fixture no longer needs to locate and run a per-directory prereq.sh.
Drop check_prerequisites() and the README entry for the file.

Assisted-by: Claude:claude-opus-4-8
bin/tests/system/README.md
bin/tests/system/conftest.py

index b442d69572d4b5792f6b2932f1a264bf721d48e7..1a1c66e97936955a381c25ddfb654262119b922e 100644 (file)
@@ -226,9 +226,6 @@ ported to python; do not add them to new tests:
 - `setup.sh`: Legacy shell test setup.  New tests use templates and a
   `bootstrap()` function instead.
 
-- `prereq.sh`: Legacy prerequisite check; when it exits non-zero, the test is
-  skipped.  New tests use pytest marks (see `isctest/mark.py`).
-
 ### Module Scope
 
 A module is a python file which contains test functions. Every system
index e0c2a08d660573ec0b6f04d412b86c2476549bff..3339eacd972a90f969bce2964a263cd91aa4323c 100644 (file)
@@ -546,14 +546,6 @@ def system_test(
             isctest.log.error("testsock.pl: exited with code %d", exc.returncode)
             pytest.skip("Network interface aliases not set up.")
 
-    def check_prerequisites():
-        try:
-            isctest.run.shell(f"{system_test_dir}/prereq.sh")
-        except FileNotFoundError:
-            pass  # prereq.sh is optional
-        except subprocess.CalledProcessError:
-            pytest.skip("Prerequisites missing.")
-
     def setup_test():
         template_data = None
         bootstrap_fn = getattr(request.module, "bootstrap", None)
@@ -613,7 +605,6 @@ def system_test(
 
     # Perform checks which may skip this test.
     check_net_interfaces()
-    check_prerequisites()
 
     # Store the fact that this fixture hasn't successfully finished yet.
     # This is checked before temporary directory teardown to decide whether