diff --git a/canaille/models.py b/canaille/models.py index 9cc04693..1e41b6a8 100644 --- a/canaille/models.py +++ b/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 diff --git a/tests/test_groups.py b/tests/test_groups.py index 8fd9d572..0af76928 100644 --- a/tests/test_groups.py +++ b/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), diff --git a/tests/test_profile.py b/tests/test_profile.py index d8f27f87..f0921b94 100644 --- a/tests/test_profile.py +++ b/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")