diff --git a/tests/foreman/ui/test_host.py b/tests/foreman/ui/test_host.py index 0b3f94bf906..8db531362ff 100644 --- a/tests/foreman/ui/test_host.py +++ b/tests/foreman/ui/test_host.py @@ -610,7 +610,7 @@ def test_positive_create_with_inherited_params( ) -def test_negative_delete_primary_interface(target_sat, host_ui_options, ui_user): +def test_negative_delete_primary_interface(module_target_sat, host_ui_options, ui_user): """Attempt to delete primary interface of a host :id: bc747e2c-38d9-4920-b4ae-6010851f704e @@ -623,7 +623,7 @@ def test_negative_delete_primary_interface(target_sat, host_ui_options, ui_user) """ values, host_name = host_ui_options interface_id = values['interfaces.interface.device_identifier'] - with target_sat.ui_session(user=ui_user.login, password=ui_user.password) as session: + with module_target_sat.ui_session(user=ui_user.login, password=ui_user.password) as session: session.location.select(values['host.location']) session.host.create(values) with pytest.raises(DisabledWidgetError) as context: @@ -711,7 +711,9 @@ def test_positive_remove_parameter_non_admin_user( organization=module_org, host_parameters_attributes=[parameter], ).create() - with target_sat.ui_session(test_name, user=user.login, password=user_password) as session: + with target_sat.ui_session( + testname=test_name, user=user.login, password=user_password + ) as session: values = session.host_new.read(host.name, 'parameters') assert values['parameters']['host_params'][0] == parameter session.host_new.update(host.name, {'parameters.host_params': []}) @@ -720,7 +722,7 @@ def test_positive_remove_parameter_non_admin_user( def test_negative_remove_parameter_non_admin_user( - test_name, module_org, smart_proxy_location, target_sat, expected_permissions + test_name, module_org, smart_proxy_location, module_target_sat, expected_permissions ): """Attempt to remove host parameter as a non-admin user with insufficient permissions @@ -737,8 +739,8 @@ def test_negative_remove_parameter_non_admin_user( user_password = gen_string('alpha') parameter = {'name': gen_string('alpha'), 'value': gen_string('alpha')} - role = target_sat.api.Role(organization=[module_org]).create() - target_sat.api_factory.create_role_permissions( + role = module_target_sat.api.Role(organization=[module_org]).create() + module_target_sat.api_factory.create_role_permissions( role, { 'Parameter': ['view_params'], @@ -746,7 +748,7 @@ def test_negative_remove_parameter_non_admin_user( 'Operatingsystem': ['view_operatingsystems'], }, ) - user = target_sat.api.User( + user = module_target_sat.api.User( role=[role], admin=False, password=user_password, @@ -755,7 +757,7 @@ def test_negative_remove_parameter_non_admin_user( default_organization=module_org, default_location=smart_proxy_location, ).create() - host = target_sat.api.Host( + host = module_target_sat.api.Host( content_facet_attributes={ 'content_view_id': module_org.default_content_view.id, 'lifecycle_environment_id': module_org.library.id, @@ -764,7 +766,9 @@ def test_negative_remove_parameter_non_admin_user( organization=module_org, host_parameters_attributes=[parameter], ).create() - with target_sat.ui_session(test_name, user=user.login, password=user_password) as session: + with module_target_sat.ui_session( + testname=test_name, user=user.login, password=user_password + ) as session: values = session.host_new.read(host.name, 'parameters') assert values['parameters']['host_params'][0] == parameter with pytest.raises(NoSuchElementException) as context: