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.20.11~17^2
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=da7cab2243820e9d695c513288b122176bce9c65;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.

(cherry picked from commit c06dc71cd5a307fcbbb20699f79ff753300e3c92)
---

diff --git a/bin/tests/system/conftest.py b/bin/tests/system/conftest.py
index 0abc30df869..a68e4f65c31 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")