diff --git a/tests/foreman/api/test_ansible.py b/tests/foreman/api/test_ansible.py index e69a9dddf26..7695e347cf9 100644 --- a/tests/foreman/api/test_ansible.py +++ b/tests/foreman/api/test_ansible.py @@ -44,12 +44,7 @@ def filtered_user(target_sat, module_org, module_location): target_sat.api.Filter(permission=permission_hostgroups, role=role).create() target_sat.api.Filter(permission=permission_facts, role=role).create() - password = gen_string('alpha') - user = target_sat.api.User( - role=[role], password=password, location=[module_location], organization=[module_org] - ).create() - - return user, password + return role @pytest.mark.upgrade @@ -275,7 +270,7 @@ def _finalize(): @pytest.mark.rhel_ver_match('[78]') @pytest.mark.tier2 def test_positive_read_facts_with_filter( - self, target_sat, rex_contenthost, filtered_user, module_org, module_location + self, request, target_sat, rex_contenthost, filtered_user, module_org, module_location ): """Read host's Ansible facts as a user with a role that has host filter @@ -291,7 +286,13 @@ def test_positive_read_facts_with_filter( :customerscenario: true """ - user, password = filtered_user + role = filtered_user + password = gen_string('alpha') + user = target_sat.api.User( + role=[role], password=password, location=[module_location], organization=[module_org] + ).create() + if is_open('SAT-18656'): + request.addfinalizer(user.delete) host = rex_contenthost.nailgun_host host.organization = module_org host.location = module_location