add already suffixed counter test

Signed-off-by: naseemkullah <naseem@transit.app>
open-telemetry · Mar 8, 2021 · 3bd2bd7 · 3bd2bd7
1 parent a52e16e
commit 3bd2bd7
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 0 deletions.
diff --git a/exporter/prometheusremotewriteexporter/helper_test.go b/exporter/prometheusremotewriteexporter/helper_test.go
@@ -289,6 +289,12 @@ func Test_getPromMetricName(t *testing.T) {
 			ns1,
 			"test_ns_" + validIntSum + counterSuffix,
 		},
+		{
+			"already_has_total_suffix",
+			validMetrics1[suffixedCounter],
+			ns1,
+			"test_ns_" + suffixedCounter,
+		},
 		{
 			"dirty_string",
 			validMetrics2[validIntGaugeDirty],

diff --git a/exporter/prometheusremotewriteexporter/testutil_test.go b/exporter/prometheusremotewriteexporter/testutil_test.go
@@ -94,6 +94,7 @@ var (
 	validIntHistogram    = "valid_IntHistogram"
 	validDoubleHistogram = "valid_DoubleHistogram"
 	validDoubleSummary   = "valid_DoubleSummary"
+	suffixedCounter      = "valid_IntSum_total"
 
 	validIntGaugeDirty = "*valid_IntGauge$"
 
@@ -136,6 +137,18 @@ var (
 				},
 			},
 		},
+		suffixedCounter: {
+			Name: suffixedCounter,
+			Data: &otlp.Metric_IntSum{
+				IntSum: &otlp.IntSum{
+					DataPoints: []*otlp.IntDataPoint{
+						getIntDataPoint(lbs1, intVal1, time1),
+						nil,
+					},
+					AggregationTemporality: otlp.AggregationTemporality_AGGREGATION_TEMPORALITY_CUMULATIVE,
+				},
+			},
+		},
 		validDoubleSum: {
 			Name: validDoubleSum,
 			Data: &otlp.Metric_DoubleSum{