Skip to content

Fixes #763 QueryTraversal problem#765

Merged
bbakerman merged 2 commits into
graphql-java:masterfrom
bbakerman:763-query-traversal-fix
Oct 17, 2017
Merged

Fixes #763 QueryTraversal problem#765
bbakerman merged 2 commits into
graphql-java:masterfrom
bbakerman:763-query-traversal-fix

Conversation

@bbakerman

@bbakerman bbakerman commented Oct 10, 2017

Copy link
Copy Markdown
Member

See #763

subFoo: String
}
""")
def visitor = Mock(QueryVisitor)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

never used according to idea

.build();
}

/**

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was used in at least 2 spots (again according to IDEA) so I moved it into a common place with better common logic and type handling

@bbakerman bbakerman changed the title Added tests for #763 that currently fail Fixes #763 QueryTraversal problem Oct 11, 2017
@bbakerman bbakerman merged commit 7e9fcf8 into graphql-java:master Oct 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant