MeteredDecoder records timer once in happy path (#1554)

Author: vitalijr2

micrometer/src/main/java/feign/micrometer/MeteredDecoder.java

@@ -58,7 +58,6 @@ public Object decode(Response response, Type type) throws IOException, FeignExce
     try {
       decoded = decoder.decode(meteredResponse, type);
       timer = createTimer(response, type, null);
-      sample.stop(timer);
     } catch (IOException | RuntimeException e) {
       timer = createTimer(response, type, e);
       createExceptionCounter(response, type, e).count();

micrometer/src/test/java/feign/micrometer/AbstractMetricsTestBase.java

@@ -86,6 +86,15 @@ public final void addMetricsCapability() {
     clientMetrics.values().stream()
         .filter(this::doesMetricHasCounter)
         .forEach(metric -> assertEquals(1, getMetricCounter(metric)));
+
+    final Map<METRIC_ID, METRIC> decoderMetrics =
+        getFeignMetrics().entrySet().stream()
+            .filter(entry -> isDecoderMetric(entry.getKey()))
+            .collect(Collectors.toMap(Map.Entry::getKey, Map.Entry::getValue));
+
+    decoderMetrics.values().stream()
+        .filter(this::doesMetricHasCounter)
+        .forEach(metric -> assertEquals(1, getMetricCounter(metric)));
   }
 
   protected abstract boolean doesMetricIncludeHost(METRIC_ID metricId);