graphql / graphql-js / 2665

98%

push

travis-ci

(Potentially Breaking) Allow serializing scalars as null. (#1104)

This changes the check for null/undefined to a check for undefined to determine if scalar serialization was successful or not, allowing `null` to be returned from serialize() without indicating error.

This is potentially breaking for any existing custom scalar which returned `null` from `serialize()` to indicate failure. To account for this change, it should either throw an error or return `undefined`.

Fixes #1057
Closes #1066

3871 of 4359 branches covered (88.8%)

9441 of 9552 relevant lines covered (98.84%)

1281.35 hits per line