PLT-4138 Fix /join command for non-admin accounts (#4265)

* Fix /join command for non-admin accounts

* Implement found check in join command unit test properly

2 files changed, 10 insertions, 3 deletions

diff --git a/api/command_join.go b/api/command_join.go
index dddba0135..b8c863425 100644
--- a/api/command_join.go
+++ b/api/command_join.go
@@ -46,7 +46,7 @@ func (me *JoinProvider) DoCommand(c *Context, channelId string, message string)
 					return &model.CommandResponse{Text: c.T("api.command_join.fail.app_error"), ResponseType: model.COMMAND_RESPONSE_TYPE_EPHEMERAL}
 				}
 
-				if err, _ := JoinChannelById(c, c.Session.UserId, channelId); err != nil {
+				if err, _ := JoinChannelById(c, c.Session.UserId, v.Id); err != nil {
 					return &model.CommandResponse{Text: c.T("api.command_join.fail.app_error"), ResponseType: model.COMMAND_RESPONSE_TYPE_EPHEMERAL}
 				}
 
diff --git a/api/command_join_test.go b/api/command_join_test.go
index d586a522a..6cf474c6b 100644
--- a/api/command_join_test.go
+++ b/api/command_join_test.go
@@ -41,7 +41,14 @@ func TestJoinCommands(t *testing.T) {
 
 	c1 := Client.Must(Client.GetChannels("")).Data.(*model.ChannelList)
 
-	if len(c1.Channels) != 5 {
-		t.Fatal("didn't join channel")
+	found := false
+	for _, c := range c1.Channels {
+		if c.Id == channel2.Id {
+			found = true
+		}
+	}
+
+	if !found {
+		t.Fatal("did not join channel")
 	}
 }