Open Jacobjeevan opened 5 days ago
The changes involve significant enhancements to various Cypress test files and page objects related to facility and patient management. Key modifications include the introduction of structured data handling for facilities and patients, refactoring of test cases to improve readability and maintainability, and the addition of new methods for managing beds and patient data. The updates also streamline interactions with the UI, ensuring that the tests are more concise and organized while retaining the original functionality and error handling mechanisms.
File Path | Change Summary |
---|---|
cypress/e2e/facility_spec/FacilityCreation.cy.ts |
Added structured data handling for facility creation, refactored test cases to use createNewFacility . |
cypress/e2e/facility_spec/FacilityHomepage.cy.ts |
Introduced new imports and constants, added a test case for bed capacity badge reflection. |
cypress/e2e/facility_spec/FacilityLocation.cy.ts |
Consolidated bed management methods, updated visibility checks, streamlined test cases. |
cypress/e2e/patient_spec/PatientRegistration.cy.ts |
Refactored patient creation to use newPatientData object, enhancing data handling. |
cypress/pageobject/Facility/FacilityCreation.ts |
Introduced FacilityData interface, added methods for creating facilities with structured data. |
cypress/pageobject/Facility/FacilityHome.ts |
Updated clickFacilityNotifyButton method for improved selector handling. |
cypress/pageobject/Facility/FacilityLocation.ts |
Added addBed and editBed methods, enhanced button click methods with { force: true } . |
cypress/pageobject/Facility/FacilityManage.ts |
Added visitViewPatients method for patient management. |
cypress/pageobject/Patient/PatientCreation.ts |
Introduced PatientData interface, added methods for streamlined patient management. |
cypress/pageobject/Patient/PatientHome.ts |
Added typePatientName method for patient name input. |
cypress/pageobject/Users/ManageUserPage.ts |
Added methods for asserting facility badge content and background color. |
src/components/Facility/FacilityCard.tsx |
Added data-test-id attribute to occupancy badge text for improved testability. |
src/components/Patient/PatientHome.tsx |
Enhanced accessibility with new id attributes, refined error handling and layout. |
Objective | Addressed | Explanation |
---|---|---|
Verify the functionality of the occupancy badge on the facility card (9107) | ✅ | |
Ensure that the badge reflects the updated bed count after adding a bed (9107) | ✅ | |
Add API request verification where required using cy.intercept() (9107) |
❌ | No specific intercepts added. |
Reuse existing custom functions and those in commands.ts where possible (9107) | ❓ | Unclear if all functions reused. |
Verify that the occupancy badge displays correctly (9107) | ✅ |
FacilityCreation.cy.ts
file.FacilityHomepage.cy.ts
.PatientRegister.tsx
.PatientRegister.tsx
.locale/en.json
.needs review
, tested
🐇 In the land of tests, we hop and play,
With structured data, we clear the way.
Facilities and patients, all in a line,
Our Cypress tests now truly shine!
So here’s to the changes, both big and small,
In the world of code, we’ll conquer all! 🌟
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Name | Link |
---|---|
Latest commit | 3af5173b73902bf4c03f945e364158c1779c694c |
Latest deploy log | https://app.netlify.com/sites/care-ohc/deploys/67405b70be9f99000808c1e9 |
Deploy Preview | https://deploy-preview-9145--care-ohc.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
:wave: Hi, @Jacobjeevan, Conflicts have been detected against the base branch. Please rebase your branch against the base branch.
This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there.
:wave: Hi, @Jacobjeevan, Conflicts have been detected against the base branch. Please rebase your branch against the base branch.
This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there.
Proposed Changes
Changed ValidateName to be more inclusive (accented characters, hyphens, dots and apostrophes allowed)ex: François-O'Neil, J.R.R. Tolkien9177 is removing character validation for names anyways, so that takes care of it.
@ohcnetwork/care-fe-code-reviewers
Merge Checklist
Summary by CodeRabbit
Release Notes
New Features
Bug Fixes
Documentation
data-test-id
attributes in components.Refactor