diff options
author | Joram Wilander <jwawilander@gmail.com> | 2018-03-21 11:33:47 -0400 |
---|---|---|
committer | Saturnino Abril <saturnino.abril@gmail.com> | 2018-03-21 23:33:47 +0800 |
commit | b1b23079c6a49df29b6f27b85e98d6a9b1d3607c (patch) | |
tree | c9ff8cf7f54be9d5f8578c5ce563cff932d1e24e /api4/team_test.go | |
parent | b4784be2de6b841529e3dc3ec4c1ae1aade0121a (diff) | |
download | chat-b1b23079c6a49df29b6f27b85e98d6a9b1d3607c.tar.gz chat-b1b23079c6a49df29b6f27b85e98d6a9b1d3607c.tar.bz2 chat-b1b23079c6a49df29b6f27b85e98d6a9b1d3607c.zip |
Fix paging for GET /teams and GET /teams/{id}/members endpoints (#8488)
Diffstat (limited to 'api4/team_test.go')
-rw-r--r-- | api4/team_test.go | 26 |
1 files changed, 26 insertions, 0 deletions
diff --git a/api4/team_test.go b/api4/team_test.go index 04a0e9ae4..c2edcdaaa 100644 --- a/api4/team_test.go +++ b/api4/team_test.go @@ -567,6 +567,10 @@ func TestGetAllTeams(t *testing.T) { _, resp := Client.CreateTeam(team) CheckNoError(t, resp) + team2 := &model.Team{DisplayName: "Name2", Name: GenerateTestTeamName(), Email: th.GenerateTestEmail(), Type: model.TEAM_OPEN, AllowOpenInvite: true} + _, resp = Client.CreateTeam(team2) + CheckNoError(t, resp) + rrteams, resp := Client.GetAllTeams("", 0, 1) CheckNoError(t, resp) @@ -611,6 +615,17 @@ func TestGetAllTeams(t *testing.T) { t.Fatal("wrong number of teams - should be 0") } + rrteams, resp = Client.GetAllTeams("", 0, 2) + CheckNoError(t, resp) + rrteams2, resp = Client.GetAllTeams("", 1, 2) + CheckNoError(t, resp) + + for _, t1 := range rrteams { + for _, t2 := range rrteams2 { + assert.NotEqual(t, t1.Id, t2.Id, "different pages should not have the same teams") + } + } + Client.Logout() _, resp = Client.GetAllTeams("", 1, 10) CheckUnauthorizedStatus(t, resp) @@ -1140,6 +1155,17 @@ func TestGetTeamMembers(t *testing.T) { t.Fatal("should be no member") } + rmembers, resp = Client.GetTeamMembers(team.Id, 0, 2, "") + CheckNoError(t, resp) + rmembers2, resp := Client.GetTeamMembers(team.Id, 1, 2, "") + CheckNoError(t, resp) + + for _, tm1 := range rmembers { + for _, tm2 := range rmembers2 { + assert.NotEqual(t, tm1.UserId+tm1.TeamId, tm2.UserId+tm2.TeamId, "different pages should not have the same members") + } + } + _, resp = Client.GetTeamMembers("junk", 0, 100, "") CheckBadRequestStatus(t, resp) |