Skip to content

Playwright_RGP_Add family member without copying proband info #2180

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

Merged
merged 4 commits into from
Aug 14, 2023

Conversation

Rinchan20
Copy link
Contributor

add family member without copying proband info

add family member without copying proband info
marked as skip until non-prod RGP stuff is fixed, can be ran and should pass so long as there is a proband tab but if it runs into the same pt i.e. no new pts created before next run,  it might fail since relationshipIDs need to be unique
@Rinchan20 Rinchan20 marked this pull request as ready for review August 10, 2023 23:42
@Rinchan20 Rinchan20 assigned aweng98 and unassigned aweng98 Aug 10, 2023
@@ -70,8 +71,8 @@ export default class RgpParticipantPage extends ParticipantPage {
const selectedRelation = await this.relation.toLocator().locator('.mat-select-value-text').innerText();
await this.submit.click();

await expect(page.locator(modal)).toHaveText('Successfully added family member');
await page.locator('h1').click(); // close popup with a click outside
await expect(page.locator(familyMemberAddedSuccessfullyModal)).toHaveText('Successfully added family member');
Copy link
Contributor Author

Choose a reason for hiding this comment

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

made change since once it turns into the 'success' modal, it seems that for a split second there are 2 webelements that fit role=dialog

@Rinchan20 Rinchan20 merged commit 7358051 into develop Aug 14, 2023
@Rinchan20 Rinchan20 deleted the create-family-member-without-copying-proband-info branch August 14, 2023 14:55
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