(test) O3-2722: Add E2E test for returning to home page and patient list page from the patient chart

[kdaud]

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

Screenshots

Related Issue

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

Other

[kdaud]

@jayasanka-sack this PR is waiting review. Thanks

[kdaud]

@vasharma05 @denniskigen @brandones could you review this PR? Thanks.

[brandones]

Where'd @jayasanka-sack go?

[vasharma05]

Hi @Piumal1999 @jayasanka-sack, please review this PR.
Thanks!

[jayasanka-sack]

Hey, I'm so sorry I missed this. Let me have a look.

[jayasanka-sack]

Hey Daud, kudos on getting the steps implemented! I've got a suggestion to make things even more organized:

create a new spec with the following:

Given I have a patient list with patients <- Use before each hook, create a list with few patients using the API

# test 1

When I navigate to the patient list
And I click on the patient {{name}} in the list
Then I should be redirected to the patient's chart page
When I click the close button
Then I should be redirected back to the patient list

...

# test 3

When I navigate to the patient list
And I click on the patient {{name}} in the list
Then I should be redirected to the patient's chart page
When I click on the visits tab
And I reload the page
And I click the close button
Then I should be redirected back to the patient list

....

// Add other tests except test 4

....

// Clean all the data at the end

@kdaud @brandones let me know what you think

[kdaud]

@jayasanka-sack I would suggest to break the test into the suggested cases without creating a new spec file since they're concerning with patient list. I'm open to suggestion.

[jayasanka-sack]

The reason for new spec is that it calls for different pre-populated data, meaning we'll need different 'Given' statements. That also means we'll have to set up new 'beforeEach' and 'afterEach' hooks.

[kdaud]

@jayasanka-sack I've addressed the suggestion. Could you review the changes? thanks.

[jayasanka-sack]

Thank you so much, Daud! I added few minor comments.

[jayasanka-sack]

Thanks a bunch, @kdaud !