fix: NoopTimer#time(Runnable) should run the Runnable (#4541)

metrics-core/src/main/java/io/dropwizard/metrics5/NoopMetricRegistry.java

@@ -468,7 +468,7 @@ public <T> T timeSupplier(Supplier<T> event) {
          */
         @Override
         public void time(Runnable event) {
-            // NOP
+            event.run();
         }
 
         /**

metrics-core/src/test/java/io/dropwizard/metrics5/NoopMetricRegistryTest.java

@@ -6,6 +6,7 @@
 import java.util.HashMap;
 import java.util.Map;
 import java.util.Set;
+import java.util.concurrent.atomic.AtomicBoolean;
 
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatNullPointerException;
@@ -494,4 +495,13 @@ void registerNullMetric() {
                 .isThrownBy(() -> registry.register(MetricName.build("any_name"), null))
                 .withMessage("metric == null");
     }
+
+    @Test
+    void timesRunnableInstances() {
+        Timer timer = registry.timer("thing");
+        final AtomicBoolean called = new AtomicBoolean();
+        timer.time(() -> called.set(true));
+
+        assertThat(called).isTrue();
+    }
 }