Skip to content

Commit 8561643

Browse files
committed
declare proxy features as a fixture
this allows to cache the result and makes re-using the data easier
1 parent 920639e commit 8561643

1 file changed

Lines changed: 7 additions & 8 deletions

File tree

tests/foreman_proxy_test.py

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,8 @@
66
FOREMAN_PROXY_PORT = 8443
77

88

9-
def get_proxy_v2_features(server, certificates, server_fqdn):
9+
@pytest.fixture(scope="module")
10+
def proxy_v2_features(server, certificates, server_fqdn):
1011
cmd = server.run(
1112
f"curl --cacert {certificates['server_ca_certificate']} "
1213
f"--cert {certificates['client_certificate']} "
@@ -56,17 +57,15 @@ def test_foreman_proxy_client_auth_to_foreman(server, certificates, server_fqdn)
5657

5758

5859
@pytest.mark.feature('bmc')
59-
def test_bmc_capabilities(server, certificates, server_fqdn):
60-
features = get_proxy_v2_features(server, certificates, server_fqdn)
61-
assert 'bmc' in features
62-
capabilities = features['bmc'].get('capabilities', [])
60+
def test_bmc_capabilities(proxy_v2_features):
61+
assert 'bmc' in proxy_v2_features
62+
capabilities = proxy_v2_features['bmc'].get('capabilities', [])
6363
assert 'ipmitool' in capabilities
6464
assert 'freeipmi' in capabilities
6565
assert 'redfish' in capabilities
6666

6767

6868
@pytest.mark.feature('bmc')
69-
def test_bmc_default_provider(server, certificates, server_fqdn):
70-
features = get_proxy_v2_features(server, certificates, server_fqdn)
71-
settings = features['bmc'].get('settings', {})
69+
def test_bmc_default_provider(proxy_v2_features):
70+
settings = proxy_v2_features['bmc'].get('settings', {})
7271
assert settings.get('bmc_default_provider') == 'ipmitool'

0 commit comments

Comments
 (0)