diff --git a/spring-test/src/main/java/org/springframework/test/web/client/MockRestServiceServer.java b/spring-test/src/main/java/org/springframework/test/web/client/MockRestServiceServer.java index 779bc7408d3..639997cc461 100644 --- a/spring-test/src/main/java/org/springframework/test/web/client/MockRestServiceServer.java +++ b/spring-test/src/main/java/org/springframework/test/web/client/MockRestServiceServer.java @@ -152,7 +152,10 @@ public final class MockRestServiceServer { * Return a builder for a {@code MockRestServiceServer} that should be used * to reply to the given {@code RestTemplate}. * @since 4.3 + * @deprecated as of 7.1 in favor of {@link #bindTo(RestClient.Builder)}. */ + @Deprecated(since = "7.1", forRemoval = true) + @SuppressWarnings("removal") public static MockRestServiceServerBuilder bindTo(RestTemplate restTemplate) { return new RestTemplateMockRestServiceServerBuilder(restTemplate); } @@ -161,7 +164,10 @@ public final class MockRestServiceServer { * Return a builder for a {@code MockRestServiceServer} that should be used * to reply to the {@code RestTemplate} for the given {@code RestGatewaySupport}. * @since 4.3 + * @deprecated as of 7.1 in favor of {@link #bindTo(RestClient.Builder)}. */ + @Deprecated(since = "7.1", forRemoval = true) + @SuppressWarnings("removal") public static MockRestServiceServerBuilder bindTo(RestGatewaySupport restGatewaySupport) { Assert.notNull(restGatewaySupport, "'restGatewaySupport' must not be null"); return new RestTemplateMockRestServiceServerBuilder(restGatewaySupport.getRestTemplate()); @@ -182,7 +188,10 @@ public final class MockRestServiceServer { * A shortcut for {@code bindTo(restTemplate).build()}. * @param restTemplate the RestTemplate to set up for mock testing * @return the mock server + * @deprecated as of 7.1 in favor of {@link #bindTo(RestClient.Builder)}. */ + @Deprecated(since = "7.1", forRemoval = true) + @SuppressWarnings("removal") public static MockRestServiceServer createServer(RestTemplate restTemplate) { return bindTo(restTemplate).build(); } @@ -191,7 +200,10 @@ public final class MockRestServiceServer { * A shortcut for {@code bindTo(restGateway).build()}. * @param restGateway the REST gateway to set up for mock testing * @return the mock server + * @deprecated as of 7.1 in favor of {@link #bindTo(RestClient.Builder)}. */ + @Deprecated(since = "7.1", forRemoval = true) + @SuppressWarnings("removal") public static MockRestServiceServer createServer(RestGatewaySupport restGateway) { return bindTo(restGateway).build(); } @@ -300,7 +312,7 @@ public final class MockRestServiceServer { } } - + @SuppressWarnings("removal") private static class RestTemplateMockRestServiceServerBuilder extends AbstractMockRestServiceServerBuilder { private final RestTemplate restTemplate; diff --git a/spring-test/src/test/java/org/springframework/test/web/client/samples/MockMvcClientHttpRequestFactoryTests.java b/spring-test/src/test/java/org/springframework/test/web/client/samples/MockMvcClientHttpRequestFactoryTests.java index e75962a6f83..026fe1dcd30 100644 --- a/spring-test/src/test/java/org/springframework/test/web/client/samples/MockMvcClientHttpRequestFactoryTests.java +++ b/spring-test/src/test/java/org/springframework/test/web/client/samples/MockMvcClientHttpRequestFactoryTests.java @@ -47,7 +47,7 @@ import static org.assertj.core.api.Assertions.assertThatExceptionOfType; * @author Juergen Hoeller */ @SpringJUnitWebConfig -@SuppressWarnings("deprecation") +@SuppressWarnings({"deprecation", "removal"}) class MockMvcClientHttpRequestFactoryTests { private final RestTemplate template; diff --git a/spring-web/src/main/java/org/springframework/http/client/support/HttpAccessor.java b/spring-web/src/main/java/org/springframework/http/client/support/HttpAccessor.java index 7cf850c52dd..c42b584916e 100644 --- a/spring-web/src/main/java/org/springframework/http/client/support/HttpAccessor.java +++ b/spring-web/src/main/java/org/springframework/http/client/support/HttpAccessor.java @@ -50,7 +50,10 @@ import org.springframework.util.Assert; * @since 3.0 * @see ClientHttpRequestFactory * @see org.springframework.web.client.RestTemplate + * @deprecated since 7.1 with no replacement. */ +@Deprecated(since = "7.1", forRemoval = true) +@SuppressWarnings("removal") public abstract class HttpAccessor { /** Logger available to subclasses. */ diff --git a/spring-web/src/main/java/org/springframework/http/client/support/InterceptingHttpAccessor.java b/spring-web/src/main/java/org/springframework/http/client/support/InterceptingHttpAccessor.java index a4436fb391e..75000affea3 100644 --- a/spring-web/src/main/java/org/springframework/http/client/support/InterceptingHttpAccessor.java +++ b/spring-web/src/main/java/org/springframework/http/client/support/InterceptingHttpAccessor.java @@ -42,7 +42,10 @@ import org.springframework.util.CollectionUtils; * @see ClientHttpRequestInterceptor * @see InterceptingClientHttpRequestFactory * @see org.springframework.web.client.RestTemplate + * @deprecated since 7.1 with no replacement. */ +@Deprecated(since = "7.1", forRemoval = true) +@SuppressWarnings("removal") public abstract class InterceptingHttpAccessor extends HttpAccessor { private final List interceptors = new ArrayList<>(); diff --git a/spring-web/src/main/java/org/springframework/web/client/DefaultResponseErrorHandler.java b/spring-web/src/main/java/org/springframework/web/client/DefaultResponseErrorHandler.java index 6e83480dd21..2ede5e8fcf1 100644 --- a/spring-web/src/main/java/org/springframework/web/client/DefaultResponseErrorHandler.java +++ b/spring-web/src/main/java/org/springframework/web/client/DefaultResponseErrorHandler.java @@ -58,7 +58,10 @@ import org.springframework.util.ObjectUtils; * @author Juergen Hoeller * @since 3.0 * @see RestTemplate#setErrorHandler + * @deprecated as of 7.1 in favor of {@link RestClient.ResponseSpec.ErrorHandler} */ +@Deprecated(since = "7.1", forRemoval = true) +@SuppressWarnings("removal") public class DefaultResponseErrorHandler implements ResponseErrorHandler { private @Nullable List> messageConverters; diff --git a/spring-web/src/main/java/org/springframework/web/client/DefaultRestClient.java b/spring-web/src/main/java/org/springframework/web/client/DefaultRestClient.java index af7ea9133e0..0b937410d14 100644 --- a/spring-web/src/main/java/org/springframework/web/client/DefaultRestClient.java +++ b/spring-web/src/main/java/org/springframework/web/client/DefaultRestClient.java @@ -814,6 +814,7 @@ final class DefaultRestClient implements RestClient { } @Override + @SuppressWarnings("removal") public ResponseSpec onStatus(ResponseErrorHandler errorHandler) { Assert.notNull(errorHandler, "ResponseErrorHandler must not be null"); diff --git a/spring-web/src/main/java/org/springframework/web/client/DefaultRestClientBuilder.java b/spring-web/src/main/java/org/springframework/web/client/DefaultRestClientBuilder.java index 9df03025e8b..0b93147e746 100644 --- a/spring-web/src/main/java/org/springframework/web/client/DefaultRestClientBuilder.java +++ b/spring-web/src/main/java/org/springframework/web/client/DefaultRestClientBuilder.java @@ -148,6 +148,7 @@ final class DefaultRestClientBuilder implements RestClient.Builder { this.observationConvention = other.observationConvention; } + @SuppressWarnings("removal") public DefaultRestClientBuilder(RestTemplate restTemplate) { Assert.notNull(restTemplate, "RestTemplate must not be null"); @@ -167,6 +168,7 @@ final class DefaultRestClientBuilder implements RestClient.Builder { this.observationConvention = restTemplate.getObservationConvention(); } + @SuppressWarnings("removal") private static @Nullable UriBuilderFactory getUriBuilderFactory(RestTemplate restTemplate) { UriTemplateHandler uriTemplateHandler = restTemplate.getUriTemplateHandler(); if (uriTemplateHandler instanceof DefaultUriBuilderFactory builderFactory) { @@ -199,6 +201,7 @@ final class DefaultRestClientBuilder implements RestClient.Builder { factory.shouldParsePath()); } + @SuppressWarnings("removal") private static ClientHttpRequestFactory getRequestFactory(RestTemplate restTemplate) { ClientHttpRequestFactory requestFactory = restTemplate.getRequestFactory(); if (requestFactory instanceof InterceptingClientHttpRequestFactory interceptingClientHttpRequestFactory) { @@ -297,6 +300,7 @@ final class DefaultRestClientBuilder implements RestClient.Builder { } @Override + @SuppressWarnings("removal") public RestClient.Builder defaultStatusHandler(ResponseErrorHandler errorHandler) { return defaultStatusHandlerInternal(StatusHandler.fromErrorHandler(errorHandler)); } diff --git a/spring-web/src/main/java/org/springframework/web/client/ExtractingResponseErrorHandler.java b/spring-web/src/main/java/org/springframework/web/client/ExtractingResponseErrorHandler.java index 0a3d69581f2..58e3de3366c 100644 --- a/spring-web/src/main/java/org/springframework/web/client/ExtractingResponseErrorHandler.java +++ b/spring-web/src/main/java/org/springframework/web/client/ExtractingResponseErrorHandler.java @@ -60,7 +60,10 @@ import org.springframework.util.CollectionUtils; * @author Arjen Poutsma * @since 5.0 * @see RestTemplate#setErrorHandler(ResponseErrorHandler) + * @deprecated as of 7.1 in favor of {@link RestClient.ResponseSpec.ErrorHandler} */ +@Deprecated(since = "7.1", forRemoval = true) +@SuppressWarnings("removal") public class ExtractingResponseErrorHandler extends DefaultResponseErrorHandler { private List> messageConverters = Collections.emptyList(); diff --git a/spring-web/src/main/java/org/springframework/web/client/HttpMessageConverterExtractor.java b/spring-web/src/main/java/org/springframework/web/client/HttpMessageConverterExtractor.java index f386798ad4a..c6a65aee383 100644 --- a/spring-web/src/main/java/org/springframework/web/client/HttpMessageConverterExtractor.java +++ b/spring-web/src/main/java/org/springframework/web/client/HttpMessageConverterExtractor.java @@ -43,7 +43,10 @@ import org.springframework.util.FileCopyUtils; * @since 3.0 * @param the data type * @see RestTemplate + * @deprecated as of 7.1 with no replacement. */ +@Deprecated(since = "7.1", forRemoval = true) +@SuppressWarnings("removal") public class HttpMessageConverterExtractor implements ResponseExtractor { private final Type responseType; diff --git a/spring-web/src/main/java/org/springframework/web/client/NoOpResponseErrorHandler.java b/spring-web/src/main/java/org/springframework/web/client/NoOpResponseErrorHandler.java index 3864923d33e..83c71248ba0 100644 --- a/spring-web/src/main/java/org/springframework/web/client/NoOpResponseErrorHandler.java +++ b/spring-web/src/main/java/org/springframework/web/client/NoOpResponseErrorHandler.java @@ -35,7 +35,10 @@ import org.springframework.web.client.RestClient.ResponseSpec.ErrorHandler; * * @author Stephane Nicoll * @since 6.1.7 + * @deprecated as of 7.1 in favor of {@link RestClient.ResponseSpec.ErrorHandler} */ +@Deprecated(since = "7.1", forRemoval = true) +@SuppressWarnings("removal") public final class NoOpResponseErrorHandler implements ResponseErrorHandler { @Override diff --git a/spring-web/src/main/java/org/springframework/web/client/RequestCallback.java b/spring-web/src/main/java/org/springframework/web/client/RequestCallback.java index 22b2b4c189f..9b671c84509 100644 --- a/spring-web/src/main/java/org/springframework/web/client/RequestCallback.java +++ b/spring-web/src/main/java/org/springframework/web/client/RequestCallback.java @@ -18,6 +18,7 @@ package org.springframework.web.client; import java.io.IOException; import java.lang.reflect.Type; +import java.util.function.Consumer; import org.springframework.http.HttpOutputMessage; import org.springframework.http.client.ClientHttpRequest; @@ -37,8 +38,11 @@ import org.springframework.http.client.ClientHttpRequest; * @author Arjen Poutsma * @since 3.0 * @see RestTemplate#execute + * @deprecated as of 7.1 in favor of {@link RestClient.RequestBodySpec#httpRequest(Consumer)}. */ @FunctionalInterface +@Deprecated(since = "7.1", forRemoval = true) +@SuppressWarnings("removal") public interface RequestCallback { /** diff --git a/spring-web/src/main/java/org/springframework/web/client/ResponseErrorHandler.java b/spring-web/src/main/java/org/springframework/web/client/ResponseErrorHandler.java index 8f961907308..b39bb52c51c 100644 --- a/spring-web/src/main/java/org/springframework/web/client/ResponseErrorHandler.java +++ b/spring-web/src/main/java/org/springframework/web/client/ResponseErrorHandler.java @@ -31,7 +31,9 @@ import org.springframework.http.client.ClientHttpResponse; * * @author Arjen Poutsma * @since 3.0 + * @deprecated as of 7.1 in favor of {@link RestClient.ResponseSpec.ErrorHandler} */ +@Deprecated(since = "7.1", forRemoval = true) public interface ResponseErrorHandler { /** diff --git a/spring-web/src/main/java/org/springframework/web/client/ResponseExtractor.java b/spring-web/src/main/java/org/springframework/web/client/ResponseExtractor.java index 5bedc904004..318c4347f2b 100644 --- a/spring-web/src/main/java/org/springframework/web/client/ResponseExtractor.java +++ b/spring-web/src/main/java/org/springframework/web/client/ResponseExtractor.java @@ -37,8 +37,11 @@ import org.springframework.http.client.ClientHttpResponse; * @since 3.0 * @param the data type * @see RestTemplate#execute + * @deprecated as of 7.1 in favor of {@link RestClient.RequestBodySpec#exchange(RestClient.RequestHeadersSpec.ExchangeFunction)}. */ @FunctionalInterface +@Deprecated(since = "7.1", forRemoval = true) +@SuppressWarnings("removal") public interface ResponseExtractor { /** diff --git a/spring-web/src/main/java/org/springframework/web/client/RestClient.java b/spring-web/src/main/java/org/springframework/web/client/RestClient.java index dc72cc5beb5..22850d80070 100644 --- a/spring-web/src/main/java/org/springframework/web/client/RestClient.java +++ b/spring-web/src/main/java/org/springframework/web/client/RestClient.java @@ -189,6 +189,7 @@ public interface RestClient { * @return a {@code RestClient} initialized with the {@code restTemplate}'s * configuration */ + @SuppressWarnings("removal") static RestClient create(RestTemplate restTemplate) { return new DefaultRestClientBuilder(restTemplate).build(); } @@ -218,6 +219,7 @@ public interface RestClient { * @return a {@code RestClient} builder initialized with {@code restTemplate}'s * configuration */ + @SuppressWarnings("removal") static RestClient.Builder builder(RestTemplate restTemplate) { return new DefaultRestClientBuilder(restTemplate); } @@ -386,7 +388,10 @@ public interface RestClient { * @param errorHandler the error handler to configure, internally adapted * and integrated into the {@link ResponseSpec.ErrorHandler} chain. * @return this builder + * @deprecated as of 7.1 in favor of {@link #defaultStatusHandler(Predicate, ResponseSpec.ErrorHandler)} */ + @Deprecated(since = "7.1", forRemoval = true) + @SuppressWarnings("removal") Builder defaultStatusHandler(ResponseErrorHandler errorHandler); /** @@ -1012,7 +1017,10 @@ public interface RestClient { * {@link RestClientException}. * @param errorHandler the error handler * @return this builder + * @deprecated as of 7.1 in favor of {@link #onStatus(Predicate, ErrorHandler)} */ + @Deprecated(since = "7.1", forRemoval = true) + @SuppressWarnings("removal") ResponseSpec onStatus(ResponseErrorHandler errorHandler); /** diff --git a/spring-web/src/main/java/org/springframework/web/client/RestOperations.java b/spring-web/src/main/java/org/springframework/web/client/RestOperations.java index 502ea82466a..0a87791cfb1 100644 --- a/spring-web/src/main/java/org/springframework/web/client/RestOperations.java +++ b/spring-web/src/main/java/org/springframework/web/client/RestOperations.java @@ -39,7 +39,10 @@ import org.springframework.http.ResponseEntity; * @author Juergen Hoeller * @since 3.0 * @see RestTemplate + * @deprecated as of 7.1, in favor of {@link RestClient}. */ +@Deprecated(since = "7.1", forRemoval = true) +@SuppressWarnings("removal") public interface RestOperations { // GET diff --git a/spring-web/src/main/java/org/springframework/web/client/RestTemplate.java b/spring-web/src/main/java/org/springframework/web/client/RestTemplate.java index 7b8558a51e3..aafde1e4dc1 100644 --- a/spring-web/src/main/java/org/springframework/web/client/RestTemplate.java +++ b/spring-web/src/main/java/org/springframework/web/client/RestTemplate.java @@ -100,7 +100,10 @@ import org.springframework.web.util.UriTemplateHandler; * @see RequestCallback * @see ResponseExtractor * @see ResponseErrorHandler + * @deprecated as of 7.1, in favor of {@link RestClient}. */ +@Deprecated(since = "7.1", forRemoval = true) +@SuppressWarnings("removal") public class RestTemplate extends InterceptingHttpAccessor implements RestOperations { private static final ClientRequestObservationConvention DEFAULT_OBSERVATION_CONVENTION = new DefaultClientRequestObservationConvention(); diff --git a/spring-web/src/main/java/org/springframework/web/client/StatusHandler.java b/spring-web/src/main/java/org/springframework/web/client/StatusHandler.java index 7b822b86df2..1769051c88d 100644 --- a/spring-web/src/main/java/org/springframework/web/client/StatusHandler.java +++ b/spring-web/src/main/java/org/springframework/web/client/StatusHandler.java @@ -90,7 +90,10 @@ final class StatusHandler { /** * Create a StatusHandler from a {@link ResponseErrorHandler}. + * @deprecated as of 7.1 in favor of {@link #of(Predicate, RestClient.ResponseSpec.ErrorHandler)} */ + @Deprecated(since = "7.1", forRemoval = true) + @SuppressWarnings("removal") public static StatusHandler fromErrorHandler(ResponseErrorHandler errorHandler) { Assert.notNull(errorHandler, "ResponseErrorHandler must not be null"); @@ -160,7 +163,7 @@ final class StatusHandler { return preface + bodyText; } - @SuppressWarnings("NullAway") + @SuppressWarnings({"NullAway", "removal"}) private static Function initBodyConvertFunction( ClientHttpResponse response, byte[] body, List> messageConverters) { diff --git a/spring-web/src/main/java/org/springframework/web/client/support/RestGatewaySupport.java b/spring-web/src/main/java/org/springframework/web/client/support/RestGatewaySupport.java index 3a0efc72781..ac163551d84 100644 --- a/spring-web/src/main/java/org/springframework/web/client/support/RestGatewaySupport.java +++ b/spring-web/src/main/java/org/springframework/web/client/support/RestGatewaySupport.java @@ -32,7 +32,10 @@ import org.springframework.web.client.RestTemplate; * @since 3.0 * @see #setRestTemplate * @see org.springframework.web.client.RestTemplate + * @deprecated as of 7.1, in favor of {@link org.springframework.web.client.RestClient}. */ +@Deprecated(since = "7.1", forRemoval = true) +@SuppressWarnings("removal") public class RestGatewaySupport { /** Logger available to subclasses. */ diff --git a/spring-web/src/main/java/org/springframework/web/client/support/RestTemplateAdapter.java b/spring-web/src/main/java/org/springframework/web/client/support/RestTemplateAdapter.java index e56d005888f..a44bdcfcd01 100644 --- a/spring-web/src/main/java/org/springframework/web/client/support/RestTemplateAdapter.java +++ b/spring-web/src/main/java/org/springframework/web/client/support/RestTemplateAdapter.java @@ -45,7 +45,10 @@ import org.springframework.web.util.UriBuilderFactory; * @author Olga Maciaszek-Sharma * @author Brian Clozel * @since 6.1 + * @deprecated as of 7.1 in favor of {@link RestClientAdapter}. */ +@Deprecated(since = "7.1", forRemoval = true) +@SuppressWarnings("removal") public final class RestTemplateAdapter implements HttpExchangeAdapter { private final RestTemplate restTemplate; diff --git a/spring-web/src/main/java/org/springframework/web/util/UriTemplateHandler.java b/spring-web/src/main/java/org/springframework/web/util/UriTemplateHandler.java index 952ae09a5a2..b039ce9bba0 100644 --- a/spring-web/src/main/java/org/springframework/web/util/UriTemplateHandler.java +++ b/spring-web/src/main/java/org/springframework/web/util/UriTemplateHandler.java @@ -26,7 +26,6 @@ import org.jspecify.annotations.Nullable; * * @author Rossen Stoyanchev * @since 4.2 - * @see org.springframework.web.client.RestTemplate#setUriTemplateHandler(UriTemplateHandler) */ public interface UriTemplateHandler { diff --git a/spring-web/src/main/kotlin/org/springframework/web/client/RestOperationsExtensions.kt b/spring-web/src/main/kotlin/org/springframework/web/client/RestOperationsExtensions.kt index 8fd277712f2..2ed06fc368f 100644 --- a/spring-web/src/main/kotlin/org/springframework/web/client/RestOperationsExtensions.kt +++ b/spring-web/src/main/kotlin/org/springframework/web/client/RestOperationsExtensions.kt @@ -14,6 +14,8 @@ * limitations under the License. */ +@file:Suppress("DEPRECATION", "REMOVAL") + package org.springframework.web.client import org.springframework.core.ParameterizedTypeReference diff --git a/spring-web/src/test/java/org/springframework/http/client/support/InterceptingHttpAccessorTests.java b/spring-web/src/test/java/org/springframework/http/client/support/InterceptingHttpAccessorTests.java index bbc41ae2449..ea134551d81 100644 --- a/spring-web/src/test/java/org/springframework/http/client/support/InterceptingHttpAccessorTests.java +++ b/spring-web/src/test/java/org/springframework/http/client/support/InterceptingHttpAccessorTests.java @@ -35,6 +35,7 @@ import static org.assertj.core.api.Assertions.assertThat; * * @author Brian Clozel */ +@SuppressWarnings("removal") class InterceptingHttpAccessorTests { @Test diff --git a/spring-web/src/test/java/org/springframework/web/client/DefaultResponseErrorHandlerHttpStatusTests.java b/spring-web/src/test/java/org/springframework/web/client/DefaultResponseErrorHandlerHttpStatusTests.java index 085158845e6..fe38c225617 100644 --- a/spring-web/src/test/java/org/springframework/web/client/DefaultResponseErrorHandlerHttpStatusTests.java +++ b/spring-web/src/test/java/org/springframework/web/client/DefaultResponseErrorHandlerHttpStatusTests.java @@ -57,6 +57,7 @@ import static org.springframework.http.HttpStatus.UNPROCESSABLE_ENTITY; * Tests for {@link DefaultResponseErrorHandler} handling of specific * HTTP status codes. */ +@SuppressWarnings("removal") class DefaultResponseErrorHandlerHttpStatusTests { private final DefaultResponseErrorHandler handler = new DefaultResponseErrorHandler(); diff --git a/spring-web/src/test/java/org/springframework/web/client/DefaultResponseErrorHandlerTests.java b/spring-web/src/test/java/org/springframework/web/client/DefaultResponseErrorHandlerTests.java index 5fbfacd397c..e0b313a1189 100644 --- a/spring-web/src/test/java/org/springframework/web/client/DefaultResponseErrorHandlerTests.java +++ b/spring-web/src/test/java/org/springframework/web/client/DefaultResponseErrorHandlerTests.java @@ -44,6 +44,7 @@ import static org.mockito.Mockito.mock; * @author Juergen Hoeller * @author Denys Ivano */ +@SuppressWarnings("removal") class DefaultResponseErrorHandlerTests { private final DefaultResponseErrorHandler handler = new DefaultResponseErrorHandler(); diff --git a/spring-web/src/test/java/org/springframework/web/client/ExtractingResponseErrorHandlerTests.java b/spring-web/src/test/java/org/springframework/web/client/ExtractingResponseErrorHandlerTests.java index 51f6b04a63c..50121cb9049 100644 --- a/spring-web/src/test/java/org/springframework/web/client/ExtractingResponseErrorHandlerTests.java +++ b/spring-web/src/test/java/org/springframework/web/client/ExtractingResponseErrorHandlerTests.java @@ -44,7 +44,7 @@ import static org.mockito.Mockito.mock; * * @author Arjen Poutsma */ -@SuppressWarnings("ALL") +@SuppressWarnings("removal") class ExtractingResponseErrorHandlerTests { private ExtractingResponseErrorHandler errorHandler; diff --git a/spring-web/src/test/java/org/springframework/web/client/HttpMessageConverterExtractorTests.java b/spring-web/src/test/java/org/springframework/web/client/HttpMessageConverterExtractorTests.java index 3ee14938577..fadf846063c 100644 --- a/spring-web/src/test/java/org/springframework/web/client/HttpMessageConverterExtractorTests.java +++ b/spring-web/src/test/java/org/springframework/web/client/HttpMessageConverterExtractorTests.java @@ -52,6 +52,7 @@ import static org.mockito.Mockito.mock; * @author Brian Clozel * @author Sam Brannen */ +@SuppressWarnings("removal") class HttpMessageConverterExtractorTests { private final HttpMessageConverter converter = mock(); diff --git a/spring-web/src/test/java/org/springframework/web/client/RestClientBuilderTests.java b/spring-web/src/test/java/org/springframework/web/client/RestClientBuilderTests.java index 041351c5311..c83371ca4f1 100644 --- a/spring-web/src/test/java/org/springframework/web/client/RestClientBuilderTests.java +++ b/spring-web/src/test/java/org/springframework/web/client/RestClientBuilderTests.java @@ -49,7 +49,7 @@ import static org.assertj.core.api.Assertions.fail; */ class RestClientBuilderTests { - @SuppressWarnings("unchecked") + @SuppressWarnings({"unchecked", "removal"}) @Test void createFromRestTemplate() { JettyClientHttpRequestFactory requestFactory = new JettyClientHttpRequestFactory(); @@ -90,6 +90,7 @@ class RestClientBuilderTests { } @Test + @SuppressWarnings("removal") void defaultUriBuilderFactory() { RestTemplate restTemplate = new RestTemplate(); diff --git a/spring-web/src/test/java/org/springframework/web/client/RestTemplateIntegrationTests.java b/spring-web/src/test/java/org/springframework/web/client/RestTemplateIntegrationTests.java index 7a9c98567ad..928b9d23196 100644 --- a/spring-web/src/test/java/org/springframework/web/client/RestTemplateIntegrationTests.java +++ b/spring-web/src/test/java/org/springframework/web/client/RestTemplateIntegrationTests.java @@ -82,6 +82,7 @@ import static org.springframework.http.MediaType.MULTIPART_MIXED; * @author Brian Clozel * @author Sam Brannen */ +@SuppressWarnings("removal") class RestTemplateIntegrationTests extends AbstractMockWebServerTests { @Retention(RetentionPolicy.RUNTIME) diff --git a/spring-web/src/test/java/org/springframework/web/client/RestTemplateObservationTests.java b/spring-web/src/test/java/org/springframework/web/client/RestTemplateObservationTests.java index 442fe5593ce..7b9fc0817d6 100644 --- a/spring-web/src/test/java/org/springframework/web/client/RestTemplateObservationTests.java +++ b/spring-web/src/test/java/org/springframework/web/client/RestTemplateObservationTests.java @@ -55,6 +55,7 @@ import static org.springframework.http.HttpMethod.GET; * Tests for the client HTTP observations with {@link RestTemplate}. * @author Brian Clozel */ +@SuppressWarnings("removal") class RestTemplateObservationTests { diff --git a/spring-web/src/test/java/org/springframework/web/client/RestTemplateTests.java b/spring-web/src/test/java/org/springframework/web/client/RestTemplateTests.java index 241b8146497..cfeb32f892f 100644 --- a/spring-web/src/test/java/org/springframework/web/client/RestTemplateTests.java +++ b/spring-web/src/test/java/org/springframework/web/client/RestTemplateTests.java @@ -83,7 +83,7 @@ import static org.springframework.http.MediaType.parseMediaType; * @author Brian Clozel * @author Sam Brannen */ -@SuppressWarnings("unchecked") +@SuppressWarnings({"unchecked", "removal"}) class RestTemplateTests { private final ClientHttpRequestFactory requestFactory = mock(); diff --git a/spring-web/src/test/java/org/springframework/web/client/support/RestClientAdapterTests.java b/spring-web/src/test/java/org/springframework/web/client/support/RestClientAdapterTests.java index f04ec866273..b097c958d48 100644 --- a/spring-web/src/test/java/org/springframework/web/client/support/RestClientAdapterTests.java +++ b/spring-web/src/test/java/org/springframework/web/client/support/RestClientAdapterTests.java @@ -80,7 +80,7 @@ import static org.assertj.core.api.Assertions.assertThatThrownBy; * @author Rossen Stoyanchev * @author Brian Clozel */ -@SuppressWarnings("JUnitMalformedDeclaration") +@SuppressWarnings({"JUnitMalformedDeclaration", "removal"}) class RestClientAdapterTests { private final MockWebServer anotherServer = new MockWebServer(); diff --git a/spring-web/src/test/kotlin/org/springframework/web/client/RestOperationsExtensionsTests.kt b/spring-web/src/test/kotlin/org/springframework/web/client/RestOperationsExtensionsTests.kt index 0771be69dfe..4a98fce0dad 100644 --- a/spring-web/src/test/kotlin/org/springframework/web/client/RestOperationsExtensionsTests.kt +++ b/spring-web/src/test/kotlin/org/springframework/web/client/RestOperationsExtensionsTests.kt @@ -34,6 +34,7 @@ import kotlin.reflect.jvm.kotlinFunction * * @author Sebastien Deleuze */ +@Suppress("REMOVAL", "DEPRECATION") class RestOperationsExtensionsTests { val template = mockk() diff --git a/spring-web/src/test/kotlin/org/springframework/web/client/support/KotlinRestTemplateHttpServiceProxyTests.kt b/spring-web/src/test/kotlin/org/springframework/web/client/support/KotlinRestTemplateHttpServiceProxyTests.kt index 4b11f2def0a..0da04bb11b1 100644 --- a/spring-web/src/test/kotlin/org/springframework/web/client/support/KotlinRestTemplateHttpServiceProxyTests.kt +++ b/spring-web/src/test/kotlin/org/springframework/web/client/support/KotlinRestTemplateHttpServiceProxyTests.kt @@ -28,7 +28,6 @@ import org.springframework.http.ResponseEntity import org.springframework.util.LinkedMultiValueMap import org.springframework.util.MultiValueMap import org.springframework.web.bind.annotation.* -import org.springframework.web.client.RestTemplate import org.springframework.web.multipart.MultipartFile import org.springframework.web.service.annotation.GetExchange import org.springframework.web.service.annotation.PostExchange @@ -46,6 +45,7 @@ import java.util.* * * @author Olga Maciaszek-Sharma */ +@Suppress("REMOVAL", "DEPRECATION") class KotlinRestTemplateHttpServiceProxyTests { private lateinit var server: MockWebServer @@ -65,7 +65,7 @@ class KotlinRestTemplateHttpServiceProxyTests { } private fun initTestService(): TestService { - val restTemplate = RestTemplate() + val restTemplate = org.springframework.web.client.RestTemplate() restTemplate.uriTemplateHandler = DefaultUriBuilderFactory(server.url("/").toString()) return HttpServiceProxyFactory.builder() .exchangeAdapter(RestTemplateAdapter.create(restTemplate)) diff --git a/spring-websocket/src/main/java/org/springframework/web/socket/sockjs/client/RestTemplateXhrTransport.java b/spring-websocket/src/main/java/org/springframework/web/socket/sockjs/client/RestTemplateXhrTransport.java index 089f7b9c8c2..14e9134b285 100644 --- a/spring-websocket/src/main/java/org/springframework/web/socket/sockjs/client/RestTemplateXhrTransport.java +++ b/spring-websocket/src/main/java/org/springframework/web/socket/sockjs/client/RestTemplateXhrTransport.java @@ -53,7 +53,10 @@ import org.springframework.web.socket.sockjs.frame.SockJsFrame; * * @author Rossen Stoyanchev * @since 4.1 + * @deprecated as of 7.1 in favor of {@link RestClientXhrTransport}. */ +@Deprecated(since = "7.1", forRemoval = true) +@SuppressWarnings("removal") public class RestTemplateXhrTransport extends AbstractXhrTransport { private final RestOperations restTemplate; diff --git a/spring-websocket/src/test/java/org/springframework/web/socket/sockjs/client/RestTemplateXhrTransportTests.java b/spring-websocket/src/test/java/org/springframework/web/socket/sockjs/client/RestTemplateXhrTransportTests.java index b8180ee4410..59f2d673f83 100644 --- a/spring-websocket/src/test/java/org/springframework/web/socket/sockjs/client/RestTemplateXhrTransportTests.java +++ b/spring-websocket/src/test/java/org/springframework/web/socket/sockjs/client/RestTemplateXhrTransportTests.java @@ -67,6 +67,7 @@ import static org.mockito.Mockito.verifyNoMoreInteractions; * * @author Rossen Stoyanchev */ +@SuppressWarnings("removal") class RestTemplateXhrTransportTests { private static final JacksonJsonSockJsMessageCodec CODEC = new JacksonJsonSockJsMessageCodec(); @@ -201,6 +202,7 @@ class RestTemplateXhrTransportTests { } + @SuppressWarnings("removal") private static class TestRestTemplate extends RestTemplate { private Queue responses = new LinkedBlockingDeque<>();