graphql / graphql-js / 2906

98%

push

travis-ci

Validate literals in a single rule with finer precision (#1144)

This generalizes the "arguments of correct type" and "default values of correct type" to a single rule "values of correct type" which has been re-written to rely on a traversal rather than the utility function `isValidLiteralValue`. To reduce breaking scope, this does not remove that utility even though it's no longer used directly within the library. Since the default values rule included another validation rule that rule was renamed to a more apt "variable default value allowed".

This also includes the original errors from custom scalars in the validation error output, solving the remainder of #821.

Closes #821

4078 of 4568 branches covered (89.27%)

10096 of 10230 relevant lines covered (98.69%)

1581.41 hits per line