From ea2864c73eb9262b203766102f3e7b535f760c2f Mon Sep 17 00:00:00 2001 From: Rossen Stoyanchev Date: Tue, 15 Aug 2017 15:08:23 +0200 Subject: [PATCH] Polish --- .../client/match/MockRestRequestMatchers.java | 17 ++++++++++------- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/spring-test/src/main/java/org/springframework/test/web/client/match/MockRestRequestMatchers.java b/spring-test/src/main/java/org/springframework/test/web/client/match/MockRestRequestMatchers.java index 0838f26c1d5..d67c6762dd9 100644 --- a/spring-test/src/main/java/org/springframework/test/web/client/match/MockRestRequestMatchers.java +++ b/spring-test/src/main/java/org/springframework/test/web/client/match/MockRestRequestMatchers.java @@ -31,9 +31,10 @@ import org.springframework.util.Assert; import org.springframework.util.MultiValueMap; import org.springframework.web.util.UriComponentsBuilder; -import static org.hamcrest.MatcherAssert.*; +import static org.hamcrest.MatcherAssert.assertThat; import static org.junit.Assert.assertNotNull; -import static org.springframework.test.util.AssertionErrors.*; +import static org.springframework.test.util.AssertionErrors.assertEquals; +import static org.springframework.test.util.AssertionErrors.assertTrue; /** * Static factory methods for {@link RequestMatcher} classes. Typically used to @@ -77,7 +78,7 @@ public abstract class MockRestRequestMatchers { } /** - * Assert the request URI string. + * Assert the request URI matches the given string. * @param expectedUri the expected URI * @return the request matcher */ @@ -87,15 +88,17 @@ public abstract class MockRestRequestMatchers { } /** - * Assert the request URI string. - * @param expectedUri the expected URI + * Variant of {@link #requestTo(URI)} that prepares the URI from a URI + * template plus optional variables via {@link UriComponentsBuilder} + * including encoding. + * @param expectedUri the expected URI template * @param uriVars zero or more URI variables to populate the expected URI * @return the request matcher */ public static RequestMatcher requestToUriTemplate(final String expectedUri, final Object... uriVars) { Assert.notNull(expectedUri, "'uri' must not be null"); - String uri = UriComponentsBuilder.fromUriString(expectedUri).buildAndExpand(uriVars).encode().toString(); - return request -> assertEquals("Request URI", uri, request.getURI().toString()); + URI uri = UriComponentsBuilder.fromUriString(expectedUri).buildAndExpand(uriVars).encode().toUri(); + return requestTo(uri); } /**