From 1812e7f400ccd6fca24fd9435a773c1a5a40ebc4 Mon Sep 17 00:00:00 2001
From: Evgeni Golov <evgeni@golov.de>
Date: Wed, 20 May 2026 09:19:04 +0200
Subject: [PATCH] declare proxy features as a fixture

this allows to cache the result and makes re-using the data easier
---
 tests/foreman_proxy_test.py | 15 +++++++--------
 1 file changed, 7 insertions(+), 8 deletions(-)

diff --git a/tests/foreman_proxy_test.py b/tests/foreman_proxy_test.py
index 4d5032b81..71ad8cbc0 100644
--- a/tests/foreman_proxy_test.py
+++ b/tests/foreman_proxy_test.py
@@ -6,7 +6,8 @@
 FOREMAN_PROXY_PORT = 8443
 
 
-def get_proxy_v2_features(server, certificates, server_fqdn):
+@pytest.fixture(scope="module")
+def proxy_v2_features(server, certificates, server_fqdn):
     cmd = server.run(
         f"curl --cacert {certificates['server_ca_certificate']} "
         f"--cert {certificates['client_certificate']} "
@@ -56,17 +57,15 @@ def test_foreman_proxy_client_auth_to_foreman(server, certificates, server_fqdn)
 
 
 @pytest.mark.feature('bmc')
-def test_bmc_capabilities(server, certificates, server_fqdn):
-    features = get_proxy_v2_features(server, certificates, server_fqdn)
-    assert 'bmc' in features
-    capabilities = features['bmc'].get('capabilities', [])
+def test_bmc_capabilities(proxy_v2_features):
+    assert 'bmc' in proxy_v2_features
+    capabilities = proxy_v2_features['bmc'].get('capabilities', [])
     assert 'ipmitool' in capabilities
     assert 'freeipmi' in capabilities
     assert 'redfish' in capabilities
 
 
 @pytest.mark.feature('bmc')
-def test_bmc_default_provider(server, certificates, server_fqdn):
-    features = get_proxy_v2_features(server, certificates, server_fqdn)
-    settings = features['bmc'].get('settings', {})
+def test_bmc_default_provider(proxy_v2_features):
+    settings = proxy_v2_features['bmc'].get('settings', {})
     assert settings.get('bmc_default_provider') == 'ipmitool'