xremove user to prevent ForeignKeyViolation

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