From 5d282fd99e24832aef5b48ea33ebd59d3943c82f Mon Sep 17 00:00:00 2001 From: rstoyanchev Date: Thu, 28 Apr 2022 20:41:05 +0100 Subject: [PATCH] Improve readability of assertion messages Closes gh-361 --- .../graphql/support/DefaultExecutionGraphQlResponse.java | 5 +++++ .../springframework/graphql/client/GraphQlClientTests.java | 3 ++- 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/spring-graphql/src/main/java/org/springframework/graphql/support/DefaultExecutionGraphQlResponse.java b/spring-graphql/src/main/java/org/springframework/graphql/support/DefaultExecutionGraphQlResponse.java index 752876a7..3bcee12f 100644 --- a/spring-graphql/src/main/java/org/springframework/graphql/support/DefaultExecutionGraphQlResponse.java +++ b/spring-graphql/src/main/java/org/springframework/graphql/support/DefaultExecutionGraphQlResponse.java @@ -152,6 +152,11 @@ public class DefaultExecutionGraphQlResponse extends AbstractGraphQlResponse imp return (this.delegate.getExtensions() != null ? this.delegate.getExtensions() : Collections.emptyMap()); } + @Override + public String toString() { + return this.delegate.toString(); + } + } diff --git a/spring-graphql/src/test/java/org/springframework/graphql/client/GraphQlClientTests.java b/spring-graphql/src/test/java/org/springframework/graphql/client/GraphQlClientTests.java index f65b5e9a..712df860 100644 --- a/spring-graphql/src/test/java/org/springframework/graphql/client/GraphQlClientTests.java +++ b/spring-graphql/src/test/java/org/springframework/graphql/client/GraphQlClientTests.java @@ -205,7 +205,8 @@ public class GraphQlClientTests extends GraphQlClientTestSupport { assertThat(nameField.getError().getParsedPath()).containsExactly("me", "name"); assertThatThrownBy(() -> nameField.toEntity(String.class)) .as("Decoding field null with direct field error should be rejected") - .isInstanceOf(FieldAccessException.class); + .isInstanceOf(FieldAccessException.class) + .hasMessageContaining("Test error"); ClientResponseField nonExistingField = response.field("me.name.other"); assertThat(nonExistingField.hasValue()).isFalse();