Skip to content

Fix intermittent R report timeouts#3027

Merged
labkey-tchad merged 3 commits into
release26.3-SNAPSHOTfrom
26.3_fb_reportLoadTimeout
Jun 2, 2026
Merged

Fix intermittent R report timeouts#3027
labkey-tchad merged 3 commits into
release26.3-SNAPSHOTfrom
26.3_fb_reportLoadTimeout

Conversation

@labkey-tchad
Copy link
Copy Markdown
Member

Rationale

The report in AbstractKnitrReportTest.moduleReportDependencies takes 25-35 seconds to load. It usually passes on the Linux agents but exceeds the 30 second timeout pretty regularly on Windows agents.

org.openqa.selenium.TimeoutException: Expected condition failed: waiting for element to disappear: xpath=//div[contains(concat(' ',normalize-space(@class),' '), ' x4-mask ')][not(ancestor-or-self::*[contains(@style,'display: none') or contains(@style,'visibility: hidden') or contains(@class, 'x-hide-display') or contains(@class, 'x4-hide-offsets') or contains(@class, 'x-hide-offsets')] or (@type = 'hidden'))]
(tried for 30 seconds with 500 milliseconds interval)
  at app//org.openqa.selenium.support.ui.FluentWait.timeoutException(FluentWait.java:278)
  at app//org.openqa.selenium.support.ui.FluentWait.until(FluentWait.java:233)
  at app//org.labkey.test.Locator.waitForElementToDisappear(Locator.java:583)
  at app//org.labkey.test.Locator.waitForElementToDisappear(Locator.java:578)
  at app//org.labkey.test.WebDriverWrapper.waitForElementToDisappear_aroundBody24(WebDriverWrapper.java:2628)
  at app//org.labkey.test.WebDriverWrapper$AjcClosure25.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.waitForElementToDisappear(WebDriverWrapper.java:2626)
  at app//org.labkey.test.util.Ext4Helper.waitForMaskToDisappear(Ext4Helper.java:682)
  at app//org.labkey.test.tests.AbstractKnitrReportTest.moduleReportDependencies(AbstractKnitrReportTest.java:237)
  at app//org.labkey.test.tests.KnitrReportTest.testModuleReportDependencies(KnitrReportTest.java:98)

Other reports tend to time out on Windows also:

org.openqa.selenium.TimeoutException: Expected condition failed: waiting for visibility of element found by css=div.reportView
(tried for 10 seconds with 500 milliseconds interval)
  at app//org.openqa.selenium.support.ui.WebDriverWait.timeoutException(WebDriverWait.java:85)
  at app//org.openqa.selenium.support.ui.FluentWait.until(FluentWait.java:233)
  at app//org.labkey.test.pages.reports.ScriptReportPage._clickReportTab(ScriptReportPage.java:232)
  at app//org.labkey.test.pages.reports.ScriptReportPage.clickReportTab(ScriptReportPage.java:213)
  at app//org.labkey.test.util.RReportHelper.clickReportTab(RReportHelper.java:474)
  at app//org.labkey.test.util.RReportHelper.executeScript(RReportHelper.java:136)
  at app//org.labkey.test.util.RReportHelper.executeScript(RReportHelper.java:122)
  at app//org.labkey.test.tests.RlabkeyTest.doRLabkeyTest(RlabkeyTest.java:307)
  at app//org.labkey.test.tests.RlabkeyTest.testRlabkeyWebDavApi(RlabkeyTest.java:259)
  at java.base@25.0.1/jdk.internal.reflect.DirectMethodHandleAccessor.invoke(DirectMethodHandleAccessor.java:104)

Related Pull Requests

  • N/A

Changes

  • Increase timeout for R reports

@labkey-tchad labkey-tchad self-assigned this Jun 1, 2026
@labkey-tchad labkey-tchad merged commit 20053a7 into release26.3-SNAPSHOT Jun 2, 2026
7 checks passed
@labkey-tchad labkey-tchad deleted the 26.3_fb_reportLoadTimeout branch June 2, 2026 20:56
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