/**
 * Copyright 2017 Pivotal Software, Inc.
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *     http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */
package org.springframework.metrics.instrument.binder;

import org.junit.jupiter.api.Test;
import org.springframework.metrics.instrument.Gauge;
import org.springframework.metrics.instrument.MeterRegistry;
import org.springframework.metrics.instrument.simple.SimpleMeterRegistry;

import java.util.concurrent.Executors;
import java.util.concurrent.atomic.AtomicInteger;

import static org.assertj.core.api.AssertionsForClassTypes.assertThat;

class ThreadMetricsTest {
    @Test
    void threadMetrics() {
        MeterRegistry registry = new SimpleMeterRegistry();
        new ThreadMetrics().bindTo(registry);

        assertThat(registry.findMeter(Gauge.class, "threads_live"))
                .hasValueSatisfying(g -> assertThat(g.value()).isGreaterThan(0));
        assertThat(registry.findMeter(Gauge.class, "threads_daemon"))
                .hasValueSatisfying(g -> assertThat(g.value()).isGreaterThan(0));

        AtomicInteger peak = new AtomicInteger(0);
        assertThat(registry.findMeter(Gauge.class, "threads_peak"))
                .hasValueSatisfying(g -> {
                    peak.set((int) g.value());
                    assertThat(g.value()).isGreaterThan(0);
                });

        // should bump the peak by one
        Executors.newSingleThreadExecutor().submit(() -> {
            // do nothing
        });

        assertThat(registry.findMeter(Gauge.class, "threads_peak"))
                .hasValueSatisfying(g -> assertThat(g.value()).isGreaterThan(peak.get()));
    }
}

@@ -884,98 +884,8 @@
 y;%0A%0A
-import java.util.concurrent.Executors;%0Aimport java.util.concurrent.atomic.AtomicInteger;%0A%0A
 impo
@@ -951,16 +951,16 @@
 rtThat;%0A
+
 %0Aclass T
@@ -1427,60 +1427,8 @@
 0));
-%0A%0A        AtomicInteger peak = new AtomicInteger(0);
 %0A   
@@ -1537,430 +1537,46 @@
  -%3E 
-%7B%0A                    peak.set((int) g.value());%0A                    assertThat(g.value()).isGreaterThan(0);%0A                %7D);%0A%0A        // should bump the peak by one%0A        Executors.newSingleThreadExecutor().submit(() -%3E %7B%0A            // do nothing%0A        %7D);%0A%0A        assertThat(registry.findMeter(Gauge.class, %22threads_peak%22))%0A                .hasValueSatisfying(g -%3E assertThat(g.value()).isGreaterThan(peak.get()
+ assertThat(g.value()).isGreaterThan(0
 ));%0A

Fix flaky ThreadMetricsTest