Skip to content

Conversation

@weilu
Copy link
Contributor

@weilu weilu commented Aug 7, 2025

With this PR E2E tests now cover the following workflows:

Auth:

  • OCM-1122: successful log in (pre-existing)
  • OCM-1123: non-existent username
  • OCM-1123: incorrect password
  • OCM-1125: Empty credentials
  • OCM-1126: Cover password input
  • log out

Admin:

  • configure individual json schema
  • configure menu - menu & submenu items, order, and position left
  • configure maker-checker for individual and group import
  • create activities
  • update an activity
  • delete an activity

Programs, Individuals, Households:

  • create an individual program (benefit plan)
  • OCM-1169: update an individual program
  • OCM-1168: delete an individual program
  • create a group/household program
  • update a household program
  • delete a household program
  • OCM-1128: import individuals (csv)
  • import households (via individual csv import)

@weilu weilu linked an issue Aug 7, 2025 that may be closed by this pull request
29 tasks
@ajinkyagadewar
Copy link

@weilu CI pipeline and SonarCloud is failing. Could you please resolve both the issues before we start reviewing PR

@weilu
Copy link
Contributor Author

weilu commented Aug 21, 2025

@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.

@weilu
Copy link
Contributor Author

weilu commented Aug 21, 2025

@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%)
21.1% Duplication on New Code (required ≤ 3%)

@weilu weilu force-pushed the feature/coremis-e2e branch from 352b911 to 5f2f7df Compare September 5, 2025 21:40
@weilu weilu marked this pull request as ready for review September 11, 2025 15:29
@sonarqubecloud
Copy link

@weilu weilu requested a review from zikani03 September 11, 2025 15:38
@weilu
Copy link
Contributor Author

weilu commented Sep 11, 2025

This PR is ready for review. The only failing E2E test caught an actual bug with the error message when the entered password is incorrect during login:

Sign in and out -- Rejects incorrect password (OCM-1124) (failed)

Filed https://openimis.atlassian.net/browse/OP-2566

@weilu
Copy link
Contributor Author

weilu commented Oct 21, 2025

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?

@zikani03
Copy link

@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

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.

Add end-to-end tests on program and project enrollment

4 participants