Added strings.TrimSuffix to Master URL call

mesos/cluster.go

@@ -7,6 +7,7 @@ import (
 	"log"
 	"net/http"
 	"net/url"
+	"strings"
 	"time"
 
 	"github.com/cloudflare/complainer"
@@ -27,8 +28,13 @@ type Cluster struct {
 
 // NewCluster creates a new cluster with the provided list of masters
 func NewCluster(masters []string) *Cluster {
+	var cleanMasters []string
+	for _, master := range masters {
+		cleanMasters = append(cleanMasters, strings.TrimSuffix(master, "/"))
+	}
+
 	return &Cluster{
-		masters: masters,
+		masters: cleanMasters,
 		client: http.Client{
 			Timeout: time.Second * 30,
 		},

mesos/cluster_test.go

@@ -0,0 +1,30 @@
+package mesos
+
+import (
+	"reflect"
+	"testing"
+)
+
+func TestNewCluster(t *testing.T) {
+	inputMasters := []string{
+		"http://master1.com",
+		"http://master2.com/",
+		"http://master3.com/fancy/path",
+		"http://master4.com/fancy/path/",
+	}
+	expectedMasters := []string{
+		"http://master1.com",
+		"http://master2.com",
+		"http://master3.com/fancy/path",
+		"http://master4.com/fancy/path",
+	}
+
+	cluster := NewCluster(inputMasters)
+	if cluster == nil {
+		t.Error("Cluster is nil. Expected mesos.Cluster pointer.")
+	}
+
+	if !reflect.DeepEqual(cluster.masters, expectedMasters) {
+		t.Errorf("Master list is not equal. Got %+v, expected %+v", cluster.masters, expectedMasters)
+	}
+}