diff options
author | Jesse Hallam <jesse.hallam@gmail.com> | 2018-10-17 11:24:12 -0400 |
---|---|---|
committer | Harrison Healey <harrisonmhealey@gmail.com> | 2018-10-17 11:24:12 -0400 |
commit | 715097cc76510a3d78ba83e8544ee7c956ed26e9 (patch) | |
tree | 0b6d41e88bf75ad34c585d9db80f04cf8d780338 /api4/user_test.go | |
parent | e8c9ccaa7e47f1cba3d2b126f6ebbb092fa43235 (diff) | |
download | chat-715097cc76510a3d78ba83e8544ee7c956ed26e9.tar.gz chat-715097cc76510a3d78ba83e8544ee7c956ed26e9.tar.bz2 chat-715097cc76510a3d78ba83e8544ee7c956ed26e9.zip |
MM-12234: configurable limit to user autocomplete and search matches (#9499)
* unit test cleanup
* allow limiting user search results
* clean up test users before starting
* model UserSearchOptions to simplify parameters
Diffstat (limited to 'api4/user_test.go')
-rw-r--r-- | api4/user_test.go | 44 |
1 files changed, 22 insertions, 22 deletions
diff --git a/api4/user_test.go b/api4/user_test.go index b99011aeb..9df062a2a 100644 --- a/api4/user_test.go +++ b/api4/user_test.go @@ -759,92 +759,92 @@ func TestAutocompleteUsers(t *testing.T) { th.App.UpdateConfig(func(cfg *model.Config) { cfg.PrivacySettings.ShowFullName = showFullName }) }() - rusers, resp := Client.AutocompleteUsersInChannel(teamId, channelId, username, "") + rusers, resp := Client.AutocompleteUsersInChannel(teamId, channelId, username, model.USER_SEARCH_DEFAULT_LIMIT, "") CheckNoError(t, resp) if len(rusers.Users) != 1 { t.Fatal("should have returned 1 user") } - rusers, resp = Client.AutocompleteUsersInChannel(teamId, channelId, "amazonses", "") + rusers, resp = Client.AutocompleteUsersInChannel(teamId, channelId, "amazonses", model.USER_SEARCH_DEFAULT_LIMIT, "") CheckNoError(t, resp) if len(rusers.Users) != 0 { t.Fatal("should have returned 0 users") } - rusers, resp = Client.AutocompleteUsersInChannel(teamId, channelId, "", "") + rusers, resp = Client.AutocompleteUsersInChannel(teamId, channelId, "", model.USER_SEARCH_DEFAULT_LIMIT, "") CheckNoError(t, resp) if len(rusers.Users) < 2 { t.Fatal("should have many users") } - rusers, resp = Client.AutocompleteUsersInChannel("", channelId, "", "") + rusers, resp = Client.AutocompleteUsersInChannel("", channelId, "", model.USER_SEARCH_DEFAULT_LIMIT, "") CheckNoError(t, resp) if len(rusers.Users) < 2 { t.Fatal("should have many users") } - rusers, resp = Client.AutocompleteUsersInTeam(teamId, username, "") + rusers, resp = Client.AutocompleteUsersInTeam(teamId, username, model.USER_SEARCH_DEFAULT_LIMIT, "") CheckNoError(t, resp) if len(rusers.Users) != 1 { t.Fatal("should have returned 1 user") } - rusers, resp = Client.AutocompleteUsers(username, "") + rusers, resp = Client.AutocompleteUsers(username, model.USER_SEARCH_DEFAULT_LIMIT, "") CheckNoError(t, resp) if len(rusers.Users) != 1 { t.Fatal("should have returned 1 users") } - rusers, resp = Client.AutocompleteUsers("", "") + rusers, resp = Client.AutocompleteUsers("", model.USER_SEARCH_DEFAULT_LIMIT, "") CheckNoError(t, resp) if len(rusers.Users) < 2 { t.Fatal("should have returned many users") } - rusers, resp = Client.AutocompleteUsersInTeam(teamId, "amazonses", "") + rusers, resp = Client.AutocompleteUsersInTeam(teamId, "amazonses", model.USER_SEARCH_DEFAULT_LIMIT, "") CheckNoError(t, resp) if len(rusers.Users) != 0 { t.Fatal("should have returned 0 users") } - rusers, resp = Client.AutocompleteUsersInTeam(teamId, "", "") + rusers, resp = Client.AutocompleteUsersInTeam(teamId, "", model.USER_SEARCH_DEFAULT_LIMIT, "") CheckNoError(t, resp) if len(rusers.Users) < 2 { t.Fatal("should have many users") } Client.Logout() - _, resp = Client.AutocompleteUsersInChannel(teamId, channelId, username, "") + _, resp = Client.AutocompleteUsersInChannel(teamId, channelId, username, model.USER_SEARCH_DEFAULT_LIMIT, "") CheckUnauthorizedStatus(t, resp) - _, resp = Client.AutocompleteUsersInTeam(teamId, username, "") + _, resp = Client.AutocompleteUsersInTeam(teamId, username, model.USER_SEARCH_DEFAULT_LIMIT, "") CheckUnauthorizedStatus(t, resp) - _, resp = Client.AutocompleteUsers(username, "") + _, resp = Client.AutocompleteUsers(username, model.USER_SEARCH_DEFAULT_LIMIT, "") CheckUnauthorizedStatus(t, resp) user := th.CreateUser() Client.Login(user.Email, user.Password) - _, resp = Client.AutocompleteUsersInChannel(teamId, channelId, username, "") + _, resp = Client.AutocompleteUsersInChannel(teamId, channelId, username, model.USER_SEARCH_DEFAULT_LIMIT, "") CheckForbiddenStatus(t, resp) - _, resp = Client.AutocompleteUsersInTeam(teamId, username, "") + _, resp = Client.AutocompleteUsersInTeam(teamId, username, model.USER_SEARCH_DEFAULT_LIMIT, "") CheckForbiddenStatus(t, resp) - _, resp = Client.AutocompleteUsers(username, "") + _, resp = Client.AutocompleteUsers(username, model.USER_SEARCH_DEFAULT_LIMIT, "") CheckNoError(t, resp) - _, resp = th.SystemAdminClient.AutocompleteUsersInChannel(teamId, channelId, username, "") + _, resp = th.SystemAdminClient.AutocompleteUsersInChannel(teamId, channelId, username, model.USER_SEARCH_DEFAULT_LIMIT, "") CheckNoError(t, resp) - _, resp = th.SystemAdminClient.AutocompleteUsersInTeam(teamId, username, "") + _, resp = th.SystemAdminClient.AutocompleteUsersInTeam(teamId, username, model.USER_SEARCH_DEFAULT_LIMIT, "") CheckNoError(t, resp) - _, resp = th.SystemAdminClient.AutocompleteUsers(username, "") + _, resp = th.SystemAdminClient.AutocompleteUsers(username, model.USER_SEARCH_DEFAULT_LIMIT, "") CheckNoError(t, resp) // Check against privacy config settings @@ -852,21 +852,21 @@ func TestAutocompleteUsers(t *testing.T) { th.LoginBasic() - rusers, resp = Client.AutocompleteUsers(username, "") + rusers, resp = Client.AutocompleteUsers(username, model.USER_SEARCH_DEFAULT_LIMIT, "") CheckNoError(t, resp) if rusers.Users[0].FirstName != "" || rusers.Users[0].LastName != "" { t.Fatal("should not show first/last name") } - rusers, resp = Client.AutocompleteUsersInChannel(teamId, channelId, username, "") + rusers, resp = Client.AutocompleteUsersInChannel(teamId, channelId, username, model.USER_SEARCH_DEFAULT_LIMIT, "") CheckNoError(t, resp) if rusers.Users[0].FirstName != "" || rusers.Users[0].LastName != "" { t.Fatal("should not show first/last name") } - rusers, resp = Client.AutocompleteUsersInTeam(teamId, username, "") + rusers, resp = Client.AutocompleteUsersInTeam(teamId, username, model.USER_SEARCH_DEFAULT_LIMIT, "") CheckNoError(t, resp) if rusers.Users[0].FirstName != "" || rusers.Users[0].LastName != "" { @@ -874,7 +874,7 @@ func TestAutocompleteUsers(t *testing.T) { } t.Run("user must have access to team id, especially when it does not match channel's team id", func(t *testing.T) { - rusers, resp = Client.AutocompleteUsersInChannel("otherTeamId", channelId, username, "") + rusers, resp = Client.AutocompleteUsersInChannel("otherTeamId", channelId, username, model.USER_SEARCH_DEFAULT_LIMIT, "") CheckErrorMessage(t, resp, "api.context.permissions.app_error") }) } |