Browse Source

Polishing

pull/33047/head
Juergen Hoeller 2 years ago
parent
commit
6c08d93992
  1. 2
      integration-tests/src/test/java/org/springframework/cache/annotation/EnableCachingIntegrationTests.java
  2. 3
      integration-tests/src/test/java/org/springframework/core/env/EnvironmentSystemIntegrationTests.java
  3. 5
      integration-tests/src/test/java/org/springframework/transaction/annotation/EnableTransactionManagementIntegrationTests.java
  4. 3
      spring-aspects/src/test/resources/org/springframework/cache/config/annotation-jcache-aspectj.xml
  5. 6
      spring-aspects/src/test/resources/org/springframework/scheduling/aspectj/annotationDrivenContext.xml

2
integration-tests/src/test/java/org/springframework/cache/annotation/EnableCachingIntegrationTests.java vendored

@ -62,7 +62,7 @@ class EnableCachingIntegrationTests { @@ -62,7 +62,7 @@ class EnableCachingIntegrationTests {
// attempt was made to look up the AJ aspect. It's due to classpath issues
// in integration-tests that it's not found.
assertThatException().isThrownBy(ctx::refresh)
.withMessageContaining("AspectJCachingConfiguration");
.withMessageContaining("AspectJCachingConfiguration");
}

3
integration-tests/src/test/java/org/springframework/core/env/EnvironmentSystemIntegrationTests.java vendored

@ -541,8 +541,7 @@ public class EnvironmentSystemIntegrationTests { @@ -541,8 +541,7 @@ public class EnvironmentSystemIntegrationTests {
{
ConfigurableApplicationContext ctx = new AnnotationConfigApplicationContext();
ctx.getEnvironment().setRequiredProperties("foo", "bar");
assertThatExceptionOfType(MissingRequiredPropertiesException.class).isThrownBy(
ctx::refresh);
assertThatExceptionOfType(MissingRequiredPropertiesException.class).isThrownBy(ctx::refresh);
}
{

5
integration-tests/src/test/java/org/springframework/transaction/annotation/EnableTransactionManagementIntegrationTests.java

@ -97,9 +97,8 @@ class EnableTransactionManagementIntegrationTests { @@ -97,9 +97,8 @@ class EnableTransactionManagementIntegrationTests {
// this test is a bit fragile, but gets the job done, proving that an
// attempt was made to look up the AJ aspect. It's due to classpath issues
// in integration-tests that it's not found.
assertThatException()
.isThrownBy(ctx::refresh)
.withMessageContaining("AspectJJtaTransactionManagementConfiguration");
assertThatException().isThrownBy(ctx::refresh)
.withMessageContaining("AspectJJtaTransactionManagementConfiguration");
}
@Test

3
spring-aspects/src/test/resources/org/springframework/cache/config/annotation-jcache-aspectj.xml vendored

@ -24,8 +24,7 @@ @@ -24,8 +24,7 @@
</property>
</bean>
<bean id="defaultCache"
class="org.springframework.cache.concurrent.ConcurrentMapCacheFactoryBean">
<bean id="defaultCache" class="org.springframework.cache.concurrent.ConcurrentMapCacheFactoryBean">
<property name="name" value="default"/>
</bean>

6
spring-aspects/src/test/resources/org/springframework/scheduling/aspectj/annotationDrivenContext.xml

@ -7,12 +7,10 @@ @@ -7,12 +7,10 @@
http://www.springframework.org/schema/task
https://www.springframework.org/schema/task/spring-task.xsd">
<task:annotation-driven mode="aspectj" executor="testExecutor"
exception-handler="testExceptionHandler"/>
<task:annotation-driven mode="aspectj" executor="testExecutor" exception-handler="testExceptionHandler"/>
<task:executor id="testExecutor"/>
<bean id="testExceptionHandler"
class="org.springframework.aop.interceptor.SimpleAsyncUncaughtExceptionHandler"/>
<bean id="testExceptionHandler" class="org.springframework.aop.interceptor.SimpleAsyncUncaughtExceptionHandler"/>
</beans>

Loading…
Cancel
Save