diff --git a/spring-boot-project/spring-boot-actuator-autoconfigure/src/main/java/org/springframework/boot/actuate/autoconfigure/metrics/integration/IntegrationMetricsAutoConfiguration.java b/spring-boot-project/spring-boot-actuator-autoconfigure/src/main/java/org/springframework/boot/actuate/autoconfigure/metrics/integration/IntegrationMetricsAutoConfiguration.java deleted file mode 100644 index fb8b00e774a7..000000000000 --- a/spring-boot-project/spring-boot-actuator-autoconfigure/src/main/java/org/springframework/boot/actuate/autoconfigure/metrics/integration/IntegrationMetricsAutoConfiguration.java +++ /dev/null @@ -1,53 +0,0 @@ -/* - * Copyright 2012-2018 the original author or authors. - * - * 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.boot.actuate.autoconfigure.metrics.integration; - -import io.micrometer.core.instrument.MeterRegistry; - -import org.springframework.boot.actuate.autoconfigure.metrics.export.simple.SimpleMetricsExportAutoConfiguration; -import org.springframework.boot.autoconfigure.AutoConfigureAfter; -import org.springframework.boot.autoconfigure.EnableAutoConfiguration; -import org.springframework.boot.autoconfigure.condition.ConditionalOnBean; -import org.springframework.boot.autoconfigure.condition.ConditionalOnClass; -import org.springframework.boot.autoconfigure.integration.IntegrationAutoConfiguration; -import org.springframework.context.annotation.Bean; -import org.springframework.context.annotation.Configuration; -import org.springframework.integration.config.EnableIntegration; -import org.springframework.integration.support.management.micrometer.MicrometerMetricsFactory; - -/** - * {@link EnableAutoConfiguration Auto-configuration} for Spring Integration Micrometer - * support. - * - * @author Phillip Webb - * @author Gary Russell - * @since 2.0.0 - */ -@Configuration -@ConditionalOnClass({ EnableIntegration.class, MeterRegistry.class }) -@AutoConfigureAfter({ IntegrationAutoConfiguration.class, - SimpleMetricsExportAutoConfiguration.class }) -@ConditionalOnBean(MeterRegistry.class) -public class IntegrationMetricsAutoConfiguration { - - @Bean - public MicrometerMetricsFactory integrationMicrometerMetricsFactory( - MeterRegistry meterRegistry) { - return new MicrometerMetricsFactory(meterRegistry); - } - -} diff --git a/spring-boot-project/spring-boot-actuator-autoconfigure/src/main/java/org/springframework/boot/actuate/autoconfigure/metrics/integration/package-info.java b/spring-boot-project/spring-boot-actuator-autoconfigure/src/main/java/org/springframework/boot/actuate/autoconfigure/metrics/integration/package-info.java deleted file mode 100644 index 294cdf100d18..000000000000 --- a/spring-boot-project/spring-boot-actuator-autoconfigure/src/main/java/org/springframework/boot/actuate/autoconfigure/metrics/integration/package-info.java +++ /dev/null @@ -1,20 +0,0 @@ -/* - * Copyright 2012-2018 the original author or authors. - * - * 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. - */ - -/** - * Actuator support for Spring Integration metrics. - */ -package org.springframework.boot.actuate.autoconfigure.metrics.integration; diff --git a/spring-boot-project/spring-boot-actuator-autoconfigure/src/test/java/org/springframework/boot/actuate/autoconfigure/metrics/integration/IntegrationMetricsAutoConfigurationTests.java b/spring-boot-project/spring-boot-actuator-autoconfigure/src/test/java/org/springframework/boot/actuate/autoconfigure/metrics/integration/IntegrationMetricsAutoConfigurationTests.java index a14dcf538c60..2a2d00990cdd 100644 --- a/spring-boot-project/spring-boot-actuator-autoconfigure/src/test/java/org/springframework/boot/actuate/autoconfigure/metrics/integration/IntegrationMetricsAutoConfigurationTests.java +++ b/spring-boot-project/spring-boot-actuator-autoconfigure/src/test/java/org/springframework/boot/actuate/autoconfigure/metrics/integration/IntegrationMetricsAutoConfigurationTests.java @@ -26,25 +26,19 @@ import org.springframework.context.annotation.Bean; import org.springframework.context.annotation.Configuration; import org.springframework.integration.support.MessageBuilder; -import org.springframework.integration.support.management.AbstractMessageChannelMetrics; -import org.springframework.integration.support.management.DefaultMetricsFactory; -import org.springframework.integration.support.management.MetricsFactory; import org.springframework.messaging.Message; import org.springframework.messaging.SubscribableChannel; -import org.springframework.test.util.ReflectionTestUtils; - -import static org.assertj.core.api.Assertions.assertThat; /** * Tests for {@link IntegrationMetricsAutoConfiguration}. * * @author Phillip Webb + * @author Gary Russell */ public class IntegrationMetricsAutoConfigurationTests { - private ApplicationContextRunner contextRunner = new ApplicationContextRunner() - .withConfiguration(AutoConfigurations.of(IntegrationAutoConfiguration.class, - IntegrationMetricsAutoConfiguration.class)) + private final ApplicationContextRunner contextRunner = new ApplicationContextRunner() + .withConfiguration(AutoConfigurations.of(IntegrationAutoConfiguration.class)) .withUserConfiguration(BaseConfiguration.class) .withPropertyValues("spring.jmx.enabled=false"); @@ -56,23 +50,10 @@ public void autoConfiguredIntegrationIsInstrumented() { SubscribableChannel.class); channel.send(message); MeterRegistry registry = context.getBean(MeterRegistry.class); - registry.get("errorChannel.timer").timer(); - registry.get("errorChannel.errorCounter").counter(); + registry.get("spring.integration.send").tag("name", "errorChannel").timer(); }); } - @Test - public void autoConfigurationBacksOffWhenHasMetricsFactory() { - this.contextRunner.withUserConfiguration(LegacyConfiguration.class) - .run((context) -> { - SubscribableChannel channel = context.getBean("errorChannel", - SubscribableChannel.class); - AbstractMessageChannelMetrics metrics = (AbstractMessageChannelMetrics) ReflectionTestUtils - .getField(channel, "channelMetrics"); - assertThat(metrics.getTimer()).isNull(); - }); - } - @Configuration static class BaseConfiguration { @@ -83,14 +64,4 @@ public SimpleMeterRegistry simpleMeterRegistry() { } - @Configuration - static class LegacyConfiguration { - - @Bean - public MetricsFactory legacyMetricsFactory() { - return new DefaultMetricsFactory(); - } - - } - }