53 lines
3.0 KiB
Diff
53 lines
3.0 KiB
Diff
From b77cf4d0b067438c3e1758bb22ad56e644c8122f Mon Sep 17 00:00:00 2001
|
|
From: Fernandez Ludovic <ludovic@containo.us>
|
|
Date: Thu, 13 Aug 2020 19:15:10 +0200
|
|
Subject: [PATCH] fix: conversion from untyped int to string yields a string of
|
|
one rune
|
|
|
|
---
|
|
pkg/metrics/statsd_test.go | 13 +++++++------
|
|
1 file changed, 7 insertions(+), 6 deletions(-)
|
|
|
|
diff --git a/pkg/metrics/statsd_test.go b/pkg/metrics/statsd_test.go
|
|
index 11955a50aa..64a585609d 100644
|
|
--- a/pkg/metrics/statsd_test.go
|
|
+++ b/pkg/metrics/statsd_test.go
|
|
@@ -3,6 +3,7 @@ package metrics
|
|
import (
|
|
"context"
|
|
"net/http"
|
|
+ "strconv"
|
|
"testing"
|
|
"time"
|
|
|
|
@@ -36,11 +37,11 @@ func TestStatsD(t *testing.T) {
|
|
}
|
|
|
|
udp.ShouldReceiveAll(t, expected, func() {
|
|
- statsdRegistry.ServiceReqsCounter().With("service", "test", "code", string(http.StatusOK), "method", http.MethodGet).Add(1)
|
|
- statsdRegistry.ServiceReqsCounter().With("service", "test", "code", string(http.StatusNotFound), "method", http.MethodGet).Add(1)
|
|
+ statsdRegistry.ServiceReqsCounter().With("service", "test", "code", strconv.Itoa(http.StatusOK), "method", http.MethodGet).Add(1)
|
|
+ statsdRegistry.ServiceReqsCounter().With("service", "test", "code", strconv.Itoa(http.StatusNotFound), "method", http.MethodGet).Add(1)
|
|
statsdRegistry.ServiceRetriesCounter().With("service", "test").Add(1)
|
|
statsdRegistry.ServiceRetriesCounter().With("service", "test").Add(1)
|
|
- statsdRegistry.ServiceReqDurationHistogram().With("service", "test", "code", string(http.StatusOK)).Observe(10000)
|
|
+ statsdRegistry.ServiceReqDurationHistogram().With("service", "test", "code", strconv.Itoa(http.StatusOK)).Observe(10000)
|
|
statsdRegistry.ConfigReloadsCounter().Add(1)
|
|
statsdRegistry.ConfigReloadsFailureCounter().Add(1)
|
|
statsdRegistry.EntryPointReqsCounter().With("entrypoint", "test").Add(1)
|
|
@@ -76,11 +77,11 @@ func TestStatsDWithPrefix(t *testing.T) {
|
|
}
|
|
|
|
udp.ShouldReceiveAll(t, expected, func() {
|
|
- statsdRegistry.ServiceReqsCounter().With("service", "test", "code", string(http.StatusOK), "method", http.MethodGet).Add(1)
|
|
- statsdRegistry.ServiceReqsCounter().With("service", "test", "code", string(http.StatusNotFound), "method", http.MethodGet).Add(1)
|
|
+ statsdRegistry.ServiceReqsCounter().With("service", "test", "code", strconv.Itoa(http.StatusOK), "method", http.MethodGet).Add(1)
|
|
+ statsdRegistry.ServiceReqsCounter().With("service", "test", "code", strconv.Itoa(http.StatusNotFound), "method", http.MethodGet).Add(1)
|
|
statsdRegistry.ServiceRetriesCounter().With("service", "test").Add(1)
|
|
statsdRegistry.ServiceRetriesCounter().With("service", "test").Add(1)
|
|
- statsdRegistry.ServiceReqDurationHistogram().With("service", "test", "code", string(http.StatusOK)).Observe(10000)
|
|
+ statsdRegistry.ServiceReqDurationHistogram().With("service", "test", "code", strconv.Itoa(http.StatusOK)).Observe(10000)
|
|
statsdRegistry.ConfigReloadsCounter().Add(1)
|
|
statsdRegistry.ConfigReloadsFailureCounter().Add(1)
|
|
statsdRegistry.EntryPointReqsCounter().With("entrypoint", "test").Add(1)
|