graphql / graphql-js / 2784

99%

master: 98%

push

travis-ci

Validate schema root types and directives

This moves validation out of GraphQLSchema's constructor (but not yet from other type constructors), which is responsible for root type validation and interface implementation checking.

Reduces time to construct GraphQLSchema significantly, shifting the time to validation.

This also allows for much looser rules within the schema builders, which implicitly validate while trying to adhere to flow types. Instead we use any casts to loosen the rules to defer that to validation where errors can be richer.

This also loosens the rule that a schema can only be constructed if it has a query type, moving that to validation as well. That makes flow typing slightly less nice, but allows for incremental schema building which is valuable

3969 of 4483 branches covered (88.53%)

9566 of 9691 relevant lines covered (98.71%)

1297.03 hits per line