Skip to content

E2E test: reload page to check for license

Jay McCure requested to merge jmc-retry-license-spec-e2e into master

What does this MR do and why?

Resolves https://gitlab.com/gitlab-org/gitlab/-/issues/451632 by making this test more robust.

This test was failing intermittently in slower jobs due to the license not rendering immediately. The reload would not occur since the element was not present on the page causing an exception.

Now the wait_until will retry on exception.

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Before After
https://gitlab-qa-allure-reports.s3.amazonaws.com/e2e-package-and-test/master/1225675724/index.html#suites/d225c9add4e032b7c04e2143e7d96217 https://gitlab-qa-allure-reports.s3.amazonaws.com/e2e-package-and-test/jmc-retry-license-spec-e2e/1225798996/index.html#suites/ee96d6389fc638194cdc4453051f05b6/2a3022c7c9f0deec/

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

Edited by Jay McCure

Merge request reports

Loading