Skip to content

Commit

Permalink
do not mock
Browse files Browse the repository at this point in the history
Signed-off-by: Seth Foster <[email protected]>
  • Loading branch information
fosterseth committed Aug 21, 2024
1 parent ae9f15d commit 8dd8d51
Showing 1 changed file with 4 additions and 6 deletions.
10 changes: 4 additions & 6 deletions awx/main/tests/functional/dab_rbac/test_dab_rbac_api.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
import pytest
from unittest import mock

from django.contrib.contenttypes.models import ContentType
from django.urls import reverse as django_reverse
Expand Down Expand Up @@ -151,11 +150,10 @@ def test_assign_credential_to_user_of_another_org(setup_managed_roles, credentia
@pytest.mark.django_db
@override_settings(ALLOW_LOCAL_RESOURCE_MANAGEMENT=False)
def test_team_member_role_not_assignable(team, rando, post, admin_user, setup_managed_roles):
with mock.patch('awx.main.models.organization.Organization.validate_role_assignment') as mock_validate:
member_rd = RoleDefinition.objects.get(name='Organization Member')
url = django_reverse('roleuserassignment-list')
r = post(url, data={'object_id': team.id, 'role_definition': member_rd.id, 'user': rando.id}, user=admin_user, expect=400)
assert 'Not managed locally' in str(r.data)
member_rd = RoleDefinition.objects.get(name='Controller Organization Member')
url = django_reverse('roleuserassignment-list')
r = post(url, data={'object_id': team.id, 'role_definition': member_rd.id, 'user': rando.id}, user=admin_user, expect=400)
assert 'Not managed locally' in str(r.data)


@pytest.mark.django_db
Expand Down

0 comments on commit 8dd8d51

Please sign in to comment.