From 595e28933290ff20f04e5acf3965ca560124f8de Mon Sep 17 00:00:00 2001 From: Gaurav Talreja Date: Wed, 31 Jan 2024 15:18:59 +0530 Subject: [PATCH] Fix parametrization in test_positive_reboot_all_pxe_hosts Signed-off-by: Gaurav Talreja --- pytest_fixtures/component/provision_pxe.py | 5 +---- tests/foreman/api/test_discoveredhost.py | 2 -- 2 files changed, 1 insertion(+), 6 deletions(-) diff --git a/pytest_fixtures/component/provision_pxe.py b/pytest_fixtures/component/provision_pxe.py index f39701f408c..2fc375bc5f1 100644 --- a/pytest_fixtures/component/provision_pxe.py +++ b/pytest_fixtures/component/provision_pxe.py @@ -244,13 +244,10 @@ def provision_multiple_hosts(module_ssh_key_file, pxe_loader, request): cd_iso = ( "" # TODO: Make this an optional fixture parameter (update vm_firmware when adding this) ) - # Keeping the default value to 2 - count = request.param if request.param is not None else 2 - with Broker( workflow="deploy-configure-pxe-provisioning-host-rhv", host_class=ContentHost, - _count=count, + _count=getattr(request, 'param', 2), target_vlan_id=vlan_id, target_vm_firmware=pxe_loader.vm_firmware, target_vm_cd_iso=cd_iso, diff --git a/tests/foreman/api/test_discoveredhost.py b/tests/foreman/api/test_discoveredhost.py index 40738be8aa2..9ee991d13ab 100644 --- a/tests/foreman/api/test_discoveredhost.py +++ b/tests/foreman/api/test_discoveredhost.py @@ -397,9 +397,7 @@ def test_positive_reboot_pxe_host( @pytest.mark.on_premises_provisioning @pytest.mark.parametrize('module_provisioning_sat', ['discovery'], indirect=True) - @pytest.mark.parametrize('pxe_loader', ['bios'], indirect=True) @pytest.mark.rhel_ver_match('9') - @pytest.mark.parametrize('provision_multiple_hosts', [2]) @pytest.mark.tier3 def test_positive_reboot_all_pxe_hosts( self,