Fix adding user to team and add test check (#5172)

mattermost · Jan 23, 2017 · 58b3c76 · 58b3c76
1 parent d1f0cac
commit 58b3c76
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/api/team.go b/api/team.go
@@ -153,7 +153,7 @@ func addUserToTeam(c *Context, w http.ResponseWriter, r *http.Request) {
 		return
 	}
 
-	if _, err := app.AddUserToTeam(c.TeamId, c.Session.UserId); err != nil {
+	if _, err := app.AddUserToTeam(c.TeamId, userId); err != nil {
 		c.Err = err
 		return
 	}

diff --git a/api/team_test.go b/api/team_test.go
@@ -79,6 +79,10 @@ func TestAddUserToTeam(t *testing.T) {
 			t.Fatal("ids didn't match")
 		}
 	}
+
+	if _, err := th.BasicClient.GetTeamMember(th.BasicTeam.Id, user2.Id); err != nil {
+		t.Fatal(err)
+	}
 }
 
 func TestRemoveUserFromTeam(t *testing.T) {