Polishing contribution

Closes: gh-1196

Author: rstoyanchev

spring-graphql/src/main/java/org/springframework/graphql/data/ArgumentValue.java

@@ -137,21 +137,10 @@ public int hashCode() {
 		return result;
 	}
 
-	/**
-	 * Returns a non-empty string representation of this {@code ArgumentValue}
-	 * suitable for debugging.
-	 *
-	 * @return the string representation of this instance
-	 */
 	@Override
 	public String toString() {
-		if (this.omitted) {
-			return "ArgumentValue.omitted";
-		}
-		if (this.value == null){
-			return "ArgumentValue.empty";
-		}
-		return "ArgumentValue[%s]".formatted(this.value);
+		String v = ((this.value != null) ? this.value.toString() : (this.omitted) ? "omitted" : "empty");
+		return "ArgumentValue[" + v + "]";
 	}
 
 

spring-graphql/src/test/java/org/springframework/graphql/data/ArgumentValueTests.java

@@ -81,17 +81,9 @@ void ifPresentShouldSkipWhenOmitted() {
 	}
 
 	@Test
-	void toStringShouldReturnOmittedWhenOmitted() {
-		assertThat(ArgumentValue.omitted()).hasToString("ArgumentValue.omitted");
-	}
-
-	@Test
-	void toStringShouldReturnEmptyWhenNull() {
-		assertThat(ArgumentValue.ofNullable(null)).hasToString("ArgumentValue.empty");
-	}
-
-	@Test
-	void toStringShouldReturnValueWhenValue() {
+	void toStringValue() {
+		assertThat(ArgumentValue.omitted()).hasToString("ArgumentValue[omitted]");
+		assertThat(ArgumentValue.ofNullable(null)).hasToString("ArgumentValue[empty]");
 		assertThat(ArgumentValue.ofNullable("hello")).hasToString("ArgumentValue[hello]");
 	}
 }