[snowflake/master] Fixed small formatting errors of log output - tor-commits

28 Jun 2019

commit f779013b2d356cf67dda9d96145cbddd87497532
Author: Cecylia Bocovich cohosh@torproject.org
Date:   Fri Jun 14 17:09:06 2019 -0400
Fixed small formatting errors of log output
- removed trailing ","s
    - removed unecessary space before seconds
---
 broker/metrics.go               |  8 +++++++-
 broker/snowflake-broker_test.go | 14 +++++++-------
 2 files changed, 14 insertions(+), 8 deletions(-)

diff --git a/broker/metrics.go b/broker/metrics.go
index f37ec8a..a45179f 100644
--- a/broker/metrics.go
+++ b/broker/metrics.go
@@ -74,6 +74,12 @@ func (s CountryStats) Display() string {
    for cc, count := range s.counts {
    	output += fmt.Sprintf("%s=%d,", cc, count)
    }
+
+	// cut off trailing ","
+	if len(output) > 0 {
+		return output[:len(output)-1]
+	}
+
    return output
 }
@@ -163,7 +169,7 @@ func (m *Metrics) logMetrics() {
 }
func (m *Metrics) printMetrics() {
-	m.logger.Println("snowflake-stats-end", time.Now().UTC().Format("2006-01-02 15:04:05"), "(", int(metricsResolution.Seconds()), "s)")
+	m.logger.Println("snowflake-stats-end", time.Now().UTC().Format("2006-01-02 15:04:05"), fmt.Sprintf("(%d s)", int(metricsResolution.Seconds())))
    m.logger.Println("snowflake-ips", m.countryStats.Display())
    m.logger.Println("snowflake-idle-count", binCount(m.proxyIdleCount))
    m.logger.Println("client-denied-count", binCount(m.clientDeniedCount))
diff --git a/broker/snowflake-broker_test.go b/broker/snowflake-broker_test.go
index 021122e..bbbad3d 100644
--- a/broker/snowflake-broker_test.go
+++ b/broker/snowflake-broker_test.go
@@ -419,7 +419,7 @@ func TestMetrics(t *testing.T) {
    		<-done
ctx.metrics.printMetrics()
-			So(buf.String(), ShouldResemble, "snowflake-stats-end "+time.Now().UTC().Format("2006-01-02 15:04:05")+" ( 86400 s)\nsnowflake-ips CA=1,\nsnowflake-idle-count 8\nclient-denied-count 0\nclient-snowflake-match-count 0\n")
+			So(buf.String(), ShouldResemble, "snowflake-stats-end "+time.Now().UTC().Format("2006-01-02 15:04:05")+" (86400 s)\nsnowflake-ips CA=1\nsnowflake-idle-count 8\nclient-denied-count 0\nclient-snowflake-match-count 0\n")
    	})
//Test addition of client failures
@@ -432,13 +432,13 @@ func TestMetrics(t *testing.T) {
    		clientOffers(ctx, w, r)
ctx.metrics.printMetrics()
-			So(buf.String(), ShouldResemble, "snowflake-stats-end "+time.Now().UTC().Format("2006-01-02 15:04:05")+" ( 86400 s)\nsnowflake-ips \nsnowflake-idle-count 0\nclient-denied-count 8\nclient-snowflake-match-count 0\n")
+			So(buf.String(), ShouldResemble, "snowflake-stats-end "+time.Now().UTC().Format("2006-01-02 15:04:05")+" (86400 s)\nsnowflake-ips \nsnowflake-idle-count 0\nclient-denied-count 8\nclient-snowflake-match-count 0\n")
// Test reset
    		buf.Reset()
    		ctx.metrics.zeroMetrics()
    		ctx.metrics.printMetrics()
-			So(buf.String(), ShouldResemble, "snowflake-stats-end "+time.Now().UTC().Format("2006-01-02 15:04:05")+" ( 86400 s)\nsnowflake-ips \nsnowflake-idle-count 0\nclient-denied-count 0\nclient-snowflake-match-count 0\n")
+			So(buf.String(), ShouldResemble, "snowflake-stats-end "+time.Now().UTC().Format("2006-01-02 15:04:05")+" (86400 s)\nsnowflake-ips \nsnowflake-idle-count 0\nclient-denied-count 0\nclient-snowflake-match-count 0\n")
    	})
    	//Test addition of client matches
    	Convey("for client-proxy match", func() {
@@ -459,7 +459,7 @@ func TestMetrics(t *testing.T) {
    		<-done
ctx.metrics.printMetrics()
-			So(buf.String(), ShouldResemble, "snowflake-stats-end "+time.Now().UTC().Format("2006-01-02 15:04:05")+" ( 86400 s)\nsnowflake-ips \nsnowflake-idle-count 0\nclient-denied-count 0\nclient-snowflake-match-count 8\n")
+			So(buf.String(), ShouldResemble, "snowflake-stats-end "+time.Now().UTC().Format("2006-01-02 15:04:05")+" (86400 s)\nsnowflake-ips \nsnowflake-idle-count 0\nclient-denied-count 0\nclient-snowflake-match-count 8\n")
    	})
    	//Test rounding boundary
    	Convey("binning boundary", func() {
@@ -478,12 +478,12 @@ func TestMetrics(t *testing.T) {
    		clientOffers(ctx, w, r)
ctx.metrics.printMetrics()
-			So(buf.String(), ShouldResemble, "snowflake-stats-end "+time.Now().UTC().Format("2006-01-02 15:04:05")+" ( 86400 s)\nsnowflake-ips \nsnowflake-idle-count 0\nclient-denied-count 8\nclient-snowflake-match-count 0\n")
+			So(buf.String(), ShouldResemble, "snowflake-stats-end "+time.Now().UTC().Format("2006-01-02 15:04:05")+" (86400 s)\nsnowflake-ips \nsnowflake-idle-count 0\nclient-denied-count 8\nclient-snowflake-match-count 0\n")
clientOffers(ctx, w, r)
    		buf.Reset()
    		ctx.metrics.printMetrics()
-			So(buf.String(), ShouldResemble, "snowflake-stats-end "+time.Now().UTC().Format("2006-01-02 15:04:05")+" ( 86400 s)\nsnowflake-ips \nsnowflake-idle-count 0\nclient-denied-count 16\nclient-snowflake-match-count 0\n")
+			So(buf.String(), ShouldResemble, "snowflake-stats-end "+time.Now().UTC().Format("2006-01-02 15:04:05")+" (86400 s)\nsnowflake-ips \nsnowflake-idle-count 0\nclient-denied-count 16\nclient-snowflake-match-count 0\n")
    	})
//Test unique ip
@@ -515,7 +515,7 @@ func TestMetrics(t *testing.T) {
    		<-done
ctx.metrics.printMetrics()
-			So(buf.String(), ShouldResemble, "snowflake-stats-end "+time.Now().UTC().Format("2006-01-02 15:04:05")+" ( 86400 s)\nsnowflake-ips CA=1,\nsnowflake-idle-count 8\nclient-denied-count 0\nclient-snowflake-match-count 0\n")
+			So(buf.String(), ShouldResemble, "snowflake-stats-end "+time.Now().UTC().Format("2006-01-02 15:04:05")+" (86400 s)\nsnowflake-ips CA=1\nsnowflake-idle-count 8\nclient-denied-count 0\nclient-snowflake-match-count 0\n")
    	})
    })
 }

    