Skip to content
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

When a keyword fails and presenter_mode is enabled the browser is not closed #3644

Open
mardukbp opened this issue Jun 10, 2024 · 2 comments
Labels
bug Something isn't working priority: critical

Comments

@mardukbp
Copy link

Describe the bug

When a keyword fails and presenter_mode is enabled the browser is not closed when the test finishes. The reason is that the keyword Record Selector is called when a keyword fails and presenter_mode is enabled.

To Reproduce

*** Settings ***
Library    Browser    enable_presenter_mode=True

*** Test Cases ***
Test
    New Browser    chromium    headless=False
    New Page       https://www.google.com
    Click          "Yahoo"

Expected behavior
Browser library should honor the setting auto_closing_level and presenter_mode should behave according to the documentation: the tests run slower and each element acted upon is highlighted.

Desktop (please complete the following information):

  • OS: Windows 10
  • Browser: Chromium installed by rfbrowser init chromium
  • Version: 18.5.1

Additional context
If a user wants to execute Record Selector when a keyword fails, there are several options (which could be mentioned in the documentation): write a Listener, connect to a running Chromium-based browser that was started with --remote-debugging-port, use try-except. This error-handling should not be forced upon all users. presenter_mode is only meant to help the users see what Playwright does.

@aaltat aaltat added bug Something isn't working priority: critical labels Jun 10, 2024
@aaltat
Copy link
Member

aaltat commented Jun 10, 2024

Interesting, it should definitely close.

@allcontributors please add @mardukbp for bugs.

Copy link
Contributor

@aaltat

@mardukbp already contributed before to bug

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working priority: critical
Projects
None yet
Development

No branches or pull requests

2 participants