graphql / graphql-js / 3315

99%

master: 98%

push

travis-ci

Changes based on feedback:

This preserves the behavior of explicit null values taking precedence over default values, greatly reducing the scope of breaking changes.

This adds a change to argument coercion to enforce null checking on variable valuesand removes one validation rule: VariablesDefaultValueAllowed. This allows supplying default values to non-nullable variables, widening the gap between "required" and "nullable". It also changes the validation rules for arguments - allowing non-null arguments with default values to be omitted.

This preserves all existing behavior (with the critical exception of no longer allowing `null` values into non-null arguments) while allowing queries which were previously considered invalid to be valid.

3664 of 4059 branches covered (90.27%)

10693 of 10834 relevant lines covered (98.7%)

1076.77 hits per line