Skip to content

Conversation

@emosbaugh
Copy link
Member

What this PR does / why we need it:

the test is failing because there are two loaders on the page

Error: expect.toBeVisible: Error: strict mode violation: locator('.Loader') resolved to 2 elements:
    1) <div class="Loader u-marginLeft--10 u-display--inlineBlock">…</div> aka locator('div').filter({ hasText: /^Updating$/ }).locator('div')
    2) <div class="Loader u-display--inlineBlock">…</div> aka locator('.flex-column > .Loader')
Call log:
  - Expect "toBeVisible" with timeout 5000ms
waiting for locator('.Loader')
Screenshot 2025-08-27 at 1 50 36 PM

https://github.com/replicatedhq/embedded-cluster/actions/runs/17257008089/job/49028266947#step:4:307

Which issue(s) this PR fixes:

Does this PR require a test?

Does this PR require a release note?


Does this PR require documentation?

@github-actions
Copy link

This PR has been released (on staging) and is available for download with a embedded-cluster-smoke-test-staging-app license ID.

Online Installer:

curl "https://staging.replicated.app/embedded/embedded-cluster-smoke-test-staging-app/ci/appver-dev-3377c9e" -H "Authorization: $EC_SMOKE_TEST_LICENSE_ID" -o embedded-cluster-smoke-test-staging-app-ci.tgz

Airgap Installer (may take a few minutes before the airgap bundle is built):

curl "https://staging.replicated.app/embedded/embedded-cluster-smoke-test-staging-app/ci-airgap/appver-dev-3377c9e?airgap=true" -H "Authorization: $EC_SMOKE_TEST_LICENSE_ID" -o embedded-cluster-smoke-test-staging-app-ci.tgz

Happy debugging!

@emosbaugh emosbaugh marked this pull request as ready for review August 27, 2025 22:26
@emosbaugh emosbaugh requested a review from sgalsaleh August 27, 2025 22:26
@emosbaugh emosbaugh enabled auto-merge (squash) August 27, 2025 22:36
@emosbaugh emosbaugh merged commit 19fa944 into main Aug 27, 2025
74 checks passed
@emosbaugh emosbaugh deleted the emosbaugh/20250827/test-flake branch August 27, 2025 22:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants