Skip to content

Commit b4cfd14

Browse files
twz123brian-brazil
authored andcommitted
Add toString() to MapperConfig (prometheus#448)
Signed-off-by: Tom Wieczorek <t.wieczorek@yieldlab.de>
1 parent e1913a0 commit b4cfd14

2 files changed

Lines changed: 22 additions & 1 deletion

File tree

simpleclient_dropwizard/src/main/java/io/prometheus/client/dropwizard/samplebuilder/MapperConfig.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -86,6 +86,11 @@ public MapperConfig(final String match, final String name, final Map<String, Str
8686
this.labels = labels;
8787
}
8888

89+
@Override
90+
public String toString() {
91+
return String.format("MapperConfig{match=%s, name=%s, labels=%s}", match, name, labels);
92+
}
93+
8994
public String getMatch() {
9095
return match;
9196
}

simpleclient_dropwizard/src/test/java/io/prometheus/client/dropwizard/samplebuilder/MapperConfigTest.java

Lines changed: 17 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
import io.prometheus.client.dropwizard.samplebuilder.MapperConfig;
44
import org.junit.Test;
55

6+
import java.util.Collections;
67
import java.util.HashMap;
78
import java.util.Map;
89

@@ -39,4 +40,19 @@ public void setLabels_WHEN_ExpressionDoesnNotMatchPattern_ThrowException() {
3940
labels.put("not valid", "${0}");
4041
mapperConfig.setLabels(labels);
4142
}
42-
}
43+
44+
@Test
45+
public void toString_WHEN_EmptyConfig_AllGood() {
46+
final MapperConfig mapperConfig = new MapperConfig();
47+
assertEquals("MapperConfig{match=null, name=null, labels={}}", mapperConfig.toString());
48+
}
49+
50+
@Test
51+
public void toString_WHEN_FullyConfigured_AllGood() {
52+
final MapperConfig mapperConfig = new MapperConfig();
53+
mapperConfig.setMatch("com.company.meter.*.foo");
54+
mapperConfig.setName("foo");
55+
mapperConfig.setLabels(Collections.singletonMap("type", "${0}"));
56+
assertEquals("MapperConfig{match=com.company.meter.*.foo, name=foo, labels={type=${0}}}", mapperConfig.toString());
57+
}
58+
}

0 commit comments

Comments
 (0)