Apply TestTypeExcludeFilter in regular applications
Add `ExcludeFilterApplicationContextInitializer` to register the `TestTypeExcludeFilter` for regular applications. Prior to this commit, the filter was only registered using the `ExcludeFilterContextCustomizerFactory` which meant that test components were filtered in tests but not when using `SpringApplication.from` with a test classpath. Fixes gh-35206
This commit is contained in:
@@ -0,0 +1,37 @@
|
|||||||
|
/*
|
||||||
|
* Copyright 2012-2023 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
|
||||||
|
*
|
||||||
|
* https://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.test.context.filter;
|
||||||
|
|
||||||
|
import org.springframework.boot.SpringApplication;
|
||||||
|
import org.springframework.context.ApplicationContextInitializer;
|
||||||
|
import org.springframework.context.ConfigurableApplicationContext;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* {@link ApplicationContextInitializer} to register the {@link TestTypeExcludeFilter} for
|
||||||
|
* when {@link SpringApplication#from} is being used with the test classpath.
|
||||||
|
*
|
||||||
|
* @author Phillip Webb
|
||||||
|
*/
|
||||||
|
class ExcludeFilterApplicationContextInitializer
|
||||||
|
implements ApplicationContextInitializer<ConfigurableApplicationContext> {
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public void initialize(ConfigurableApplicationContext applicationContext) {
|
||||||
|
TestTypeExcludeFilter.registerWith(applicationContext.getBeanFactory());
|
||||||
|
}
|
||||||
|
|
||||||
|
}
|
||||||
@@ -1,5 +1,5 @@
|
|||||||
/*
|
/*
|
||||||
* Copyright 2012-2019 the original author or authors.
|
* Copyright 2012-2023 the original author or authors.
|
||||||
*
|
*
|
||||||
* Licensed under the Apache License, Version 2.0 (the "License");
|
* Licensed under the Apache License, Version 2.0 (the "License");
|
||||||
* you may not use this file except in compliance with the License.
|
* you may not use this file except in compliance with the License.
|
||||||
@@ -32,7 +32,7 @@ class ExcludeFilterContextCustomizer implements ContextCustomizer {
|
|||||||
@Override
|
@Override
|
||||||
public void customizeContext(ConfigurableApplicationContext context,
|
public void customizeContext(ConfigurableApplicationContext context,
|
||||||
MergedContextConfiguration mergedContextConfiguration) {
|
MergedContextConfiguration mergedContextConfiguration) {
|
||||||
context.getBeanFactory().registerSingleton(TestTypeExcludeFilter.class.getName(), new TestTypeExcludeFilter());
|
TestTypeExcludeFilter.registerWith(context.getBeanFactory());
|
||||||
}
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
|
|||||||
@@ -1,5 +1,5 @@
|
|||||||
/*
|
/*
|
||||||
* Copyright 2012-2020 the original author or authors.
|
* Copyright 2012-2023 the original author or authors.
|
||||||
*
|
*
|
||||||
* Licensed under the Apache License, Version 2.0 (the "License");
|
* Licensed under the Apache License, Version 2.0 (the "License");
|
||||||
* you may not use this file except in compliance with the License.
|
* you may not use this file except in compliance with the License.
|
||||||
@@ -18,6 +18,7 @@ package org.springframework.boot.test.context.filter;
|
|||||||
|
|
||||||
import java.io.IOException;
|
import java.io.IOException;
|
||||||
|
|
||||||
|
import org.springframework.beans.factory.config.ConfigurableListableBeanFactory;
|
||||||
import org.springframework.boot.context.TypeExcludeFilter;
|
import org.springframework.boot.context.TypeExcludeFilter;
|
||||||
import org.springframework.boot.test.context.TestComponent;
|
import org.springframework.boot.test.context.TestComponent;
|
||||||
import org.springframework.core.type.classreading.MetadataReader;
|
import org.springframework.core.type.classreading.MetadataReader;
|
||||||
@@ -32,6 +33,8 @@ import org.springframework.core.type.classreading.MetadataReaderFactory;
|
|||||||
*/
|
*/
|
||||||
class TestTypeExcludeFilter extends TypeExcludeFilter {
|
class TestTypeExcludeFilter extends TypeExcludeFilter {
|
||||||
|
|
||||||
|
private static final String BEAN_NAME = TestTypeExcludeFilter.class.getName();
|
||||||
|
|
||||||
private static final String[] CLASS_ANNOTATIONS = { "org.junit.runner.RunWith",
|
private static final String[] CLASS_ANNOTATIONS = { "org.junit.runner.RunWith",
|
||||||
"org.junit.jupiter.api.extension.ExtendWith", "org.junit.platform.commons.annotation.Testable",
|
"org.junit.jupiter.api.extension.ExtendWith", "org.junit.platform.commons.annotation.Testable",
|
||||||
"org.testng.annotations.Test" };
|
"org.testng.annotations.Test" };
|
||||||
@@ -39,6 +42,8 @@ class TestTypeExcludeFilter extends TypeExcludeFilter {
|
|||||||
private static final String[] METHOD_ANNOTATIONS = { "org.junit.Test",
|
private static final String[] METHOD_ANNOTATIONS = { "org.junit.Test",
|
||||||
"org.junit.platform.commons.annotation.Testable", "org.testng.annotations.Test" };
|
"org.junit.platform.commons.annotation.Testable", "org.testng.annotations.Test" };
|
||||||
|
|
||||||
|
private static final TestTypeExcludeFilter INSTANCE = new TestTypeExcludeFilter();
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
public boolean match(MetadataReader metadataReader, MetadataReaderFactory metadataReaderFactory)
|
public boolean match(MetadataReader metadataReader, MetadataReaderFactory metadataReaderFactory)
|
||||||
throws IOException {
|
throws IOException {
|
||||||
@@ -91,4 +96,10 @@ class TestTypeExcludeFilter extends TypeExcludeFilter {
|
|||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
static void registerWith(ConfigurableListableBeanFactory beanFactory) {
|
||||||
|
if (!beanFactory.containsSingleton(BEAN_NAME)) {
|
||||||
|
beanFactory.registerSingleton(BEAN_NAME, INSTANCE);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -16,3 +16,7 @@ org.springframework.boot.test.mock.mockito.ResetMocksTestExecutionListener
|
|||||||
# Environment Post Processors
|
# Environment Post Processors
|
||||||
org.springframework.boot.env.EnvironmentPostProcessor=\
|
org.springframework.boot.env.EnvironmentPostProcessor=\
|
||||||
org.springframework.boot.test.web.SpringBootTestRandomPortEnvironmentPostProcessor
|
org.springframework.boot.test.web.SpringBootTestRandomPortEnvironmentPostProcessor
|
||||||
|
|
||||||
|
# Application Context Initializers
|
||||||
|
org.springframework.context.ApplicationContextInitializer=\
|
||||||
|
org.springframework.boot.test.context.filter.ExcludeFilterApplicationContextInitializer
|
||||||
@@ -0,0 +1,58 @@
|
|||||||
|
/*
|
||||||
|
* Copyright 2012-2023 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
|
||||||
|
*
|
||||||
|
* https://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.test.context.filter;
|
||||||
|
|
||||||
|
import org.junit.jupiter.api.Test;
|
||||||
|
|
||||||
|
import org.springframework.boot.SpringApplication;
|
||||||
|
import org.springframework.boot.WebApplicationType;
|
||||||
|
import org.springframework.boot.context.TypeExcludeFilter;
|
||||||
|
import org.springframework.boot.test.context.TestConfiguration;
|
||||||
|
import org.springframework.boot.test.context.assertj.AssertableApplicationContext;
|
||||||
|
import org.springframework.context.annotation.ComponentScan;
|
||||||
|
import org.springframework.context.annotation.ComponentScan.Filter;
|
||||||
|
import org.springframework.context.annotation.FilterType;
|
||||||
|
|
||||||
|
import static org.assertj.core.api.Assertions.assertThat;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Tests {@link ExcludeFilterApplicationContextInitializer}.
|
||||||
|
*
|
||||||
|
* @author Phillip Webb
|
||||||
|
*/
|
||||||
|
class ExcludeFilterApplicationContextInitializerTests {
|
||||||
|
|
||||||
|
@Test
|
||||||
|
void testConfigurationIsExcluded() {
|
||||||
|
SpringApplication application = new SpringApplication(TestApplication.class);
|
||||||
|
application.setWebApplicationType(WebApplicationType.NONE);
|
||||||
|
AssertableApplicationContext applicationContext = AssertableApplicationContext.get(() -> application.run());
|
||||||
|
assertThat(applicationContext).hasSingleBean(TestApplication.class);
|
||||||
|
assertThat(applicationContext).doesNotHaveBean(ExcludedTestConfiguration.class);
|
||||||
|
}
|
||||||
|
|
||||||
|
@ComponentScan(excludeFilters = { @Filter(type = FilterType.CUSTOM, classes = TypeExcludeFilter.class) })
|
||||||
|
static class TestApplication {
|
||||||
|
|
||||||
|
}
|
||||||
|
|
||||||
|
@TestConfiguration(proxyBeanMethods = false)
|
||||||
|
static class ExcludedTestConfiguration {
|
||||||
|
|
||||||
|
}
|
||||||
|
|
||||||
|
}
|
||||||
Reference in New Issue
Block a user