Commit 6e2ecb8a authored by Phillip Webb's avatar Phillip Webb

Fix broken endpoint integration tests

Ensure that JSON response is returned when extracting data from the
error details.

See gh-12513
parent 72e36ead
...@@ -168,7 +168,8 @@ public abstract class AbstractWebEndpointIntegrationTests<T extends Configurable ...@@ -168,7 +168,8 @@ public abstract class AbstractWebEndpointIntegrationTests<T extends Configurable
public void readOperationWithMappingFailureProducesBadRequestResponse() { public void readOperationWithMappingFailureProducesBadRequestResponse() {
load(QueryEndpointConfiguration.class, (client) -> { load(QueryEndpointConfiguration.class, (client) -> {
WebTestClient.BodyContentSpec body = client.get().uri("/query?two=two") WebTestClient.BodyContentSpec body = client.get().uri("/query?two=two")
.exchange().expectStatus().isBadRequest().expectBody(); .accept(MediaType.APPLICATION_JSON).exchange().expectStatus()
.isBadRequest().expectBody();
validateErrorBody(body, HttpStatus.BAD_REQUEST, "/endpoints/query", validateErrorBody(body, HttpStatus.BAD_REQUEST, "/endpoints/query",
"Missing parameters: one"); "Missing parameters: one");
}); });
...@@ -290,7 +291,8 @@ public abstract class AbstractWebEndpointIntegrationTests<T extends Configurable ...@@ -290,7 +291,8 @@ public abstract class AbstractWebEndpointIntegrationTests<T extends Configurable
public void readOperationWithMissingRequiredParametersReturnsBadRequestResponse() { public void readOperationWithMissingRequiredParametersReturnsBadRequestResponse() {
load(RequiredParameterEndpointConfiguration.class, (client) -> { load(RequiredParameterEndpointConfiguration.class, (client) -> {
WebTestClient.BodyContentSpec body = client.get().uri("/requiredparameters") WebTestClient.BodyContentSpec body = client.get().uri("/requiredparameters")
.exchange().expectStatus().isBadRequest().expectBody(); .accept(MediaType.APPLICATION_JSON).exchange().expectStatus()
.isBadRequest().expectBody();
validateErrorBody(body, HttpStatus.BAD_REQUEST, validateErrorBody(body, HttpStatus.BAD_REQUEST,
"/endpoints/requiredparameters", "Missing parameters: foo"); "/endpoints/requiredparameters", "Missing parameters: foo");
}); });
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment