refactor: LDAP backend tolerates double deletions

yaal-coop · Apr 28, 2024 · 69b565e · 69b565e
1 parent 867de91
commit 69b565e
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/canaille/backends/ldap/ldapobject.py b/canaille/backends/ldap/ldapobject.py
@@ -477,4 +477,7 @@ def save(self):
 
     def delete(self):
         conn = Backend.get().connection
-        conn.delete_s(self.dn)
+        try:
+            conn.delete_s(self.dn)
+        except ldap.NO_SUCH_OBJECT:
+            pass
diff --git a/tests/backends/test_models.py b/tests/backends/test_models.py
@@ -61,6 +61,8 @@ def test_model_lifecycle(testclient, backend):
     assert not models.User.query(id=user.id)
     assert not models.User.get(id=user.id)
 
+    user.delete()
+
 
 def test_model_attribute_edition(testclient, backend):
     user = models.User(