Merge pull request #44 from knweiss/golint

Trivial golint fixes
grpc-ecosystem · Apr 18, 2018 · b089b26 · b089b26
2 parents 5f56f5a + 83d16be
commit b089b26
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/client_metrics.go b/client_metrics.go
@@ -119,7 +119,7 @@ func (m *ClientMetrics) UnaryClientInterceptor() func(ctx context.Context, metho
 	}
 }
 
-// StreamServerInterceptor is a gRPC client-side interceptor that provides Prometheus monitoring for Streaming RPCs.
+// StreamClientInterceptor is a gRPC client-side interceptor that provides Prometheus monitoring for Streaming RPCs.
 func (m *ClientMetrics) StreamClientInterceptor() func(ctx context.Context, desc *grpc.StreamDesc, cc *grpc.ClientConn, method string, streamer grpc.Streamer, opts ...grpc.CallOption) (grpc.ClientStream, error) {
 	return func(ctx context.Context, desc *grpc.StreamDesc, cc *grpc.ClientConn, method string, streamer grpc.Streamer, opts ...grpc.CallOption) (grpc.ClientStream, error) {
 		monitor := newClientReporter(m, clientStreamType(desc), method)

diff --git a/client_test.go b/client_test.go
@@ -210,7 +210,7 @@ func (s *ClientInterceptorTestSuite) TestStreamingIncrementsMessageCounts() {
 			break
 		}
 		require.NoError(s.T(), err, "reading pingList shouldn't fail")
-		count += 1
+		count++
 	}
 	require.EqualValues(s.T(), countListResponses, count, "Number of received msg on the wire must match")
 	afterSent := sumCountersForMetricAndLabels(s.T(), "grpc_client_msg_sent_total", "PingList", "server_stream")

diff --git a/server_test.go b/server_test.go
@@ -231,7 +231,7 @@ func (s *ServerInterceptorTestSuite) TestStreamingIncrementsMessageCounts() {
 			break
 		}
 		require.NoError(s.T(), err, "reading pingList shouldn't fail")
-		count += 1
+		count++
 	}
 	require.EqualValues(s.T(), countListResponses, count, "Number of received msg on the wire must match")
 	afterSent := sumCountersForMetricAndLabels(s.T(), "grpc_server_msg_sent_total", "PingList", "server_stream")