1
98%
master: 98%

Ran 19 Jul 2016 10:40PM UTC

Files 137

Run time 8s

Badge

Embed ▾

Committed 19 Jul 2016 10:27PM UTC coverage: 98.302% (-0.01%) from 98.312%

Job # 1082.1

Build Type

push

travis-ci

Committed by

web-flow

Commit Message

[FIX] astFromValue uses `serialize()` and requires type. (#441)

A bug was reported in #435 and reproduced in #438 which illustrates an issue where defaultValue would serialize incorrectly or even throw an invariant violation when working with input objects or enums backed by values other than their names.

The root of the issue is that astFromValue accepts an *internal* value, but then produces an AST directly from that value without serializing it to an *external* value first via calling `type.serialize(value)`. This critical missing step is responsible for the reported issue.

In order to fix correctly, this refactored this method to be a closer dual to [`valueFromAST`](https://github.com/graphql/graphql-js/blob/master/src/utilities/valueFromAST.js), relying on the GraphQL type rather than the JavaScript type to guide coercion. As a consequence, `astFromValue` now *requires* a type rather than accepting one optionally.

Run Details

7178 of 7302 relevant lines covered (98.3%)

1371.15 hits per line

graphql / graphql-js / 1082 / 1
98%
master: 98%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1082.1

graphql / graphql-js / 1082 / 1 98% master: 98%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1082.1

graphql / graphql-js / 1082 / 1
98%
master: 98%

README BADGES
x