RFB can not handle two consecutives alerts #3785

wojtekwp · 2024-09-12T09:41:10Z

Describe the bug
RFB not able to handle two consecutives alerts. It fails with below message:

TimeoutError: page.waitForEvent: Timeout 5000ms exceeded while waiting for event “dialog”
=========================== logs ===========================
waiting for event “dialog”
Tip: Use “Set Browser Timeout” for increasing the timeout.

To Reproduce
I prep 2 simple files to easily reproduce an issue. Create new folder and copy 2 files content ex. to c:/alert_demo, then run start CMDs

cd c:/alert_demo
robot -d example/1 -t Alert-Debug example.robot

1 file (index.html) - example page with 2 consecutive alerts (one alert after another)

<html lang="pl">
<head>
    <meta charset="UTF-8">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <title>Multiple alerts issue - example</title>
</head>
<body>
    <button onclick="showAlerts()">Click to show alerts</button>

    <script>
        function showAlerts() {
            confirm("First alert accepted?");
            alert("Second alert!");
        }
    </script>
</body>
</html>

2 file (example.robot) - steps followed to reproduce an issue.
Please notice on line 8 it needs to be correct index.html path to open sample page with 2 alerts :

*** Settings ***
Library    Browser

*** Test Cases ***
Alert-Debug
    Browser Setup
    #provide correct index.html path below
    Go To    C:/alert_demo/index.html
    Click    //button
    Validate Alert Response    First alert accepted?
    Validate Alert Response    Second alert!

*** Keywords ***
Browser Setup
    Set Browser Timeout    30
    New Browser     browser=chromium     headless=false    args=["--start-maximized"]
    New Context    viewport=${None}
    New Page        url=about:blank

Validate Alert Response
    [Arguments]     ${messageValue}
    ${message} =    Wait For Alert    action=accept    text=${messageValue}    timeout=5

Expected behavior
Two alerts should be handled correctly

Desktop :
chromium
Win 11 E
Python 3.11.9
NodeJS v20.15.1
Browser library 18.6.3
Robot Framework version: 7.0.1
Playwright 1.45.0

Additional context
More details: https://forum.robotframework.org/t/can-not-handle-two-consecutives-alerts/3347

The text was updated successfully, but these errors were encountered:

aaltat · 2024-09-13T19:34:37Z

@allcontributors please add @wojtekwp for bugs.

allcontributors · 2024-09-13T19:34:47Z

@aaltat

I've put up a pull request to add @wojtekwp! 🎉

aaltat · 2024-09-28T21:53:35Z

Playwright does not allow to register two dialog handlers and therefore it crashes. I think that is by design. But I can also see a problem, because if you do only this:

    ${promise} =    Promise To    Wait For Alert    action=accept    timeout=5
    Click    id=alert
    ${text} =    Wait For      ${promise}
    Log    ${text}

You get only text from first alter and not from the second on. The second on is automatically dismissed by Playwright.

The problem can be solved by changing the code to of the page to call multiple times waitForEvent, but problem is that current keyword does not allow one to define different actions or expect different text for the alert. Like your example, you are excepting alerts to have different texts.

To solve problem correctly, we would need new keyword which would allow handling one or more alerts. Also that keyword should allow handling each alert differently, meaning have different values for action, prompt_inputand text (I think same timeout can be used for all alerts.) But what would be good name and syntax for this new keyword?

One option could be to kwargs and suffix each alert handling with number.
Example this would handle two different alerts:

    ${promise}    Promise To    Wait For Alerts    action1=accecpt    text1=Text One Here    action2=dismiss    text2=Two is here
    Click    id=alert
    ${text} =            Wait For      ${promise}    # ${text} would be list of two string

But there could better ways to define handling of multiple alerts and I am open to ideas? Would you idea for keyword syntax?

wojtekwp · 2024-09-29T22:36:32Z

I tried to add below snippet to Browser\keywords\interaction.py and prep next rfb example code with declared list variable & promise but it not always works as expected - don't know why but sometimes still failing to catch and retrieve response from dialog within expected 5s timeout.

If this "Wait For Alerts" keyword fails please try several times - it's not stable - works as expected usually 4/10 tries, sometimes more.

add below to interaction.py

@keyword(tags=("Wait", "PageContent"))
def wait_for_alerts(
    self,
    actions: List[DialogAction],
    prompt_inputs: List[str] = None,
    texts: List[Optional[str]] = None,
    timeout: Optional[timedelta] = None,
):
    if prompt_inputs is None:
        prompt_inputs = [""] * len(actions)
    if texts is None:
        texts = [None] * len(actions)

    responses = []
    for i, action in enumerate(actions):
        with self.playwright.grpc_channel() as stub:
            response = stub.WaitForAlert(
                Request().AlertAction(
                    alertAction=action.name,
                    promptInput=prompt_inputs[i],
                    timeout=self.get_timeout(timeout),
                )
            )
        logger.debug(response.log)
        if texts[i] is not None:
            assert (
                texts[i] == response.body
            ), f'Alert text was: "{response.body}" but it should have been: "{texts[i]}"'
        else:
            logger.debug("Not verifying alert text.")
        responses.append(response.body)
    return responses

example.py

*** Settings ***
Library    Browser

*** Variables ***
@{ACTIONS}    accept    dismiss
@{TEXTS}      First alert accepted?    Second alert!
#@{TEXTS}      First alert acceptedd?    Second alerttt!

*** Test Cases ***
Alert-Debug
    Browser Setup
    #provide correct index.html path below
    Go To    C:/alert_demo/index.html
    Click    //button
    Validate Alert Response    First alert accepted?
    Validate Alert Response    Second alert!

Alert-Debug2
    Browser Setup
    #provide correct index.html path below
    Go To    C:/alert_demo/index.html
    ${promise} =    Promise To    Wait For Alerts    actions=${ACTIONS}    texts=${TEXTS}    timeout=5	
    Click    //button
    ${texts} =    Wait For    ${promise}
    Log    ${texts}
	
*** Keywords ***
Browser Setup
    Set Browser Timeout    30
    New Browser     browser=chromium     headless=false    args=["--start-maximized"]
    New Context    viewport=${None}
    New Page        url=about:blank

Validate Alert Response
    [Arguments]     ${messageValue}
    ${message} =    Wait For Alert    action=accept    text=${messageValue}    timeout=5

execute from cmd with

cd c:/alert_demo
robot -d example/2 -t Alert-Debug2 example.robot

aaltat · 2024-09-30T08:18:37Z

There is also changes needed in the node side. In any case, your way to declare keyword syntax is valid one, because then automatic argument conversion should work. The downside might be that if there are many alerts, seeing what arguments belongs to which alert may come hard. But I think that is pretty rare case, because handling many alerts is not that common. I think if better ideas don’t arrive, I will go with this approach.

Fixes MarketSquare#3785

aaltat added bug Something isn't working priority: medium labels Sep 13, 2024

allcontributors bot mentioned this issue Sep 13, 2024

docs: add wojtekwp as a contributor for bug #3786

Merged

henriquediassep mentioned this issue Sep 23, 2024

RFB closing socket and ending playwright execution when handling multiple dialogs #3801

Closed

aaltat added enhancement New feature or request and removed bug Something isn't working labels Oct 6, 2024

aaltat added a commit to aaltat/robotframework-playwright that referenced this issue Oct 6, 2024

Handle multiple alerts with one promise

fea52cc

Fixes MarketSquare#3785

aaltat mentioned this issue Oct 6, 2024

Handle multiple alerts with one promise #3823

Merged

aaltat added a commit to aaltat/robotframework-playwright that referenced this issue Oct 6, 2024

Handle multiple alerts with one promise

bb75d0e

Fixes MarketSquare#3785

aaltat closed this as completed in #3823 Oct 6, 2024

aaltat closed this as completed in c6b5a33 Oct 6, 2024

aaltat added this to the v18.9.0 milestone Oct 6, 2024

aaltat mentioned this issue Oct 21, 2024

"Steam removed" issue happens once "Handle Future Dialogs" is used more than once #3367

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

RFB can not handle two consecutives alerts #3785

RFB can not handle two consecutives alerts #3785

wojtekwp commented Sep 12, 2024

aaltat commented Sep 13, 2024

allcontributors bot commented Sep 13, 2024

aaltat commented Sep 28, 2024

wojtekwp commented Sep 29, 2024

aaltat commented Sep 30, 2024

RFB can not handle two consecutives alerts #3785

RFB can not handle two consecutives alerts #3785

Comments

wojtekwp commented Sep 12, 2024

aaltat commented Sep 13, 2024

allcontributors bot commented Sep 13, 2024

aaltat commented Sep 28, 2024

wojtekwp commented Sep 29, 2024

aaltat commented Sep 30, 2024