Skip to content
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.

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

Update Test for Issue 376 to Produce Error #403

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

brianlenz
Copy link

  • Updated the test case for issue 376 so that it exposes at least one scenario where this error is occurring. I restructured the schema a bit and added further typings to mirror the schema I have been testing with.

  • Apparently the projection is added in some simple cases with findMany(), but when using findOne() (at a minimum), it appears to not work. There are now two tests: the first one fails (with findOne(), and the second one succeeds (with findMany()).

  • Updated to graphql-compose 9.0.8 to ensure testing is against the latest version.

#376
graphql-compose/graphql-compose#382

* Updated the test case for issue 376 so that it exposes at least one scenario where this error is occurring. I restructured the schema a bit and added further typings to mirror the schema I have been testing with.

* Apparently the projection is added in some simple cases with findMany(), but when using findOne() (at a minimum), it appears to not work. There are now two tests: the first one fails (with findOne(), and the second one succeeds (with findMany()).

* Updated to graphql-compose 9.0.8 to ensure testing is against the latest version.

graphql-compose#376
graphql-compose/graphql-compose#382
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