Skip to content

Conversation

@labkey-tchad
Copy link
Member

Rationale

Waiting for the selected admin tab to be visible should fix this intermittent test failure.

org.openqa.selenium.NoSuchElementException: Expected condition failed: waiting for link=Export Diagnostics (tried for 2 second(s) with 100 milliseconds interval)
  at app//org.labkey.test.Locator.waitForElement(Locator.java:561)
  at app//org.labkey.test.Locator.waitForElement(Locator.java:537)
  at app//org.labkey.test.Locator.waitForElement(Locator.java:530)
  at app//org.labkey.test.WebDriverWrapper.waitForElement_aroundBody10(WebDriverWrapper.java:2448)
  at app//org.labkey.test.WebDriverWrapper$AjcClosure11.run(WebDriverWrapper.java:1)
  at app//org.aspectj.runtime.reflect.JoinPointImpl.proceed(JoinPointImpl.java:164)
  at app//org.labkey.test.aspects.StaleElementRetryAspect.beforeLoggedMethod(StaleElementRetryAspect.java:40)
  at app//org.labkey.test.WebDriverWrapper.waitForElement(WebDriverWrapper.java:2446)
  at app//org.labkey.test.tests.cloudServices.ExportDiagnosticsTest.testPremiumLinksRegistered(ExportDiagnosticsTest.java:59)

Additionally, ShowAdminPage is unnecessarily verbose and repetitive. There is no need to cache any of the settings links, they are only ever clicked to navigate away from the page. We can also pull some shared functionality out of all the methods that click settings links.

Related Pull Requests

  • N/A

Changes

  • Wait for tabs to become visible in ShowAdminPage
  • Remove unnecessary items from element cache

@labkey-tchad labkey-tchad changed the title Simplify ShowAdminPage Wait for panel visibility in ShowAdminPage Aug 11, 2025
@labkey-tchad labkey-tchad merged commit ffd5c3d into release25.7-SNAPSHOT Aug 11, 2025
7 of 8 checks passed
@labkey-tchad labkey-tchad deleted the 25.7_fb_showAdminPageUpdate branch August 11, 2025 22:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants