user [nfc]: Cut long-disused function sortAlphabetically

This hasn't been used since b1bd9c7, in 2017.
chrisbobbe · Mar 17, 2022 · 0c114b7 · 0c114b7
1 parent 0fca8ef
commit 0c114b7
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 21 deletions.
diff --git a/src/users/__tests__/userHelpers-test.js b/src/users/__tests__/userHelpers-test.js
@@ -7,7 +7,6 @@ import {
   filterUserList,
   getAutocompleteSuggestion,
   getAutocompleteUserGroupSuggestions,
-  sortAlphabetically,
   filterUserStartWith,
   filterUserThatContains,
   filterUserMatchesEmail,
@@ -209,22 +208,6 @@ describe('sortUserList', () => {
   });
 });
 
-describe('sortAlphabetically', () => {
-  test('alphabetically sort user list by full_name', () => {
-    const user1 = eg.makeUser({ full_name: 'zoe' });
-    const user2 = eg.makeUser({ full_name: 'Ring' });
-    const user3 = eg.makeUser({ full_name: 'watch' });
-    const user4 = eg.makeUser({ full_name: 'mobile' });
-    const user5 = eg.makeUser({ full_name: 'Ring' });
-    const user6 = eg.makeUser({ full_name: 'hardware' });
-    const user7 = eg.makeUser({ full_name: 'Bob' });
-    const users = deepFreeze([user1, user2, user3, user4, user5, user6, user7]);
-
-    const expectedUsers = [user7, user6, user4, user2, user5, user3, user1];
-    expect(sortAlphabetically(users)).toEqual(expectedUsers);
-  });
-});
-
 describe('filterUserStartWith', () => {
   test('returns users whose name starts with filter excluding self', () => {
     const user1 = eg.makeUser({ full_name: 'Apple' });

diff --git a/src/users/userHelpers.js b/src/users/userHelpers.js
@@ -6,7 +6,6 @@ import * as typeahead from '@zulip/shared/js/typeahead';
 import type {
   MutedUsersState,
   UserPresence,
-  User,
   UserId,
   UserGroup,
   PresenceState,
@@ -78,9 +77,6 @@ export const filterUserList = (
       || user.email.toLowerCase().includes(filter.toLowerCase()),
   );
 
-export const sortAlphabetically = (users: $ReadOnlyArray<User>): $ReadOnlyArray<User> =>
-  [...users].sort((x1, x2) => x1.full_name.toLowerCase().localeCompare(x2.full_name.toLowerCase()));
-
 export const filterUserStartWith = (
   users: $ReadOnlyArray<AutocompleteOption>,
   filter: string = '',