Polishing

This commit is contained in:
Sam Brannen 2024-03-20 12:55:35 +01:00
parent 7f40b49f4d
commit 1732844137
2 changed files with 9 additions and 7 deletions

View File

@ -22,11 +22,13 @@ import org.springframework.test.context.TestContext;
/**
* {@code TestExecutionListener} which makes sure that caches are cleared once
* they are no longer required. Clears the resource cache of the
* {@link ApplicationContext} as it is only required during the beans
* initialization phase. Runs after {@link DirtiesContextTestExecutionListener}
* as dirtying the context will remove it from the cache and make this
* unnecessary.
* they are no longer required.
*
* <p>Clears the resource caches of the {@link ApplicationContext} since they are
* only required during the bean initialization phase. Runs after
* {@link DirtiesContextTestExecutionListener} since dirtying the context will
* close it and remove it from the context cache, making it unnecessary to clear
* the associated resource caches.
*
* @author Stephane Nicoll
* @since 6.2

View File

@ -37,7 +37,7 @@ class CommonCacheTestExecutionListenerTests {
@Test
void afterTestClassWhenContextIsAvailable() throws Exception {
AbstractApplicationContext applicationContext = mock(AbstractApplicationContext.class);
AbstractApplicationContext applicationContext = mock();
TestContext testContext = mock(TestContext.class);
given(testContext.hasApplicationContext()).willReturn(true);
given(testContext.getApplicationContext()).willReturn(applicationContext);
@ -47,7 +47,7 @@ class CommonCacheTestExecutionListenerTests {
@Test
void afterTestClassCWhenContextIsNotAvailable() throws Exception {
TestContext testContext = mock(TestContext.class);
TestContext testContext = mock();
given(testContext.hasApplicationContext()).willReturn(false);
listener.afterTestClass(testContext);
verify(testContext).hasApplicationContext();