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.

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

The GraphQL service should have it's API layer tested #2362

Closed
willmcvay opened this issue Aug 13, 2020 · 0 comments · Fixed by #2407
Closed

The GraphQL service should have it's API layer tested #2362

willmcvay opened this issue Aug 13, 2020 · 0 comments · Fixed by #2407
Assignees
@vuhuucuong
Labels
chore Maintenance or housekeeping tasks front-end Relates to front end issues graphql-server
Milestone
Graph QL Alpha Re...

Comments

@willmcvay
Copy link
Contributor

Background context or User story:

We are missing tests in the GraphQL service for the api.ts files for each entity. Really need to get this service to a very high level of testing confidence as it is a public service

Specification or Acceptance Criteria:

  • API files should have unit tests
@willmcvay willmcvay added chore Maintenance or housekeeping tasks graphql-server front-end Relates to front end issues labels Aug 13, 2020
@willmcvay willmcvay added this to the Graph QL Alpha Release milestone Aug 13, 2020
@vuhuucuong vuhuucuong self-assigned this Aug 18, 2020
vuhuucuong pushed a commit that referenced this issue Aug 19, 2020
chore: #2362 graphql api layer tests
6e1ee78
@vuhuucuong vuhuucuong mentioned this issue Aug 19, 2020
Merged
14 tasks
vuhuucuong pushed a commit that referenced this issue Aug 20, 2020
chore: #2362 graphql api layer tests (#2407)
0cd3f3d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vuhuucuong vuhuucuong

Labels
chore Maintenance or housekeeping tasks front-end Relates to front end issues graphql-server
Projects
None yet
Milestone
Graph QL Alpha Release
Development

Successfully merging a pull request may close this issue.

chore: #2362 graphql api layer tests reapit/foundations
2 participants
@willmcvay @vuhuucuong