diff --git a/zerver/lib/user_groups.py b/zerver/lib/user_groups.py index 7e18412503..9f42aab098 100644 --- a/zerver/lib/user_groups.py +++ b/zerver/lib/user_groups.py @@ -22,11 +22,6 @@ def access_user_group_by_id(user_group_id: int, user_profile: UserProfile) -> Us return user_group -def user_groups_in_realm(realm: Realm) -> List[UserGroup]: - user_groups = UserGroup.objects.filter(realm=realm) - return list(user_groups) - - def user_groups_in_realm_serialized(realm: Realm) -> List[Dict[str, Any]]: """This function is used in do_events_register code path so this code should be performant. We need to do 2 database queries because diff --git a/zerver/tests/test_user_groups.py b/zerver/tests/test_user_groups.py index 9ec33b366c..a77d6666ab 100644 --- a/zerver/tests/test_user_groups.py +++ b/zerver/tests/test_user_groups.py @@ -14,7 +14,6 @@ from zerver.lib.user_groups import ( create_user_group, get_memberships_of_users, get_user_groups, - user_groups_in_realm, user_groups_in_realm_serialized, ) from zerver.models import Realm, UserGroup, UserGroupMembership, UserProfile, get_realm @@ -27,15 +26,6 @@ class UserGroupTestCase(ZulipTestCase): members = [self.example_user("othello")] return create_user_group(group_name, members, realm) - def test_user_groups_in_realm(self) -> None: - realm = get_realm("zulip") - self.assert_length(user_groups_in_realm(realm), 1) - self.create_user_group_for_test("support") - user_groups = user_groups_in_realm(realm) - self.assert_length(user_groups, 2) - names = {ug.name for ug in user_groups} - self.assertEqual(names, {"hamletcharacters", "support"}) - def test_user_groups_in_realm_serialized(self) -> None: realm = get_realm("zulip") user_group = UserGroup.objects.first()