Fix typos

dondonz · dondonz · commit a05f6c419439 · 2022-11-26T15:50:27.000+11:00
diff --git a/src/main/java/graphql/scalar/GraphqlStringCoercing.java b/src/main/java/graphql/scalar/GraphqlStringCoercing.java
@@ -28,7 +28,7 @@ private String toStringImpl(Object input) {
         return String.valueOf(input);
     }
 
-    private String parseValueImpl(@NotNull Object input, Locale locale) {
+    private String parseValueImpl(@NotNull Object input, @NotNull Locale locale) {
         if (!(input instanceof String)) {
             throw new CoercingParseValueException(
                     i18nMsg(locale, "String.unexpectedRawValueType", typeName(input))
diff --git a/src/test/groovy/graphql/ScalarsStringTest.groovy b/src/test/groovy/graphql/ScalarsStringTest.groovy
@@ -75,14 +75,12 @@ class ScalarsStringTest extends Specification {
         customObject | "foo"
     }
 
-    @Unroll
-    def "String parseValue #value into #result"() {
+    def "String parseValue value into result"() {
         expect:
         Scalars.GraphQLString.getCoercing().parseValue("123ab", GraphQLContext.default, Locale.default) == "123ab"
     }
 
-    @Unroll
-    def "String parseValue #value into #result with deprecated method"() {
+    def "String parseValue value into result with deprecated method"() {
         expect:
         Scalars.GraphQLString.getCoercing().parseValue("123ab") == "123ab" // Retain deprecated method for test coverage
     }

Original file line number	Diff line number	Diff line change
`@@ -28,7 +28,7 @@ private String toStringImpl(Object input) {`
`28`	`28`	`return String.valueOf(input);`
`29`	`29`	`}`
`30`	`30`
`31`		`- private String parseValueImpl(@NotNull Object input, Locale locale) {`
	`31`	`+ private String parseValueImpl(@NotNull Object input, @NotNull Locale locale) {`
`32`	`32`	`if (!(input instanceof String)) {`
`33`	`33`	`throw new CoercingParseValueException(`
`34`	`34`	`i18nMsg(locale, "String.unexpectedRawValueType", typeName(input))`
Original file line number	Diff line number	Diff line change
`@@ -75,14 +75,12 @@ class ScalarsStringTest extends Specification {`
`75`	`75`	`customObject \| "foo"`
`76`	`76`	`}`
`77`	`77`
`78`		`- @Unroll`
`79`		`- def "String parseValue #value into #result"() {`
	`78`	`+ def "String parseValue value into result"() {`
`80`	`79`	`expect:`
`81`	`80`	`Scalars.GraphQLString.getCoercing().parseValue("123ab", GraphQLContext.default, Locale.default) == "123ab"`
`82`	`81`	`}`
`83`	`82`
`84`		`- @Unroll`
`85`		`- def "String parseValue #value into #result with deprecated method"() {`
	`83`	`+ def "String parseValue value into result with deprecated method"() {`
`86`	`84`	`expect:`
`87`	`85`	`Scalars.GraphQLString.getCoercing().parseValue("123ab") == "123ab" // Retain deprecated method for test coverage`
`88`	`86`	`}`