lazy group loading

canaille/models.py

@@ -31,7 +31,6 @@ class User(LDAPObject):
         user = super().get(dn, filter, conn)
         if user:
             user.load_permissions(conn)
-            user.load_groups(conn=conn)
 
         return user
 

tests/test_groups.py

@@ -12,6 +12,7 @@ def test_set_groups(app, slapd_connection, user, foo_group, bar_group):
         User.attr_type_by_name(conn=slapd_connection)
 
         user = User.get(dn=user.dn, conn=slapd_connection)
+        user.load_groups(conn=slapd_connection)
         assert set(Group.available_groups(conn=slapd_connection)) == {
             ("foo", foo_group.dn),
             ("bar", bar_group.dn),

tests/test_profile.py

@@ -32,6 +32,8 @@ def test_edition(
 
     with testclient.app.app_context():
         logged_user = User.get(dn=logged_user.dn, conn=slapd_connection)
+        logged_user.load_groups(conn=slapd_connection)
+
     assert ["user"] == logged_user.uid
     assert ["given_name"] == logged_user.givenName
     assert ["family_name"] == logged_user.sn
@@ -217,6 +219,7 @@ def test_user_creation_edition_and_deletion(
     res = res.form.submit(name="action", value="edit", status=302).follow(status=200)
     with testclient.app.app_context():
         george = User.get("george", conn=slapd_connection)
+        george.load_groups(conn=slapd_connection)
         assert "George" == george.givenName[0]
         assert george.groups == []
         assert george.check_password("totoyolo")
@@ -234,6 +237,7 @@ def test_user_creation_edition_and_deletion(
     res = res.form.submit(name="action", value="edit", status=200)
     with testclient.app.app_context():
         george = User.get("george", conn=slapd_connection)
+        george.load_groups(conn=slapd_connection)
         assert "Georgio" == george.givenName[0]
         assert george.check_password("totoyolo")