From: Nicki Křížek <nicki@isc.org>
Date: Tue, 24 Jun 2025 11:16:33 +0000 (+0200)
Subject: Make extra_artifacts check optional
X-Git-Tag: v9.21.10~31^2
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=c06dc71cd5a307fcbbb20699f79ff753300e3c92;p=thirdparty%2Fbind9.git

Make extra_artifacts check optional

There is an ongoing debate about the usefulness of the extra artifacts
check. While it might be useful to detect unexpected behaviour in some
tests, it feels extraneous in many cases. This change provides a middle
ground by making the artifact checking optional. This might be
especially useful for writing new tests, since the author gets to decide
whether the check is useful -- and can utilize it, or can skip it for
sake of brevity.
---

diff --git a/bin/tests/system/conftest.py b/bin/tests/system/conftest.py
index 4a0f4e4a652..89bbad7d19d 100644
--- a/bin/tests/system/conftest.py
+++ b/bin/tests/system/conftest.py
@@ -331,7 +331,7 @@ def expected_artifacts(request):
     if test_specific_artifacts:
         return common_artifacts + test_specific_artifacts.args[0]
 
-    return common_artifacts
+    return None
 
 
 @pytest.fixture(scope="module")