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
Ratest 302-Add an identifier for a patient #492
base: master
Are you sure you want to change the base?
Conversation
…dministrationPage changed to verifyAdministrationPageIsLoaded
import org.openmrs.contrib.qaframework.helper.Page; | ||
import org.openqa.selenium.By; | ||
|
||
public class PatientFormPage extends Page { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this a patient registration form, a clinicianFacing patient form, what is it this form ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This page has been deleted
import org.openqa.selenium.By; | ||
import org.openqa.selenium.Keys; | ||
|
||
public class PatientPage extends Page{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is patientManagementPage
...dd-tests/src/test/java/org/openmrs/contrib/qaframework/automation/AddNewIdentifierSteps.java
Show resolved
Hide resolved
...dd-tests/src/test/java/org/openmrs/contrib/qaframework/automation/AddNewIdentifierSteps.java
Show resolved
Hide resolved
qaframework-bdd-tests/src/test/resources/org/openmrs/uitestframework/test.properties
Outdated
Show resolved
Hide resolved
…ent step in addNewIdentifier feature file and the step definition in AddNewIdentifierSteps
@EdisonMeks could you add a ticket url to this PR? |
@kdaud I have added the ticket url. |
Could you add it under the Requirements section here? |
…for PREFERRED_BUTTON
…d and added patient to select
@@ -24,4 +24,9 @@ public void searchPatientIdentifierOrPatientName(String text) { | |||
waitForTextToBePresentInElement(SEARCH_STATUS,text.substring(0, text.length() - 1)); | |||
} | |||
|
|||
public void clickOnFirstPatient(String text) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This method can only work for reference application not legacy ui, so if you are working towards legacyui, please continue.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks did the needful
Could you check the failure please. |
Requirements
Summary
This is a test for adding a new identifier for a patient
Screenshots
None.
Related Issue
https://issues.openmrs.org/browse/RATEST-302
Other
None.