graphql / graphql-js / 2573

98%

push

travis-ci

Return normal objects for args and vars. (#1056)

This is a follow up to #1048 which addresses an issue where flow typed resolver functions would need to be aware of the fact that arguments were prototype-less. That's something we maintain for internal usage, but is more burdensome for external use. Instead, this is just more cautious when using these values internally, using hasOwnProperty, but now returns prototypefull objects to user code.

Also follows up with a few additional conversions to ObjMap, such that the only remaining uses of `{[string]: mixed}` are for variables and arguments.

3754 of 4227 branches covered (88.81%)

9153 of 9340 relevant lines covered (98.0%)

1252.02 hits per line