Skip to content

Commit 03eb741

Browse files
committed
Fixing tests after merge
1 parent a1a33ff commit 03eb741

File tree

2 files changed

+6
-5
lines changed

2 files changed

+6
-5
lines changed

spring-cloud-gateway-server-mvc/src/main/java/org/springframework/cloud/gateway/server/mvc/filter/BeforeFilterFunctions.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -350,7 +350,7 @@ public static Function<ServerRequest, ServerRequest> rewriteRequestParameter(Str
350350
queryParams.add(name, replacement);
351351
}
352352

353-
MultiValueMap<String, String> encodedQueryParams = UriUtils.encodeQueryParams(queryParams);
353+
MultiValueMap<String, String> encodedQueryParams = MvcUtils.encodeQueryParams(queryParams);
354354
URI rewrittenUri = UriComponentsBuilder.fromUri(request.uri())
355355
.replaceQueryParams(unmodifiableMultiValueMap(encodedQueryParams))
356356
.build(true)

spring-cloud-gateway-server-mvc/src/test/java/org/springframework/cloud/gateway/server/mvc/filter/BeforeFilterFunctionsTests.java

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -117,9 +117,9 @@ void rewriteOnlyFirstRequestParameter() {
117117
@Test
118118
void rewriteEncodedRequestParameter() {
119119
MockHttpServletRequest servletRequest = MockMvcRequestBuilders.get("http://localhost/path")
120-
.param("foo", "bar")
121-
.param("baz[]", "qux[]")
122-
.param("quux", "corge+")
120+
.param("foo[]", "bar")
121+
.param("baz", "qux")
122+
.param("quux", "corge+")
123123
.buildRequest(null);
124124

125125
ServerRequest request = ServerRequest.create(servletRequest, Collections.emptyList());
@@ -128,7 +128,8 @@ void rewriteEncodedRequestParameter() {
128128

129129
assertThat(result.param("foo[]")).isPresent().hasValue("replacement[]");
130130
assertThat(result.param("quux")).isPresent().hasValue("corge+");
131-
assertThat(result.uri().toString()).hasToString("http://localhost/path?baz=qux&foo%5B%5D=replacement%5B%5D&quux=corge%2B");
131+
assertThat(result.uri().toString())
132+
.hasToString("http://localhost/path?quux=corge%2B&baz=qux&foo%5B%5D=replacement%5B%5D");
132133
}
133134

134135
@Test

0 commit comments

Comments
 (0)