fixing implementation of configstring List String() ; fixing test

cosminrentea · Apr 5, 2017 · 1ea9346 · 1ea9346
1 parent d4e743c
commit 1ea9346
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/server/config_test.go b/server/config_test.go
@@ -196,7 +196,7 @@ func assertArguments(a *assert.Assertions) {
 	a.Equal("dev", *Config.EnvName)
 	a.Equal("mem", *Config.Profile)
 
-	a.Equal("[ 127.0.0.1:9092 127.0.0.1:9091]", (*Config.KafkaProducer.Brokers).String())
+	a.Equal("[127.0.0.1:9092 127.0.0.1:9091]", (*Config.KafkaProducer.Brokers).String())
 	a.Equal("sms_reporting_topic", *Config.SMS.KafkaReportingTopic)
 
 	assertClusterRemotes(a)

diff --git a/server/configstring/configstringlist.go b/server/configstring/configstringlist.go
@@ -34,8 +34,12 @@ func (sl *List) IsEmpty() bool {
 
 func (sl List) String() string {
 	res := "["
-	for _, s := range sl {
-		res = res + " " + s
+	for i, s := range sl {
+		if i == 0 {
+			res = res + s
+		} else {
+			res = res + " " + s
+		}
 	}
 	res = res + "]"
 	return res