-
Notifications
You must be signed in to change notification settings - Fork 41
Feature: add base and coremis end-to-end tests #79
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
base: develop
Are you sure you want to change the base?
Conversation
|
@weilu CI pipeline and SonarCloud is failing. Could you please resolve both the issues before we start reviewing PR |
This PR is still in draft, so it's not yet ready for review. I will mark it as ready for review (out of draft mode) when it's ready. |
|
@delcroip can we configure SonarCloud to not scan test code for test coverage? And lower the threshold for duplication of test code? 0.0% Coverage on New Code (required ≥ 80%) |
As it is all Dockerfiles and configs
As suggested by sonar scan’s security results
352b911 to
5f2f7df
Compare
|
Also give time for group creation
|
The latest docker build is broken, evident by cypress CI and local pull. @zikani03 is it a matter of release branch changes not yet merged into develop? |
|
@weilu that is possibly the cause yes, we will be merging the changes into the develop branch between now and thursday. We would like to expedite merging of a few more PRs before doing so |




With this PR E2E tests now cover the following workflows:
Auth:
Admin:
Programs, Individuals, Households: