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

C2C-345: E2E tests for Bahmni-Odoo flows #13

Merged
merged 1 commit into from
Aug 23, 2024
Merged

C2C-345: E2E tests for Bahmni-Odoo flows #13

merged 1 commit into from
Aug 23, 2024

Conversation

kdaud
Copy link
Contributor

@kdaud kdaud commented Aug 22, 2024

Ticket: https://mekomsolutions.atlassian.net/browse/C2C-345

Description: This PR adds E2E tests for Bahmni-Odoo flows and covers the following test cases;

  • Ordering a lab test for a Bahmni patient creates the corresponding Odoo customer.
  • Editing the details of a Bahmni patient with a synced lab order edits the corresponding Odoo customer details.
  • Revising a synced Bahmni lab order edits the corresponding Odoo quotation line.
  • Discontinuing a synced Bahmni lab order for an Odoo customer removes the corresponding quotation.
  • Ordering a drug for a Bahmni patient creates the corresponding Odoo customer customer with a filled quotation.
  • Editing the details of an Bahmni patient with a synced drug order edits the corresponding Odoo customer details.
  • Revising a synced Bahmni drug order edits the corresponding Odoo quotation line.
  • Discontinuing a synced Bahmni drug order for an Odoo customer removes the corresponding quotation.

@kdaud kdaud force-pushed the C2C-345 branch 5 times, most recently from c4f26c7 to 4f9b3f8 Compare August 23, 2024 02:47
@kdaud kdaud marked this pull request as ready for review August 23, 2024 02:50
Copy link
Member

@Ruhanga Ruhanga left a comment

Choose a reason for hiding this comment

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

Generally LGTM.

@kdaud kdaud merged commit a50ad7c into master Aug 23, 2024
1 check passed
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.

2 participants