Skip to content

Implement missing executor tests #10

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
jhgg opened this issue Oct 6, 2015 · 1 comment
Closed

Implement missing executor tests #10

jhgg opened this issue Oct 6, 2015 · 1 comment
Assignees
@jhgg
Labels
enhancement
Milestone
v0.4.7

Comments

@jhgg
Copy link
Member

jhgg commented Oct 6, 2015

Looks like we're missing some tests: https://github.com/graphql/graphql-js/tree/master/src/execution/__tests__

And some stuff is not-to-spec. Going to look through this and finish implementation of all the tests that I can.
@jhgg jhgg self-assigned this Oct 6, 2015
@jhgg jhgg added this to the v0.1 milestone Oct 6, 2015
@jhgg
Copy link
Member Author

jhgg commented Oct 13, 2015

#23 outlines what needs to be done. Closing this issue in favor of that one.

@jhgg jhgg closed this as completed Oct 13, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jhgg jhgg

Labels
enhancement
Projects
None yet
Milestone
v0.4.7
Development

No branches or pull requests
1 participant
@jhgg