webonyx / graphql-php / 395

86%

Pull #245

travis-ci

(Potentially Breaking) Allow serializing scalars as null.

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`.

ref: graphql/graphql-js#1104

102 of 102 new or added lines in 14 files covered. (100.0%)

6919 of 7447 relevant lines covered (92.91%)

104.49 hits per line