pr review

Signed-off-by: Gregor Zeitlinger <gregor.zeitlinger@grafana.com>

Author: zeitlinger

prometheus-metrics-core/src/test/java/io/prometheus/metrics/core/metrics/HistogramTest.java

@@ -1384,7 +1384,7 @@ void testObserve() {
 
   @Test
   // See https://github.com/prometheus/client_java/issues/646
-  public void testNegativeAmount() {
+  void testNegativeAmount() {
     Histogram histogram =
         Histogram.builder()
             .name("histogram")
@@ -1527,7 +1527,7 @@ void testObserveMultithreaded()
   }
 
   @Test
-  public void testNativeResetDuration() {
+  void testNativeResetDuration() {
     // Test that nativeResetDuration can be configured without error and the histogram
     // functions correctly. The reset duration schedules internal reset behavior but
     // is not directly observable in the snapshot.
@@ -1551,7 +1551,7 @@ public void testNativeResetDuration() {
   }
 
   @Test
-  public void testNativeResetDurationNegativeValue() {
+  void testNativeResetDurationNegativeValue() {
     assertThatExceptionOfType(IllegalArgumentException.class)
         .isThrownBy(
             () ->
@@ -1564,7 +1564,7 @@ public void testNativeResetDurationNegativeValue() {
   }
 
   @Test
-  public void testNativeResetDurationZeroValue() {
+  void testNativeResetDurationZeroValue() {
     assertThatExceptionOfType(IllegalArgumentException.class)
         .isThrownBy(
             () ->
@@ -1577,7 +1577,7 @@ public void testNativeResetDurationZeroValue() {
   }
 
   @Test
-  public void testNativeResetDurationSubSecond() {
+  void testNativeResetDurationSubSecond() {
     // Sub-second durations should be rejected as they truncate to 0 seconds
     assertThatExceptionOfType(IllegalArgumentException.class)
         .isThrownBy(