Skip to content
This repository has been archived by the owner on Feb 3, 2023. It is now read-only.

Handle popups/alerts #20

Open
vaidasmaciulis opened this issue Feb 24, 2020 · 2 comments · May be fixed by #117
Open

Handle popups/alerts #20

vaidasmaciulis opened this issue Feb 24, 2020 · 2 comments · May be fixed by #117
Assignees
@JelenaRy
Labels
good first issue Good for newcomers

Comments

@vaidasmaciulis
Copy link
Collaborator

vaidasmaciulis commented Feb 24, 2020 •
edited
Loading

created by Vaidas.Maciulis
@vaidasmaciulis
Copy link
Collaborator Author

For alerts we already have logic used, need to extract to helpers:

    it("should block any request during action", async () => {
        //Arrange
        await helpers.goToUrlAndLoad(DemoOpenCartSite);
        await page.on("dialog", dialog => {
            console.log(`Alert was detected: '${dialog.message()}'`);
            dialog.dismiss();
        });

@vaidasmaciulis vaidasmaciulis added the good first issue Good for newcomers label Sep 25, 2020
@JelenaRy JelenaRy assigned JelenaRy and unassigned dariusnemeiksis Apr 27, 2021
@JelenaRy JelenaRy linked a pull request May 28, 2021 that will close this issue
20 add popups handling #117
Open
@JelenaRy
Copy link
Collaborator

Tests are still passing after changes:
image

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@JelenaRy JelenaRy

Labels
good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

20 add popups handling devbridge/test-juggler
3 participants
@vaidasmaciulis @dariusnemeiksis @JelenaRy