diff --git a/hobo/provisionning/utils.py b/hobo/provisionning/utils.py index 713e841..a5735c4 100644 --- a/hobo/provisionning/utils.py +++ b/hobo/provisionning/utils.py @@ -198,8 +198,7 @@ class NotificationProcessing: if full and action == 'provision': Role.objects.exclude(uuid__in=uuids).delete() elif action == 'deprovision': - for role in Role.objects.filter(uuid__in=uuids): - role.delete() + Role.objects.filter(uuid__in=uuids).delete() @classmethod def provision(cls, object_type, issuer, action, data, full): diff --git a/tests_multitenant/test_hobo_notify.py b/tests_multitenant/test_hobo_notify.py index de5bc2e..166df2b 100644 --- a/tests_multitenant/test_hobo_notify.py +++ b/tests_multitenant/test_hobo_notify.py @@ -338,7 +338,7 @@ def test_hobo_notify_roles_db_queries(caplog, tenants): } with CaptureQueriesContext(connection) as ctx: Command.process_notification(tenant, notification) - assert len(ctx.captured_queries) == 39 + assert len(ctx.captured_queries) == 15 assert Group.objects.count() == 0 assert Role.objects.count() == 0