Test recursion avoidance correctly #29
leebyron
merged
1 commit into
graphql:master
from
dittos:fix-recursion-test
Test recursion avoidance correctly
0b4fb811
leebyron9 years ago
Great catch, thanks!
leebyron9 years ago
Yes, that's correct. This is testing the behavior of non-verified queries, to ensure behavior is correct in the case a query is executed before being validated.
If validated, the fragment recursion would have been caught before execution, as would the mistyped type information.
Login to write a write a comment.
Reviewers
No reviews
Assignees
No one assigned
Labels
CLA Signed
Milestone
No milestone
DataTypeis unknown type, so...Fragfragment spread was ignored and recursion was not tested correctly.Additional question: is it intended to just ignore a fragment on the unknown type?