Skip to content

Commit

Permalink
Polishing
Browse files Browse the repository at this point in the history
  • Loading branch information
sbrannen committed Mar 20, 2024
1 parent 7f40b49 commit 1732844
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -22,11 +22,13 @@

/**
* {@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
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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);
Expand All @@ -47,7 +47,7 @@ void afterTestClassWhenContextIsAvailable() throws Exception {

@Test
void afterTestClassCWhenContextIsNotAvailable() throws Exception {
TestContext testContext = mock(TestContext.class);
TestContext testContext = mock();
given(testContext.hasApplicationContext()).willReturn(false);
listener.afterTestClass(testContext);
verify(testContext).hasApplicationContext();
Expand Down

0 comments on commit 1732844

Please sign in to comment.