
* lib/sync/outgoing: add dry run Signed-off-by: Jens Langhammer <jens@goauthentik.io> * add option to temporarily override dry run Signed-off-by: Jens Langhammer <jens@goauthentik.io> * web a Signed-off-by: Jens Langhammer <jens@goauthentik.io> * web b Signed-off-by: Jens Langhammer <jens@goauthentik.io> * format Signed-off-by: Jens Langhammer <jens@goauthentik.io> * add some test Signed-off-by: Jens Langhammer <jens@goauthentik.io> * add more tests Signed-off-by: Jens Langhammer <jens@goauthentik.io> * add dry run label Signed-off-by: Jens Langhammer <jens@goauthentik.io> * add support for entra too Signed-off-by: Jens Langhammer <jens@goauthentik.io> * add web Signed-off-by: Jens Langhammer <jens@goauthentik.io> * add entra test and improve error handling Signed-off-by: Jens Langhammer <jens@goauthentik.io> --------- Signed-off-by: Jens Langhammer <jens@goauthentik.io>
450 lines
19 KiB
Python
450 lines
19 KiB
Python
"""Microsoft Entra User tests"""
|
|
|
|
from unittest.mock import AsyncMock, MagicMock, patch
|
|
|
|
from azure.identity.aio import ClientSecretCredential
|
|
from django.urls import reverse
|
|
from msgraph.generated.models.group_collection_response import GroupCollectionResponse
|
|
from msgraph.generated.models.organization import Organization
|
|
from msgraph.generated.models.organization_collection_response import OrganizationCollectionResponse
|
|
from msgraph.generated.models.user import User as MSUser
|
|
from msgraph.generated.models.user_collection_response import UserCollectionResponse
|
|
from msgraph.generated.models.verified_domain import VerifiedDomain
|
|
from rest_framework.test import APITestCase
|
|
|
|
from authentik.blueprints.tests import apply_blueprint
|
|
from authentik.core.models import Application, Group, User
|
|
from authentik.core.tests.utils import create_test_admin_user
|
|
from authentik.enterprise.providers.microsoft_entra.models import (
|
|
MicrosoftEntraProvider,
|
|
MicrosoftEntraProviderMapping,
|
|
MicrosoftEntraProviderUser,
|
|
)
|
|
from authentik.enterprise.providers.microsoft_entra.tasks import microsoft_entra_sync
|
|
from authentik.events.models import Event, EventAction
|
|
from authentik.lib.generators import generate_id
|
|
from authentik.lib.sync.outgoing.models import OutgoingSyncDeleteAction
|
|
from authentik.tenants.models import Tenant
|
|
|
|
|
|
class MicrosoftEntraUserTests(APITestCase):
|
|
"""Microsoft Entra User tests"""
|
|
|
|
@apply_blueprint("system/providers-microsoft-entra.yaml")
|
|
def setUp(self) -> None:
|
|
# Delete all users and groups as the mocked HTTP responses only return one ID
|
|
# which will cause errors with multiple users
|
|
Tenant.objects.update(avatars="none")
|
|
User.objects.all().exclude_anonymous().delete()
|
|
Group.objects.all().delete()
|
|
self.provider: MicrosoftEntraProvider = MicrosoftEntraProvider.objects.create(
|
|
name=generate_id(),
|
|
client_id=generate_id(),
|
|
client_secret=generate_id(),
|
|
tenant_id=generate_id(),
|
|
exclude_users_service_account=True,
|
|
)
|
|
self.app: Application = Application.objects.create(
|
|
name=generate_id(),
|
|
slug=generate_id(),
|
|
)
|
|
self.app.backchannel_providers.add(self.provider)
|
|
self.provider.property_mappings.add(
|
|
MicrosoftEntraProviderMapping.objects.get(
|
|
managed="goauthentik.io/providers/microsoft_entra/user"
|
|
)
|
|
)
|
|
self.provider.property_mappings_group.add(
|
|
MicrosoftEntraProviderMapping.objects.get(
|
|
managed="goauthentik.io/providers/microsoft_entra/group"
|
|
)
|
|
)
|
|
self.creds = ClientSecretCredential(generate_id(), generate_id(), generate_id())
|
|
|
|
def test_user_create(self):
|
|
"""Test user creation"""
|
|
uid = generate_id()
|
|
with (
|
|
patch(
|
|
"authentik.enterprise.providers.microsoft_entra.models.MicrosoftEntraProvider.microsoft_credentials",
|
|
MagicMock(return_value={"credentials": self.creds}),
|
|
),
|
|
patch(
|
|
"msgraph.generated.organization.organization_request_builder.OrganizationRequestBuilder.get",
|
|
AsyncMock(
|
|
return_value=OrganizationCollectionResponse(
|
|
value=[
|
|
Organization(verified_domains=[VerifiedDomain(name="goauthentik.io")])
|
|
]
|
|
)
|
|
),
|
|
),
|
|
patch(
|
|
"msgraph.generated.users.users_request_builder.UsersRequestBuilder.post",
|
|
AsyncMock(return_value=MSUser(id=generate_id())),
|
|
) as user_create,
|
|
):
|
|
user = User.objects.create(
|
|
username=uid,
|
|
name=f"{uid} {uid}",
|
|
email=f"{uid}@goauthentik.io",
|
|
)
|
|
microsoft_user = MicrosoftEntraProviderUser.objects.filter(
|
|
provider=self.provider, user=user
|
|
).first()
|
|
self.assertIsNotNone(microsoft_user)
|
|
self.assertFalse(Event.objects.filter(action=EventAction.SYSTEM_EXCEPTION).exists())
|
|
user_create.assert_called_once()
|
|
|
|
def test_user_create_dry_run(self):
|
|
"""Test user creation (dry run)"""
|
|
self.provider.dry_run = True
|
|
self.provider.save()
|
|
uid = generate_id()
|
|
with (
|
|
patch(
|
|
"authentik.enterprise.providers.microsoft_entra.models.MicrosoftEntraProvider.microsoft_credentials",
|
|
MagicMock(return_value={"credentials": self.creds}),
|
|
),
|
|
patch(
|
|
"msgraph.generated.organization.organization_request_builder.OrganizationRequestBuilder.get",
|
|
AsyncMock(
|
|
return_value=OrganizationCollectionResponse(
|
|
value=[
|
|
Organization(verified_domains=[VerifiedDomain(name="goauthentik.io")])
|
|
]
|
|
)
|
|
),
|
|
),
|
|
):
|
|
user = User.objects.create(
|
|
username=uid,
|
|
name=f"{uid} {uid}",
|
|
email=f"{uid}@goauthentik.io",
|
|
)
|
|
microsoft_user = MicrosoftEntraProviderUser.objects.filter(
|
|
provider=self.provider, user=user
|
|
).first()
|
|
self.assertIsNone(microsoft_user)
|
|
self.assertFalse(Event.objects.filter(action=EventAction.SYSTEM_EXCEPTION).exists())
|
|
|
|
def test_user_not_created(self):
|
|
"""Test without property mappings, no group is created"""
|
|
self.provider.property_mappings.clear()
|
|
uid = generate_id()
|
|
with (
|
|
patch(
|
|
"authentik.enterprise.providers.microsoft_entra.models.MicrosoftEntraProvider.microsoft_credentials",
|
|
MagicMock(return_value={"credentials": self.creds}),
|
|
),
|
|
patch(
|
|
"msgraph.generated.organization.organization_request_builder.OrganizationRequestBuilder.get",
|
|
AsyncMock(
|
|
return_value=OrganizationCollectionResponse(
|
|
value=[
|
|
Organization(verified_domains=[VerifiedDomain(name="goauthentik.io")])
|
|
]
|
|
)
|
|
),
|
|
),
|
|
patch(
|
|
"msgraph.generated.users.users_request_builder.UsersRequestBuilder.post",
|
|
AsyncMock(return_value=MSUser(id=generate_id())),
|
|
) as user_create,
|
|
):
|
|
user = User.objects.create(
|
|
username=uid,
|
|
name=f"{uid} {uid}",
|
|
email=f"{uid}@goauthentik.io",
|
|
)
|
|
microsoft_user = MicrosoftEntraProviderUser.objects.filter(
|
|
provider=self.provider, user=user
|
|
).first()
|
|
self.assertIsNone(microsoft_user)
|
|
self.assertFalse(Event.objects.filter(action=EventAction.SYSTEM_EXCEPTION).exists())
|
|
user_create.assert_not_called()
|
|
|
|
def test_user_create_update(self):
|
|
"""Test user updating"""
|
|
uid = generate_id()
|
|
with (
|
|
patch(
|
|
"authentik.enterprise.providers.microsoft_entra.models.MicrosoftEntraProvider.microsoft_credentials",
|
|
MagicMock(return_value={"credentials": self.creds}),
|
|
),
|
|
patch(
|
|
"msgraph.generated.organization.organization_request_builder.OrganizationRequestBuilder.get",
|
|
AsyncMock(
|
|
return_value=OrganizationCollectionResponse(
|
|
value=[
|
|
Organization(verified_domains=[VerifiedDomain(name="goauthentik.io")])
|
|
]
|
|
)
|
|
),
|
|
),
|
|
patch(
|
|
"msgraph.generated.users.users_request_builder.UsersRequestBuilder.post",
|
|
AsyncMock(return_value=MSUser(id=generate_id())),
|
|
) as user_create,
|
|
patch(
|
|
"msgraph.generated.users.item.user_item_request_builder.UserItemRequestBuilder.patch",
|
|
AsyncMock(return_value=MSUser(id=generate_id())),
|
|
) as user_patch,
|
|
):
|
|
user = User.objects.create(
|
|
username=uid,
|
|
name=f"{uid} {uid}",
|
|
email=f"{uid}@goauthentik.io",
|
|
)
|
|
microsoft_user = MicrosoftEntraProviderUser.objects.filter(
|
|
provider=self.provider, user=user
|
|
).first()
|
|
self.assertIsNotNone(microsoft_user)
|
|
|
|
user.name = "new name"
|
|
user.save()
|
|
self.assertFalse(Event.objects.filter(action=EventAction.SYSTEM_EXCEPTION).exists())
|
|
user_create.assert_called_once()
|
|
user_patch.assert_called_once()
|
|
|
|
def test_user_create_delete(self):
|
|
"""Test user deletion"""
|
|
uid = generate_id()
|
|
with (
|
|
patch(
|
|
"authentik.enterprise.providers.microsoft_entra.models.MicrosoftEntraProvider.microsoft_credentials",
|
|
MagicMock(return_value={"credentials": self.creds}),
|
|
),
|
|
patch(
|
|
"msgraph.generated.organization.organization_request_builder.OrganizationRequestBuilder.get",
|
|
AsyncMock(
|
|
return_value=OrganizationCollectionResponse(
|
|
value=[
|
|
Organization(verified_domains=[VerifiedDomain(name="goauthentik.io")])
|
|
]
|
|
)
|
|
),
|
|
),
|
|
patch(
|
|
"msgraph.generated.users.users_request_builder.UsersRequestBuilder.post",
|
|
AsyncMock(return_value=MSUser(id=generate_id())),
|
|
) as user_create,
|
|
patch(
|
|
"msgraph.generated.users.item.user_item_request_builder.UserItemRequestBuilder.delete",
|
|
AsyncMock(),
|
|
) as user_delete,
|
|
):
|
|
user = User.objects.create(
|
|
username=uid,
|
|
name=f"{uid} {uid}",
|
|
email=f"{uid}@goauthentik.io",
|
|
)
|
|
microsoft_user = MicrosoftEntraProviderUser.objects.filter(
|
|
provider=self.provider, user=user
|
|
).first()
|
|
self.assertIsNotNone(microsoft_user)
|
|
|
|
user.delete()
|
|
self.assertFalse(Event.objects.filter(action=EventAction.SYSTEM_EXCEPTION).exists())
|
|
user_create.assert_called_once()
|
|
user_delete.assert_called_once()
|
|
|
|
def test_user_create_delete_suspend(self):
|
|
"""Test user deletion (delete action = Suspend)"""
|
|
self.provider.user_delete_action = OutgoingSyncDeleteAction.SUSPEND
|
|
self.provider.save()
|
|
uid = generate_id()
|
|
with (
|
|
patch(
|
|
"authentik.enterprise.providers.microsoft_entra.models.MicrosoftEntraProvider.microsoft_credentials",
|
|
MagicMock(return_value={"credentials": self.creds}),
|
|
),
|
|
patch(
|
|
"msgraph.generated.organization.organization_request_builder.OrganizationRequestBuilder.get",
|
|
AsyncMock(
|
|
return_value=OrganizationCollectionResponse(
|
|
value=[
|
|
Organization(verified_domains=[VerifiedDomain(name="goauthentik.io")])
|
|
]
|
|
)
|
|
),
|
|
),
|
|
patch(
|
|
"msgraph.generated.users.users_request_builder.UsersRequestBuilder.post",
|
|
AsyncMock(return_value=MSUser(id=generate_id())),
|
|
) as user_create,
|
|
patch(
|
|
"msgraph.generated.users.item.user_item_request_builder.UserItemRequestBuilder.patch",
|
|
AsyncMock(return_value=MSUser(id=generate_id())),
|
|
) as user_patch,
|
|
patch(
|
|
"msgraph.generated.users.item.user_item_request_builder.UserItemRequestBuilder.delete",
|
|
AsyncMock(),
|
|
) as user_delete,
|
|
):
|
|
user = User.objects.create(
|
|
username=uid,
|
|
name=f"{uid} {uid}",
|
|
email=f"{uid}@goauthentik.io",
|
|
)
|
|
microsoft_user = MicrosoftEntraProviderUser.objects.filter(
|
|
provider=self.provider, user=user
|
|
).first()
|
|
self.assertIsNotNone(microsoft_user)
|
|
|
|
user.delete()
|
|
self.assertFalse(
|
|
MicrosoftEntraProviderUser.objects.filter(
|
|
provider=self.provider, user__username=uid
|
|
).exists()
|
|
)
|
|
user_create.assert_called_once()
|
|
user_patch.assert_called_once()
|
|
self.assertFalse(user_patch.call_args[0][0].account_enabled)
|
|
user_delete.assert_not_called()
|
|
|
|
def test_user_create_delete_do_nothing(self):
|
|
"""Test user deletion (delete action = do nothing)"""
|
|
self.provider.user_delete_action = OutgoingSyncDeleteAction.DO_NOTHING
|
|
self.provider.save()
|
|
uid = generate_id()
|
|
with (
|
|
patch(
|
|
"authentik.enterprise.providers.microsoft_entra.models.MicrosoftEntraProvider.microsoft_credentials",
|
|
MagicMock(return_value={"credentials": self.creds}),
|
|
),
|
|
patch(
|
|
"msgraph.generated.organization.organization_request_builder.OrganizationRequestBuilder.get",
|
|
AsyncMock(
|
|
return_value=OrganizationCollectionResponse(
|
|
value=[
|
|
Organization(verified_domains=[VerifiedDomain(name="goauthentik.io")])
|
|
]
|
|
)
|
|
),
|
|
),
|
|
patch(
|
|
"msgraph.generated.users.users_request_builder.UsersRequestBuilder.post",
|
|
AsyncMock(return_value=MSUser(id=generate_id())),
|
|
) as user_create,
|
|
patch(
|
|
"msgraph.generated.users.item.user_item_request_builder.UserItemRequestBuilder.patch",
|
|
AsyncMock(return_value=MSUser(id=generate_id())),
|
|
) as user_patch,
|
|
patch(
|
|
"msgraph.generated.users.item.user_item_request_builder.UserItemRequestBuilder.delete",
|
|
AsyncMock(),
|
|
) as user_delete,
|
|
):
|
|
user = User.objects.create(
|
|
username=uid,
|
|
name=f"{uid} {uid}",
|
|
email=f"{uid}@goauthentik.io",
|
|
)
|
|
microsoft_user = MicrosoftEntraProviderUser.objects.filter(
|
|
provider=self.provider, user=user
|
|
).first()
|
|
self.assertIsNotNone(microsoft_user)
|
|
|
|
user.delete()
|
|
self.assertFalse(
|
|
MicrosoftEntraProviderUser.objects.filter(
|
|
provider=self.provider, user__username=uid
|
|
).exists()
|
|
)
|
|
user_create.assert_called_once()
|
|
user_patch.assert_not_called()
|
|
user_delete.assert_not_called()
|
|
|
|
def test_sync_task(self):
|
|
"""Test user discovery"""
|
|
uid = generate_id()
|
|
self.app.backchannel_providers.remove(self.provider)
|
|
different_user = User.objects.create(
|
|
username=uid,
|
|
email=f"{uid}@goauthentik.io",
|
|
)
|
|
self.app.backchannel_providers.add(self.provider)
|
|
with (
|
|
patch(
|
|
"authentik.enterprise.providers.microsoft_entra.models.MicrosoftEntraProvider.microsoft_credentials",
|
|
MagicMock(return_value={"credentials": self.creds}),
|
|
),
|
|
patch(
|
|
"msgraph.generated.organization.organization_request_builder.OrganizationRequestBuilder.get",
|
|
AsyncMock(
|
|
return_value=OrganizationCollectionResponse(
|
|
value=[
|
|
Organization(verified_domains=[VerifiedDomain(name="goauthentik.io")])
|
|
]
|
|
)
|
|
),
|
|
),
|
|
patch(
|
|
"msgraph.generated.users.item.user_item_request_builder.UserItemRequestBuilder.patch",
|
|
AsyncMock(return_value=MSUser(id=generate_id())),
|
|
),
|
|
patch(
|
|
"msgraph.generated.users.users_request_builder.UsersRequestBuilder.get",
|
|
AsyncMock(
|
|
return_value=UserCollectionResponse(
|
|
value=[MSUser(mail=f"{uid}@goauthentik.io", id=uid)]
|
|
)
|
|
),
|
|
) as user_list,
|
|
patch(
|
|
"msgraph.generated.groups.groups_request_builder.GroupsRequestBuilder.get",
|
|
AsyncMock(return_value=GroupCollectionResponse(value=[])),
|
|
),
|
|
):
|
|
microsoft_entra_sync.delay(self.provider.pk).get()
|
|
self.assertTrue(
|
|
MicrosoftEntraProviderUser.objects.filter(
|
|
user=different_user, provider=self.provider
|
|
).exists()
|
|
)
|
|
self.assertFalse(Event.objects.filter(action=EventAction.SYSTEM_EXCEPTION).exists())
|
|
user_list.assert_called_once()
|
|
|
|
def test_connect_manual(self):
|
|
"""test manual user connection"""
|
|
uid = generate_id()
|
|
self.app.backchannel_providers.remove(self.provider)
|
|
admin = create_test_admin_user()
|
|
different_user = User.objects.create(
|
|
username=uid,
|
|
email=f"{uid}@goauthentik.io",
|
|
)
|
|
self.app.backchannel_providers.add(self.provider)
|
|
with (
|
|
patch(
|
|
"authentik.enterprise.providers.microsoft_entra.models.MicrosoftEntraProvider.microsoft_credentials",
|
|
MagicMock(return_value={"credentials": self.creds}),
|
|
),
|
|
patch(
|
|
"msgraph.generated.organization.organization_request_builder.OrganizationRequestBuilder.get",
|
|
AsyncMock(
|
|
return_value=OrganizationCollectionResponse(
|
|
value=[
|
|
Organization(verified_domains=[VerifiedDomain(name="goauthentik.io")])
|
|
]
|
|
)
|
|
),
|
|
),
|
|
patch(
|
|
"authentik.enterprise.providers.microsoft_entra.clients.users.MicrosoftEntraUserClient.update_single_attribute",
|
|
MagicMock(),
|
|
) as user_get,
|
|
):
|
|
self.client.force_login(admin)
|
|
response = self.client.post(
|
|
reverse("authentik_api:microsoftentraprovideruser-list"),
|
|
data={
|
|
"microsoft_id": generate_id(),
|
|
"user": different_user.pk,
|
|
"provider": self.provider.pk,
|
|
},
|
|
)
|
|
self.assertEqual(response.status_code, 201)
|
|
user_get.assert_called_once()
|