globuzma/canaille-globuzma
1
0
Fork 0
forked from Github-Mirrors/canaille
Code Pull requests Activity

refactor: LDAP backend tolerates double deletions

Browse source
This commit is contained in:
Éloi Rivard 2024-04-28 16:18:24 +02:00
parent 867de91740
commit 69b565e1ad
No known key found for this signature in database
GPG key ID: 7EDA204EA57DD184
2 changed files with 6 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
canaille/backends/ldap/ldapobject.py
View file

@ -477,4 +477,7 @@ class LDAPObject(BackendModel, metaclass=LDAPObjectMetaclass):
def delete(self): def delete(self):
conn = Backend.get().connection conn = Backend.get().connection
try:
conn.delete_s(self.dn) conn.delete_s(self.dn)
except ldap.NO_SUCH_OBJECT:
pass

2
tests/backends/test_models.py
View file

@ -61,6 +61,8 @@ def test_model_lifecycle(testclient, backend):
assert not models.User.query(id=user.id) assert not models.User.query(id=user.id)
assert not models.User.get(id=user.id) assert not models.User.get(id=user.id)
user.delete()
def test_model_attribute_edition(testclient, backend): def test_model_attribute_edition(testclient, backend):
user = models.User( user = models.User(