From be85b5b67fad6ce471c31fbff42676b2127301b3 Mon Sep 17 00:00:00 2001 From: Dirk Julich Date: Thu, 18 Jun 2026 15:18:38 +0200 Subject: [PATCH] Revert setup_managed_roles from ext_auditor tests MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The setup_managed_roles fixture conflicts with the ext_auditor_rd fixture by deleting the Alien Auditor role definition. These tests don't need it — the defensive view code handles missing role definitions gracefully. Co-Authored-By: Claude Opus 4.6 --- awx/main/tests/functional/dab_rbac/test_external_auditor.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/awx/main/tests/functional/dab_rbac/test_external_auditor.py b/awx/main/tests/functional/dab_rbac/test_external_auditor.py index d05c11687d..c3602d7366 100644 --- a/awx/main/tests/functional/dab_rbac/test_external_auditor.py +++ b/awx/main/tests/functional/dab_rbac/test_external_auditor.py @@ -66,7 +66,7 @@ class TestExternalAuditorRoleAllModels: assert obj not in get_user_queryset(rando, model) assert obj in get_user_queryset(ext_auditor, model) - def test_global_list(self, obj_factory, model, ext_auditor, rando, get, setup_managed_roles): + def test_global_list(self, obj_factory, model, ext_auditor, rando, get): fixture_name = model._meta.verbose_name.replace(' ', '_') obj_factory(fixture_name) @@ -85,7 +85,7 @@ class TestExternalAuditorRoleAllModels: r = get(url, user=ext_auditor, expect=200) assert r.data['count'] == initial_ct + 1 - def test_detail_view(self, obj_factory, model, ext_auditor, rando, get, setup_managed_roles): + def test_detail_view(self, obj_factory, model, ext_auditor, rando, get): fixture_name = model._meta.verbose_name.replace(' ', '_') obj = obj_factory(fixture_name)