[chore] add unit test for histogram in mdatagen (open-telemetry#10089)

Signed-off-by: Alex Boten <[email protected]>
andrzej-stencel · May 27, 2024 · e599780 · e599780
1 parent b497ace
commit e599780
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/cmd/mdatagen/metricdata.go b/cmd/mdatagen/metricdata.go
@@ -209,11 +209,11 @@ func (d histogram) Type() string {
 }
 
 func (d histogram) HasMonotonic() bool {
-	return true
+	return false
 }
 
 func (d histogram) HasAggregated() bool {
-	return true
+	return false
 }
 
 func (d histogram) Instrument() string {

diff --git a/cmd/mdatagen/metricdata_test.go b/cmd/mdatagen/metricdata_test.go
@@ -18,6 +18,7 @@ func TestMetricData(t *testing.T) {
 	}{
 		{&gauge{}, "Gauge", false, false},
 		{&sum{}, "Sum", true, true},
+		{&histogram{}, "Histogram", false, false},
 	} {
 		assert.Equal(t, arg.typ, arg.metricData.Type())
 		assert.Equal(t, arg.hasAggregated, arg.metricData.HasAggregated())