From c06dc71cd5a307fcbbb20699f79ff753300e3c92 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Nicki=20K=C5=99=C3=AD=C5=BEek?= Date: Tue, 24 Jun 2025 13:16:33 +0200 Subject: [PATCH] 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. --- bin/tests/system/conftest.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/bin/tests/system/conftest.py b/bin/tests/system/conftest.py index 4a0f4e4a65..89bbad7d19 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")