From 1ea934684bfd38894a06bf63f58b12639e250a6a Mon Sep 17 00:00:00 2001 From: Cosmin Rentea Date: Wed, 5 Apr 2017 16:24:01 +0300 Subject: [PATCH] fixing implementation of configstring List String() ; fixing test --- server/config_test.go | 2 +- server/configstring/configstringlist.go | 8 ++++++-- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/server/config_test.go b/server/config_test.go index d5334afd..ed1f631f 100644 --- 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 index 45ea66e6..4111d29e 100644 --- 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