Skip to content

Commit c39a23a

Browse files
committed
Updated tests for framework changes
1 parent c713d39 commit c39a23a

File tree

3 files changed

+21
-10
lines changed

3 files changed

+21
-10
lines changed

spring-cloud-gateway-server/src/test/java/org/springframework/cloud/gateway/filter/factory/TokenRelayGatewayFilterFactoryTests.java

Lines changed: 14 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,11 @@ public void whenPrincipalExistsAuthorizationHeaderAdded() {
121121
GatewayFilter filter = new TokenRelayGatewayFilterFactory(objectProvider).apply();
122122
filter.filter(exchange, filterChain).block(TIMEOUT);
123123

124-
assertThat(request.getHeaders()).containsEntry(HttpHeaders.AUTHORIZATION,
124+
ArgumentCaptor<ServerWebExchange> sweCaptor = ArgumentCaptor.forClass(ServerWebExchange.class);
125+
verify(filterChain).filter(sweCaptor.capture());
126+
ServerWebExchange mutatedServerWebExchange = sweCaptor.getValue();
127+
128+
assertThat(mutatedServerWebExchange.getRequest().getHeaders()).containsEntry(HttpHeaders.AUTHORIZATION,
125129
Collections.singletonList("Bearer mytoken"));
126130

127131
ArgumentCaptor<OAuth2AuthorizeRequest> authorizeRequestCaptor = ArgumentCaptor
@@ -161,7 +165,11 @@ public void whenClientRegistrationIdConfiguredAuthorizationHeaderAdded() {
161165
GatewayFilter filter = new TokenRelayGatewayFilterFactory(objectProvider).apply(config);
162166
filter.filter(exchange, filterChain).block(TIMEOUT);
163167

164-
assertThat(request.getHeaders()).containsEntry(HttpHeaders.AUTHORIZATION,
168+
ArgumentCaptor<ServerWebExchange> sweCaptor = ArgumentCaptor.forClass(ServerWebExchange.class);
169+
verify(filterChain).filter(sweCaptor.capture());
170+
ServerWebExchange mutatedServerWebExchange = sweCaptor.getValue();
171+
172+
assertThat(mutatedServerWebExchange.getRequest().getHeaders()).containsEntry(HttpHeaders.AUTHORIZATION,
165173
Collections.singletonList("Bearer mytoken"));
166174

167175
ArgumentCaptor<OAuth2AuthorizeRequest> authorizeRequestCaptor = ArgumentCaptor
@@ -212,7 +220,10 @@ public void whenPrincipalIsNotOAuth2AuthenticationTokenAndClientRegistrationIdCo
212220
GatewayFilter filter = new TokenRelayGatewayFilterFactory(objectProvider).apply(config);
213221
filter.filter(exchange, filterChain).block(TIMEOUT);
214222

215-
assertThat(request.getHeaders()).containsEntry(HttpHeaders.AUTHORIZATION,
223+
ArgumentCaptor<ServerWebExchange> sweCaptor = ArgumentCaptor.forClass(ServerWebExchange.class);
224+
verify(filterChain).filter(sweCaptor.capture());
225+
ServerWebExchange mutatedServerWebExchange = sweCaptor.getValue();
226+
assertThat(mutatedServerWebExchange.getRequest().getHeaders()).containsEntry(HttpHeaders.AUTHORIZATION,
216227
Collections.singletonList("Bearer mytoken"));
217228

218229
ArgumentCaptor<OAuth2AuthorizeRequest> authorizeRequestCaptor = ArgumentCaptor

spring-cloud-gateway-server/src/test/java/org/springframework/cloud/gateway/filter/factory/rewrite/ModifyRequestBodyGatewayFilterFactoryTests.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -86,10 +86,10 @@ public void modifyRequestBodyToLarge() {
8686
.body(BodyInserters.fromValue("request"))
8787
.exchange()
8888
.expectStatus()
89-
.isEqualTo(HttpStatus.INTERNAL_SERVER_ERROR)
89+
.isEqualTo(HttpStatus.PAYLOAD_TOO_LARGE)
9090
.expectBody()
91-
.jsonPath("message")
92-
.isEqualTo("Exceeded limit on max bytes to buffer : 13");
91+
.jsonPath("error")
92+
.isEqualTo("Payload Too Large");
9393
}
9494

9595
@Test

spring-cloud-gateway-server/src/test/java/org/springframework/cloud/gateway/filter/factory/rewrite/ModifyResponseBodyGatewayFilterFactoryTests.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -87,18 +87,18 @@ public void testModificationOfContentType() {
8787
}
8888

8989
@Test
90-
public void modifyResponeBodyToLarge() {
90+
public void modifyResponseBodyToLarge() {
9191
testClient.post()
9292
.uri("/post")
9393
.header("Host", "www.modifyresponsebodyjavatoolarge.org")
9494
.header(HttpHeaders.CONTENT_TYPE, MediaType.APPLICATION_JSON_VALUE)
9595
.body(BodyInserters.fromValue(toLarge))
9696
.exchange()
9797
.expectStatus()
98-
.isEqualTo(HttpStatus.INTERNAL_SERVER_ERROR)
98+
.isEqualTo(HttpStatus.PAYLOAD_TOO_LARGE)
9999
.expectBody()
100-
.jsonPath("message")
101-
.isEqualTo("Exceeded limit on max bytes to buffer : 40");
100+
.jsonPath("error")
101+
.isEqualTo("Payload Too Large");
102102
}
103103

104104
@EnableAutoConfiguration

0 commit comments

Comments
 (0)