PrometheusNaming changes were incompatible - so deprecating instead

Signed-off-by: Gregor Zeitlinger <[email protected]>

Author: zeitlinger

prometheus-metrics-model/src/test/java/io/prometheus/metrics/model/snapshots/PrometheusNamesTest.java

@@ -204,4 +204,18 @@ void testValidMetricName() {
     assertThat(PrometheusNames.isValidMetricName("invalid_metric_name_total")).isFalse();
     assertThat(PrometheusNames.isValidMetricName("0abc.def")).isTrue();
   }
+
+  @Test
+  void testValidLabelName() {
+    assertThat(PrometheusNames.isValidLabelName("valid_label_name")).isTrue();
+    assertThat(PrometheusNames.isValidLabelName("0invalid_label_name")).isTrue();
+    assertThat(PrometheusNames.isValidLabelName("invalid-label-name")).isTrue();
+  }
+
+  @Test
+  void testValidUnitName() {
+    assertThat(PrometheusNames.isValidUnitName("seconds")).isTrue();
+    assertThat(PrometheusNames.isValidUnitName("seconds_total")).isFalse();
+    assertThat(PrometheusNames.isValidUnitName("2")).isTrue();
+  }
 }

prometheus-metrics-model/src/test/java/io/prometheus/metrics/model/snapshots/PrometheusNamingTest.java

@@ -126,4 +126,18 @@ void testValidMetricName() {
     assertThat(PrometheusNaming.isValidMetricName("invalid_metric_name_total")).isFalse();
     assertThat(PrometheusNaming.isValidMetricName("0abc.def")).isFalse();
   }
+
+  @Test
+  void testValidLabelName() {
+    assertThat(PrometheusNaming.isValidLabelName("valid_label_name")).isTrue();
+    assertThat(PrometheusNaming.isValidLabelName("0invalid_label_name")).isFalse();
+    assertThat(PrometheusNaming.isValidLabelName("invalid-label-name")).isFalse();
+  }
+
+  @Test
+  void testValidUnitName() {
+    assertThat(PrometheusNaming.isValidUnitName("seconds")).isTrue();
+    assertThat(PrometheusNaming.isValidUnitName("seconds_total")).isFalse();
+    assertThat(PrometheusNaming.isValidUnitName("2")).isTrue();
+  }
 }