Skip to content

Commit 32b1995

Browse files
committed
Use real histogram factory in MetricSerialiserTest
1 parent 2fa3c0c commit 32b1995

2 files changed

Lines changed: 18 additions & 0 deletions

File tree

dd-trace-core/src/test/groovy/datadog/trace/common/metrics/SerializingMetricWriterTest.groovy

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
package datadog.trace.common.metrics
22

3+
import datadog.metrics.api.Histograms
4+
import datadog.metrics.impl.DDSketchHistograms
35
import datadog.trace.api.Config
46
import datadog.trace.api.ProcessTags
57
import datadog.trace.api.WellKnownTags
@@ -17,6 +19,18 @@ import static java.util.concurrent.TimeUnit.MILLISECONDS
1719
import static java.util.concurrent.TimeUnit.SECONDS
1820

1921
class SerializingMetricWriterTest extends DDSpecification {
22+
23+
static Histograms.Factory originalHistogramFactory
24+
25+
def setupSpec() {
26+
originalHistogramFactory = Histograms.factory
27+
Histograms.register(DDSketchHistograms.FACTORY)
28+
}
29+
30+
def cleanupSpec() {
31+
Histograms.register(originalHistogramFactory)
32+
}
33+
2034
def "should produce correct message #iterationIndex with process tags enabled #withProcessTags" () {
2135
setup:
2236
if (withProcessTags) {

products/metrics/metrics-api/src/main/java/datadog/metrics/api/Histograms.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,10 @@ public static void register(Factory histograms) {
1616
}
1717
}
1818

19+
static Histograms.Factory getFactory() {
20+
return Histograms.factory;
21+
}
22+
1923
public interface Factory {
2024
Histogram newHistogram();
2125

0 commit comments

Comments
 (0)