Remove comments

prometheus · Mar 7, 2024 · 7707188 · 7707188
1 parent 4f6f26e
commit 7707188
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 2 deletions.
diff --git a/...-formats/src/test/java/io/prometheus/metrics/expositionformats/ExpositionFormatsTest.java b/...-formats/src/test/java/io/prometheus/metrics/expositionformats/ExpositionFormatsTest.java
@@ -1869,7 +1869,6 @@ public void testFindWriter() {
         writer = expositionFormats.findWriter(acceptHeaderValue);
         Assert.assertEquals(expectedFmt, writer.getContentType() + escapingScheme.toHeaderFormat());
 
-        // TODO review if this is ok
         nameEscapingScheme = EscapingScheme.VALUE_ENCODING_ESCAPING;
 
         // OM format, no version

diff --git a/...trics-model/src/test/java/io/prometheus/metrics/model/snapshots/PrometheusNamingTest.java b/...trics-model/src/test/java/io/prometheus/metrics/model/snapshots/PrometheusNamingTest.java
@@ -120,7 +120,6 @@ public void testEscapeName() {
         got = unescapeName(got, EscapingScheme.VALUE_ENCODING_ESCAPING);
         Assert.assertEquals("no:escaping_required", got);
 
-        // TODO should add test for legacy validation?
         // name with dots
         got = escapeName("mysystem.prod.west.cpu.load", EscapingScheme.UNDERSCORE_ESCAPING);
         Assert.assertEquals("mysystem_prod_west_cpu_load", got);