From 1885d2ec2ab6870b735f08752f5354e9f624d87f Mon Sep 17 00:00:00 2001 From: JoramWilander Date: Tue, 8 Aug 2017 16:20:07 -0400 Subject: Minor updates to team --- api/team_test.go | 24 ++++++++++++++++++------ 1 file changed, 18 insertions(+), 6 deletions(-) (limited to 'api/team_test.go') diff --git a/api/team_test.go b/api/team_test.go index e09bf42ef..14c9311a3 100644 --- a/api/team_test.go +++ b/api/team_test.go @@ -787,12 +787,15 @@ func TestGetTeamByName(t *testing.T) { th := Setup().InitSystemAdmin().InitBasic() Client := th.BasicClient - team := &model.Team{DisplayName: "Name", Name: "z-z-" + model.NewId() + "a", Email: "success+" + model.NewId() + "@simulator.amazonses.com", Type: model.TEAM_INVITE} + team := &model.Team{DisplayName: "Name", Name: "z-z-" + model.NewId() + "a", Email: "success+" + model.NewId() + "@simulator.amazonses.com", Type: model.TEAM_OPEN, AllowOpenInvite: false} team = Client.Must(Client.CreateTeam(team)).Data.(*model.Team) - team2 := &model.Team{DisplayName: "Name", Name: "z-z-" + model.NewId() + "a", Email: "success+" + model.NewId() + "@simulator.amazonses.com", Type: model.TEAM_OPEN} + team2 := &model.Team{DisplayName: "Name", Name: "z-z-" + model.NewId() + "a", Email: "success+" + model.NewId() + "@simulator.amazonses.com", Type: model.TEAM_OPEN, AllowOpenInvite: true} team2 = Client.Must(Client.CreateTeam(team2)).Data.(*model.Team) + team3 := &model.Team{DisplayName: "Name", Name: "z-z-" + model.NewId() + "a", Email: "success+" + model.NewId() + "@simulator.amazonses.com", Type: model.TEAM_INVITE, AllowOpenInvite: true} + team3 = Client.Must(Client.CreateTeam(team3)).Data.(*model.Team) + if _, err := Client.GetTeamByName(team.Name); err != nil { t.Fatal("Failed to get team") } @@ -813,7 +816,7 @@ func TestGetTeamByName(t *testing.T) { Client.Login(user2.Email, "passwd1") - // TEAM_INVITE and user is not part of the team + // AllowInviteOpen is false and team is open and user is not part of the team if _, err := Client.GetTeamByName(team.Name); err == nil { t.Fatal("Should fail dont have permissions to get the team") } @@ -822,21 +825,30 @@ func TestGetTeamByName(t *testing.T) { t.Fatal("Should not exist this team") } - // TEAM_OPEN and user is not part of the team + // AllowInviteOpen is true and is open and user is not part of the team if _, err := Client.GetTeamByName(team2.Name); err != nil { t.Fatal("Should not fail team is open") } + // AllowInviteOpen is true and is invite only and user is not part of the team + if _, err := Client.GetTeamByName(team3.Name); err == nil { + t.Fatal("Should fail team is invite only") + } + Client.Must(Client.Logout()) th.BasicClient.Logout() th.LoginSystemAdmin() if _, err := th.SystemAdminClient.GetTeamByName(team.Name); err != nil { - t.Fatal("Should not failed to get team the user is admin") + t.Fatal("Should not fail to get team the user is admin") } if _, err := th.SystemAdminClient.GetTeamByName(team2.Name); err != nil { - t.Fatal("Should not failed to get team the user is admin and team is open") + t.Fatal("Should not fail to get team the user is admin and team is open") + } + + if _, err := th.SystemAdminClient.GetTeamByName(team3.Name); err != nil { + t.Fatal("Should not fail to get team the user is admin and team is invite") } if _, err := Client.GetTeamByName("InvalidTeamName"); err == nil { -- cgit v1.2.3-1-g7c22 From 638c38cc0d2296335a0fbd5bde8b6d2cbf9f9062 Mon Sep 17 00:00:00 2001 From: George Goldberg Date: Fri, 11 Aug 2017 18:20:10 +0100 Subject: PLT-7336: Fix team API endpoints. --- api/team_test.go | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+) (limited to 'api/team_test.go') diff --git a/api/team_test.go b/api/team_test.go index 14c9311a3..3c05588ce 100644 --- a/api/team_test.go +++ b/api/team_test.go @@ -239,6 +239,11 @@ func TestGetAllTeams(t *testing.T) { } else if receivedTeam, ok := teams[team.Id]; !ok || receivedTeam.Id != team.Id { t.Fatal("admin should've received team that they aren't a member of") } + + Client.Logout() + if _, err := Client.GetAllTeams(); err == nil { + t.Fatal("Should have failed due to not being logged in.") + } } func TestGetAllTeamListings(t *testing.T) { @@ -855,4 +860,18 @@ func TestGetTeamByName(t *testing.T) { t.Fatal("Should not exist this team") } + Client.Logout() + if _, err := Client.GetTeamByName(th.BasicTeam.Name); err == nil { + t.Fatal("Should have failed when not logged in.") + } +} + +func TestFindTeamByName(t *testing.T) { + th := Setup().InitBasic() + Client := th.BasicClient + Client.Logout() + + if _, err := Client.FindTeamByName(th.BasicTeam.Name); err == nil { + t.Fatal("Should have failed when not logged in.") + } } -- cgit v1.2.3-1-g7c22