Skip to content

Upgrade apollo fix ut #82

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
Open
wants to merge 4 commits into
base: alpha
Choose a base branch
from
Open

Upgrade apollo fix ut #82

wants to merge 4 commits into from

Conversation

@YizYah
Copy link
Collaborator

@YizYah YizYah commented Jul 21, 2021 •
edited by thabungm
Loading

  • Implemented Integration testing.
  • Added .env.example which must be renamed to .env
  • All env specific parameters to be put in .env
  • the parameter RUN_MOCK in .env must be set to true if we want to run with mocked data instead of calling actual API

@thabungm thabungm force-pushed the upgrade-apollo-fix-UT branch from 7599a78 to d9bb728 Compare July 22, 2021 14:59
@thabungm
fix: fix integration
ff86267 
fix integration in progress
@thabungm thabungm force-pushed the upgrade-apollo-fix-UT branch from d9bb728 to ff86267 Compare July 27, 2021 15:26
@thabungm
fix: integration test
672cec6 
integration test of mutations
@YizYah YizYah assigned YizYah and unassigned YizYah Jul 29, 2021
@thabungm thabungm assigned thabungm and YizYah and unassigned YizYah Jul 29, 2021
@thabungm thabungm requested a review from Venus713 July 29, 2021 08:16
@YizYah YizYah requested review from thabungm and removed request for Venus713 July 29, 2021 08:23
thabungm
thabungm approved these changes Jul 29, 2021
View reviewed changes
Copy link

@thabungm thabungm left a comment

Choose a reason for hiding this comment

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

Looks good to me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@thabungm thabungm thabungm approved these changes

Assignees

@thabungm thabungm

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@YizYah @thabungm