Polishing

Author: rstoyanchev

spring-web/src/main/java/org/springframework/http/client/BufferingClientHttpRequestFactory.java

@@ -54,6 +54,7 @@ public BufferingClientHttpRequestFactory(ClientHttpRequestFactory requestFactory
 	/**
 	 * Constructor variant with an additional predicate to decide whether to
 	 * buffer the response.
+	 * @since 7.0
 	 */
 	public BufferingClientHttpRequestFactory(
 			ClientHttpRequestFactory requestFactory,

spring-web/src/main/java/org/springframework/web/client/RestClient.java

@@ -386,8 +386,8 @@ Builder defaultStatusHandler(Predicate<HttpStatusCode> statusPredicate,
 		Builder requestInterceptors(Consumer<List<ClientHttpRequestInterceptor>> interceptorsConsumer);
 
 		/**
-		 * Enable buffering of request and response content making it possible to
-		 * read the request and the response body multiple times.
+		 * Enable buffering of request and response, aggregating all content before
+		 * it is sent, and making it possible to read the response body repeatedly.
 		 * @param predicate to determine whether to buffer for the given request
 		 * @return this builder
 		 * @since 7.0

spring-web/src/test/java/org/springframework/web/client/RestClientIntegrationTests.java

@@ -834,9 +834,8 @@ void requestInterceptorWithResponseBuffering(ClientHttpRequestFactory requestFac
 				.retrieve()
 				.body(String.class);
 
-		assertThat(result).isEqualTo("Hello Spring!");
-
 		expectRequestCount(1);
+		assertThat(result).isEqualTo("Hello Spring!");
 	}
 
 	@ParameterizedRestClientTest
@@ -860,9 +859,8 @@ void bufferContent(ClientHttpRequestFactory requestFactory) {
 					return new String(bytes, UTF_8);
 				});
 
-		assertThat(result).isEqualTo("Hello Spring!");
-
 		expectRequestCount(1);
+		assertThat(result).isEqualTo("Hello Spring!");
 	}
 
 	@ParameterizedRestClientTest