Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

(test) O3-3766: Fill a form with a browser slightly ahead of time. #1958

Merged
merged 4 commits into from
Aug 20, 2024

Conversation

kdaud
Copy link
Member

@kdaud kdaud commented Aug 9, 2024

Requirements

  • This PR has a title that briefly describes the work done including the ticket number. If there is a ticket, make sure your PR title includes a conventional commit label. See existing PR titles for inspiration.
  • My work conforms to the OpenMRS 3.0 Styleguide and design documentation.
  • My work includes tests or is validated by existing tests.

Summary

This PR adds an E2E test for a clinical form should saving inputs even if the encounter time is after the current time. The test covers the following user experience:
Moving time forward by 1 minute in the testing environment(browser)
Navigating to the Clinical forms on the patient chart page
Launching the "Laboratory Test Results" form
Filling in lab test results
Submitting the form
Verifying: that no error is thrown when attempting to save the obs
And also verifying the presence of a success notification

@denniskigen there was an issue (O3-3001) where obs couldn't be saved when encounter time is after the current time. The forms are now using RFE but we can add this test case to catch such issue in-case it popup.

Screenshots

Related Issue

https://issues.openmrs.org/browse/O3-3766

Other

@kdaud kdaud requested a review from denniskigen August 9, 2024 10:10
@kdaud kdaud changed the title O3-3766: (test) Clinical form should save inputs even if the encounter time is after the current time O3-3766: (test) Fill a form with a browser slightly ahead of time. Aug 20, 2024
@jayasanka-sack jayasanka-sack merged commit bd2020f into openmrs:main Aug 20, 2024
3 checks passed
@jayasanka-sack
Copy link
Member

Thanks @kdaud !

@jayasanka-sack jayasanka-sack changed the title O3-3766: (test) Fill a form with a browser slightly ahead of time. (test) O3-3766: Fill a form with a browser slightly ahead of time. Aug 20, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants