graphql / graphql-js / 3528

99%

master: 98%

push

travis-ci

BREAKING/BUGFIX Strict coercion of scalar types

This no longer accepts incoming variable values in a potentially lossy way, mirroring the existing behavior for literals. This fixes an issue with GraphQL.js being not spec compliant.

This is breaking since servers which used to accept incorrect variable values will now return errors to clients.

Serialization of values is not affected in the same way, since this is not a client-visible behavior.

As a bonus, this adds unique serialization and coercion functions for the ID type, allowing to be more restrictive on numeric types and un-stringable object types, while directly supporting valueOf() methods (ala MongoDB). The changes to how the ID type serializes and coerces data could be potentially breaking.

Fixes #1324

3448 of 3706 branches covered (93.04%)

87 of 87 new or added lines in 6 files covered. (100.0%)

10784 of 10925 relevant lines covered (98.71%)

1100.85 hits per line