Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Use Playwright expect() for code block tab stop tests #2160

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

gabalafou
Copy link
Collaborator

@gabalafou gabalafou commented Mar 10, 2025

Maybe fixes #2158.

This takes advantage of Playwright's ability to wait for a condition to be true rather than hard-coding manual wait times with the page.wait_for_timeout() function.

From the Playwright docs:

Playwright includes assertions that will wait until the expected condition is met. Using these assertions allows making the tests non-flaky and resilient.
-Writing Tests: Assertions

This takes advantage of Playwright's own ability to wait for the
condition to be true rather than calling wait_for_timeout manually
@gabalafou gabalafou changed the title For the code block tab stop tests use Playwright expect() assertion Use Playwright expect() assertion for the code tab stop tests Mar 10, 2025
@gabalafou gabalafou changed the title Use Playwright expect() assertion for the code tab stop tests Use Playwright expect() for the code tab stop tests Mar 10, 2025
@gabalafou gabalafou changed the title Use Playwright expect() for the code tab stop tests Use Playwright expect() for code block tab stop tests Mar 10, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

flaky accessibility tests: tab traps (notebooks)
1 participant