Skip to content

The two consent-modal event tests have never passed: the wizard's Next click is intercepted by .blast-form #1714

Description

@rubenvdlinde

What

tests/e2e/event-wiring.spec.ts:396 and :419 have failed on every run since they were added, and both fail the same way: a click on the wizard's Next button is intercepted, and openConsentModal() retries it until the 120s test timeout.

Error: locator.click: Test timeout of 120000ms exceeded.
Call log:
  - waiting for locator('button').filter({ hasText: /^Next$/ }).first()
    - locator resolved to <button type="button" class="button-vue button-vue--primary">…
  - attempting click action
    - element is visible, enabled and stable
    - performing click action
    - <div schema="blast" class="blast-form" register="pipelinq">…</div> intercepts pointer events
  - retrying click action

Provenance

Both tests, and the openConsentModal() helper they share, were added by bb8788a (#1693, "repair the consent notification, and cover three more event chains"). The run immediately before that commit shows neither test in its failure list, because neither existed yet. They have never been green.

Why it reads as a timeout and is not one

The button is visible, enabled and stable, so Playwright does not fail on the overlay. It keeps retrying the click, and the only thing that ever ends it is the test budget. That is why the error names a timeout and says nothing about the interception unless you read the call log.

The interceptor is div.blast-form itself, so this is a layering or step-transition problem in the wizard rather than a slow page. Raising the timeout would not help.

Adjacent, and probably related

openConsentModal() advances the wizard with an unbounded loop:

while (await next.isVisible().catch(() => false)) {
  await next.click()
}

If a click stops advancing the wizard, that loop cannot terminate on its own. Even once the interception is fixed, this is worth bounding with a step-count assertion so the failure is fast and names the step it stuck on.

Not the routing change

Ruled out: #1684 (createWebHistory) was already merged in the run where these tests did not appear, and the other five failures in the same run were all resolved by #1706, #1711 and #1713 without touching this one.

Activity

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

Metadata

Metadata

Assignees

Labels

triageAwaiting triage

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions