Skip to content

integration tests: switch poll_for_navigation to Playwright wait_for_url #6384

@BABTUNA

Description

@BABTUNA

Summary\nAfter #6378 migrated integration tests to Playwright, ests/integration/utils.py::poll_for_navigation still waits for URL changes via AppHarness.expect(...) (sleep-poll loop).\n\n## Problem\nThe custom polling loop is less reliable than Playwright's native navigation waiting and can miss/flake around redirect timing.\n\n## Proposed fix\nUpdate poll_for_navigation to use Playwright-native waiting:\n- capture prev_url before yielding\n- after the action, call page.wait_for_url(lambda url: url != prev_url, timeout=timeout * 1000)\n\nThis keeps current behavior (wait until URL changes) while using Playwright's event-driven waiter.\n\n## Acceptance criteria\n- poll_for_navigation no longer calls AppHarness.expect\n- it calls page.wait_for_url with an URL-change predicate\n- regression test covers the helper behavior\n

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions