Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Merge pull request spring-projects#18689 from dreis2211
* pr/18689:
  Fix Mockito deprecations

Closes spring-projectsgh-18689
  • Loading branch information
snicoll committed Oct 23, 2019
2 parents 511a2a0 + 8ecdf91 commit ce8360d
Show file tree
Hide file tree
Showing 19 changed files with 47 additions and 47 deletions.
Expand Up @@ -38,7 +38,7 @@
import static org.mockito.Mockito.inOrder;
import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.verify;
import static org.mockito.Mockito.verifyZeroInteractions;
import static org.mockito.Mockito.verifyNoInteractions;

/**
* Tests for {@link MeterRegistryConfigurer}.
Expand Down Expand Up @@ -128,7 +128,7 @@ void configureShouldNotApplyBinderWhenCompositeExists() {
MeterRegistryConfigurer configurer = new MeterRegistryConfigurer(createObjectProvider(this.customizers),
createObjectProvider(this.filters), createObjectProvider(this.binders), false, true);
configurer.configure(this.mockRegistry);
verifyZeroInteractions(this.mockBinder);
verifyNoInteractions(this.mockBinder);
}

@Test
Expand Down
Expand Up @@ -35,7 +35,7 @@
import static org.mockito.BDDMockito.given;
import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.verify;
import static org.mockito.Mockito.verifyZeroInteractions;
import static org.mockito.Mockito.verifyNoInteractions;

/**
* Tests for {@link HealthWebEndpointResponseMapper}.
Expand All @@ -56,8 +56,8 @@ void mapDetailsWithDisableDetailsDoesNotInvokeSupplier() {
SecurityContext securityContext = mock(SecurityContext.class);
WebEndpointResponse<Health> response = mapper.mapDetails(supplier, securityContext);
assertThat(response.getStatus()).isEqualTo(HttpStatus.NOT_FOUND.value());
verifyZeroInteractions(supplier);
verifyZeroInteractions(securityContext);
verifyNoInteractions(supplier);
verifyNoInteractions(securityContext);
}

@Test
Expand All @@ -68,7 +68,7 @@ void mapDetailsWithUnauthorizedUserDoesNotInvokeSupplier() {
WebEndpointResponse<Health> response = mapper.mapDetails(supplier, securityContext);
assertThat(response.getStatus()).isEqualTo(HttpStatus.NOT_FOUND.value());
assertThat(response.getBody()).isNull();
verifyZeroInteractions(supplier);
verifyNoInteractions(supplier);
verify(securityContext).isUserInRole("ACTUATOR");
}

Expand All @@ -94,7 +94,7 @@ void mapDetailsWithUnavailableHealth() {
assertThat(response.getStatus()).isEqualTo(HttpStatus.NOT_FOUND.value());
assertThat(response.getBody()).isNull();
verify(supplier).get();
verifyZeroInteractions(securityContext);
verifyNoInteractions(securityContext);
}

@SuppressWarnings("unchecked")
Expand Down
Expand Up @@ -45,7 +45,7 @@
import static org.mockito.BDDMockito.given;
import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.verify;
import static org.mockito.Mockito.verifyZeroInteractions;
import static org.mockito.Mockito.verifyNoInteractions;

/**
* Integration tests for {@link LoggersEndpoint} when exposed via Jersey, Spring MVC, and
Expand Down Expand Up @@ -165,7 +165,7 @@ void setLoggerOrLoggerGroupWithWrongLogLevelResultInBadRequestResponse() {
this.client.post().uri("/actuator/loggers/ROOT").contentType(MediaType.APPLICATION_JSON)
.bodyValue(Collections.singletonMap("configuredLevel", "other")).exchange().expectStatus()
.isBadRequest();
verifyZeroInteractions(this.loggingSystem);
verifyNoInteractions(this.loggingSystem);
}

@WebEndpointTest
Expand Down
Expand Up @@ -44,7 +44,7 @@
import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.never;
import static org.mockito.Mockito.verify;
import static org.mockito.Mockito.verifyZeroInteractions;
import static org.mockito.Mockito.verifyNoInteractions;

/**
* Tests for {@link PrometheusPushGatewayManager}.
Expand Down Expand Up @@ -162,7 +162,7 @@ void shutdownWhenShutdownOperationIsNoneDoesNothing() {
this.scheduler, this.pushRate, "job", this.groupingKey, ShutdownOperation.NONE);
manager.shutdown();
verify(this.future).cancel(false);
verifyZeroInteractions(this.pushGateway);
verifyNoInteractions(this.pushGateway);
}

@Test
Expand Down
Expand Up @@ -40,7 +40,7 @@
import org.springframework.util.ClassUtils;

import static org.assertj.core.api.Assertions.assertThat;
import static org.mockito.Mockito.verifyZeroInteractions;
import static org.mockito.Mockito.verifyNoInteractions;

/**
* Tests for {@link ImportAutoConfigurationImportSelector}.
Expand Down Expand Up @@ -83,7 +83,7 @@ void importsAreSelectedUsingClassesAttribute() throws Exception {
void propertyExclusionsAreNotApplied() throws Exception {
AnnotationMetadata annotationMetadata = getAnnotationMetadata(ImportFreeMarker.class);
this.importSelector.selectImports(annotationMetadata);
verifyZeroInteractions(this.environment);
verifyNoInteractions(this.environment);
}

@Test
Expand Down
Expand Up @@ -67,7 +67,7 @@

import static org.assertj.core.api.Assertions.assertThat;
import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.verifyZeroInteractions;
import static org.mockito.Mockito.verifyNoInteractions;

/**
* Tests for {@link QuartzAutoConfiguration}.
Expand Down Expand Up @@ -157,7 +157,7 @@ void withTaskExecutor() {
Scheduler scheduler = context.getBean(Scheduler.class);
assertThat(scheduler.getMetaData().getThreadPoolSize()).isEqualTo(50);
Executor executor = context.getBean(Executor.class);
verifyZeroInteractions(executor);
verifyNoInteractions(executor);
});
}

Expand Down
Expand Up @@ -42,7 +42,7 @@
import static org.mockito.BDDMockito.willReturn;
import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.verify;
import static org.mockito.Mockito.verifyZeroInteractions;
import static org.mockito.Mockito.verifyNoInteractions;

/**
* Tests for {@link DispatcherFilter}.
Expand Down Expand Up @@ -82,7 +82,7 @@ void ignoresNotServletRequests() throws Exception {
ServletRequest request = mock(ServletRequest.class);
ServletResponse response = mock(ServletResponse.class);
this.filter.doFilter(request, response, this.chain);
verifyZeroInteractions(this.dispatcher);
verifyNoInteractions(this.dispatcher);
verify(this.chain).doFilter(request, response);
}

Expand All @@ -100,7 +100,7 @@ void handledByDispatcher() throws Exception {
HttpServletResponse response = new MockHttpServletResponse();
willReturn(true).given(this.dispatcher).handle(any(ServerHttpRequest.class), any(ServerHttpResponse.class));
this.filter.doFilter(request, response, this.chain);
verifyZeroInteractions(this.chain);
verifyNoInteractions(this.chain);
verify(this.dispatcher).handle(this.serverRequestCaptor.capture(), this.serverResponseCaptor.capture());
ServerHttpRequest dispatcherRequest = this.serverRequestCaptor.getValue();
ServletServerHttpRequest actualRequest = (ServletServerHttpRequest) dispatcherRequest;
Expand Down
Expand Up @@ -41,7 +41,7 @@
import static org.mockito.Mockito.inOrder;
import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.verify;
import static org.mockito.Mockito.verifyZeroInteractions;
import static org.mockito.Mockito.verifyNoInteractions;
import static org.mockito.Mockito.withSettings;

/**
Expand Down Expand Up @@ -91,7 +91,7 @@ void accessManagerVetoRequest() throws Exception {
given(mapper.getHandler(any(ServerHttpRequest.class))).willReturn(handler);
Dispatcher dispatcher = new Dispatcher(this.accessManager, Collections.singleton(mapper));
dispatcher.handle(this.serverRequest, this.serverResponse);
verifyZeroInteractions(handler);
verifyNoInteractions(handler);
assertThat(this.response.getStatus()).isEqualTo(403);
}

Expand Down
Expand Up @@ -47,7 +47,7 @@
import static org.mockito.ArgumentMatchers.any;
import static org.mockito.BDDMockito.given;
import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.verifyZeroInteractions;
import static org.mockito.Mockito.verifyNoInteractions;

/**
* Tests for {@link Restarter}.
Expand Down Expand Up @@ -136,7 +136,7 @@ void getOrAddAttributeWithExistingAttribute() {
ObjectFactory objectFactory = mock(ObjectFactory.class);
Object attribute = Restarter.getInstance().getOrAddAttribute("x", objectFactory);
assertThat(attribute).isEqualTo("abc");
verifyZeroInteractions(objectFactory);
verifyNoInteractions(objectFactory);
}

@Test
Expand Down
Expand Up @@ -38,7 +38,7 @@
import static org.assertj.core.api.Assertions.assertThat;
import static org.assertj.core.api.Assertions.assertThatIllegalArgumentException;
import static org.mockito.Mockito.verify;
import static org.mockito.Mockito.verifyZeroInteractions;
import static org.mockito.Mockito.verifyNoInteractions;

/**
* Tests for {@link HttpRestartServer}.
Expand Down Expand Up @@ -92,7 +92,7 @@ void sendNoContent() throws Exception {
MockHttpServletRequest request = new MockHttpServletRequest();
MockHttpServletResponse response = new MockHttpServletResponse();
this.server.handle(new ServletServerHttpRequest(request), new ServletServerHttpResponse(response));
verifyZeroInteractions(this.delegate);
verifyNoInteractions(this.delegate);
assertThat(response.getStatus()).isEqualTo(500);

}
Expand All @@ -103,7 +103,7 @@ void sendBadData() throws Exception {
MockHttpServletResponse response = new MockHttpServletResponse();
request.setContent(new byte[] { 0, 0, 0 });
this.server.handle(new ServletServerHttpRequest(request), new ServletServerHttpResponse(response));
verifyZeroInteractions(this.delegate);
verifyNoInteractions(this.delegate);
assertThat(response.getStatus()).isEqualTo(500);
}

Expand Down
Expand Up @@ -33,7 +33,7 @@
import static org.assertj.core.api.Assertions.assertThatExceptionOfType;
import static org.mockito.BDDMockito.given;
import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.verifyZeroInteractions;
import static org.mockito.Mockito.verifyNoInteractions;

/**
* Tests for {@link PropertyMappingContextCustomizerFactory}.
Expand All @@ -53,7 +53,7 @@ void getContextCustomizerWhenHasNoMappingShouldNotAddPropertySource() {
given(context.getEnvironment()).willReturn(environment);
given(context.getBeanFactory()).willReturn(beanFactory);
customizer.customizeContext(context, null);
verifyZeroInteractions(environment);
verifyNoInteractions(environment);
}

@Test
Expand Down
Expand Up @@ -23,7 +23,7 @@

import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.verify;
import static org.mockito.Mockito.verifyZeroInteractions;
import static org.mockito.Mockito.verifyNoInteractions;

/**
* Tests for {@link SpringBootExceptionHandler}.
Expand All @@ -43,7 +43,7 @@ void uncaughtExceptionDoesNotForwardLoggedErrorToParent() {
Exception ex = new Exception();
this.handler.registerLoggedException(ex);
this.handler.uncaughtException(thread, ex);
verifyZeroInteractions(this.parent);
verifyNoInteractions(this.parent);
}

@Test
Expand Down
Expand Up @@ -33,7 +33,7 @@
import static org.assertj.core.api.Assertions.assertThatIllegalArgumentException;
import static org.mockito.BDDMockito.given;
import static org.mockito.Mockito.verify;
import static org.mockito.Mockito.verifyZeroInteractions;
import static org.mockito.Mockito.verifyNoInteractions;

/**
* Tests for {@link BindResult}.
Expand Down Expand Up @@ -100,7 +100,7 @@ void ifBoundWhenHasValueShouldCallConsumer() {
void ifBoundWhenHasNoValueShouldNotCallConsumer() {
BindResult<String> result = BindResult.of(null);
result.ifBound(this.consumer);
verifyZeroInteractions(this.consumer);
verifyNoInteractions(this.consumer);
}

@Test
Expand All @@ -121,7 +121,7 @@ void mapWhenHasValueShouldCallMapper() {
void mapWhenHasNoValueShouldNotCallMapper() {
BindResult<String> result = BindResult.of(null);
result.map(this.mapper);
verifyZeroInteractions(this.mapper);
verifyNoInteractions(this.mapper);
}

@Test
Expand All @@ -140,7 +140,7 @@ void orElseWhenHasValueNoShouldReturnOther() {
void orElseGetWhenHasValueShouldReturnValue() {
BindResult<String> result = BindResult.of("foo");
assertThat(result.orElseGet(this.supplier)).isEqualTo("foo");
verifyZeroInteractions(this.supplier);
verifyNoInteractions(this.supplier);
}

@Test
Expand Down
Expand Up @@ -26,8 +26,8 @@
import static org.assertj.core.api.Assertions.assertThat;
import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.verify;
import static org.mockito.Mockito.verifyNoInteractions;
import static org.mockito.Mockito.verifyNoMoreInteractions;
import static org.mockito.Mockito.verifyZeroInteractions;

/**
* Tests for {@link DeferredLog}.
Expand Down Expand Up @@ -168,7 +168,7 @@ void clearsOnReplayTo() {
verify(this.log).info("1", null);
verify(this.log).fatal("2", null);
verifyNoMoreInteractions(this.log);
verifyZeroInteractions(log2);
verifyNoInteractions(log2);
}

@SuppressWarnings("unchecked")
Expand Down
Expand Up @@ -31,7 +31,7 @@
import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.spy;
import static org.mockito.Mockito.verify;
import static org.mockito.Mockito.verifyZeroInteractions;
import static org.mockito.Mockito.verifyNoInteractions;

/**
* Tests for {@link TaskExecutorBuilder}.
Expand Down Expand Up @@ -125,7 +125,7 @@ void customizersShouldReplaceExisting() {
TaskExecutorCustomizer customizer2 = mock(TaskExecutorCustomizer.class);
ThreadPoolTaskExecutor executor = this.builder.customizers(customizer1)
.customizers(Collections.singleton(customizer2)).build();
verifyZeroInteractions(customizer1);
verifyNoInteractions(customizer1);
verify(customizer2).customize(executor);
}

Expand Down
Expand Up @@ -29,7 +29,7 @@
import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.spy;
import static org.mockito.Mockito.verify;
import static org.mockito.Mockito.verifyZeroInteractions;
import static org.mockito.Mockito.verifyNoInteractions;

/**
* Tests for {@link TaskSchedulerBuilder}.
Expand Down Expand Up @@ -101,7 +101,7 @@ void customizersShouldReplaceExisting() {
TaskSchedulerCustomizer customizer2 = mock(TaskSchedulerCustomizer.class);
ThreadPoolTaskScheduler scheduler = this.builder.customizers(customizer1)
.customizers(Collections.singleton(customizer2)).build();
verifyZeroInteractions(customizer1);
verifyNoInteractions(customizer1);
verify(customizer2).customize(scheduler);
}

Expand Down
Expand Up @@ -35,7 +35,7 @@
import static org.mockito.BDDMockito.given;
import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.verify;
import static org.mockito.Mockito.verifyZeroInteractions;
import static org.mockito.Mockito.verifyNoInteractions;

/**
* Tests for {@link LambdaSafe}.
Expand Down Expand Up @@ -88,7 +88,7 @@ void callbackInvokeWhenHasResolvableGenericNonMatchShouldNotInvokeCallback() {
GenericCallback<?> callbackInstance = mock(StringBuilderCallback.class);
String argument = "foo";
LambdaSafe.callback(GenericCallback.class, callbackInstance, argument).invoke((c) -> c.handle(argument));
verifyZeroInteractions(callbackInstance);
verifyNoInteractions(callbackInstance);
}

@Test
Expand Down Expand Up @@ -164,7 +164,7 @@ void callbackInvokeAndWhenHasResolvableGenericNonMatchShouldReturnNoResult() {
InvocationResult<Integer> result = LambdaSafe.callback(GenericFactory.class, callbackInstance, argument)
.invokeAnd((c) -> c.handle(argument));
assertThat(result.hasResult()).isFalse();
verifyZeroInteractions(callbackInstance);
verifyNoInteractions(callbackInstance);
}

@Test
Expand Down Expand Up @@ -229,7 +229,7 @@ void callbacksInvokeWhenHasResolvableGenericNonMatchShouldNotInvokeCallback() {
String argument = "foo";
LambdaSafe.callbacks(GenericCallback.class, Collections.singleton(callbackInstance), argument)
.invoke((c) -> c.handle(null));
verifyZeroInteractions(callbackInstance);
verifyNoInteractions(callbackInstance);
}

@Test
Expand Down
Expand Up @@ -62,7 +62,7 @@
import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.spy;
import static org.mockito.Mockito.verify;
import static org.mockito.Mockito.verifyZeroInteractions;
import static org.mockito.Mockito.verifyNoInteractions;
import static org.springframework.test.web.client.match.MockRestRequestMatchers.requestTo;
import static org.springframework.test.web.client.response.MockRestResponseCreators.withSuccess;

Expand Down Expand Up @@ -391,7 +391,7 @@ void customizersShouldReplaceExisting() {
RestTemplateCustomizer customizer2 = mock(RestTemplateCustomizer.class);
RestTemplate template = this.builder.customizers(customizer1).customizers(Collections.singleton(customizer2))
.build();
verifyZeroInteractions(customizer1);
verifyNoInteractions(customizer1);
verify(customizer2).customize(template);
}

Expand Down

0 comments on commit ce8360d

Please sign in to comment.