entrouvert
/
hobo
16
0
Fork 0
Code Issues Pull Requests 18 Releases Activity

provisionning: delete roles in one query when deprovisionning (#55043)
gitea-wip/hobo/pipeline/head There was a failure building this commit Details
gitea/hobo/pipeline/head Something is wrong with the build of this commit Details

This commit is contained in:
Emmanuel Cazenave 2021-06-21 17:57:36 +02:00
parent 5dc88fc9a7
commit 6e896dbcdd
2 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
hobo/provisionning/utils.py
View File

@ -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):

2
tests_multitenant/test_hobo_notify.py
View File

@ -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