Wait for readyState when switching test windows

[labkey-tchad]

Rationale

A change to how we switch windows was accidentally included in a previous PR. It attempts to prevent errors with assertTextPresent throwing an NPE after switching windows by waiting for document.readyState to be "complete". This mostly works, except one place in WNPRC that opens a window in a unique method. In Firefox, the readyState of that page is the nonstandard "uninitialized".

org.junit.runners.model.TestTimedOutException: test timed out after 30 minutes
  at java.base@17.0.7/java.lang.Thread.sleep(Native Method)
  at app//org.openqa.selenium.support.ui.Sleeper.lambda$static$0(Sleeper.java:25)
  at app//org.openqa.selenium.support.ui.Sleeper$$Lambda$675/0x00000008016d5478.sleep(Unknown Source)
  at app//org.openqa.selenium.support.ui.FluentWait.until(FluentWait.java:232)
  at app//org.labkey.test.WebDriverWrapper.switchToWindow(WebDriverWrapper.java:1011)
  at app//org.labkey.test.tests.wnprc_ehr.WNPRC_EHRTest.testClinpathVirologyBulkUploadPreviewAndDate(WNPRC_EHRTest.java:3294)

Note: the test timeout was caused by using seconds instead of milliseconds for the readyState timeout. That has also been fixed.

Related Pull Requests

• Decode reportId to avoid double-encoding later #2182

Changes

• Handle "uninitialized" readyState when switching windows
• Use Duration.ofMillis instead of Duration.ofSeconds for timeout

[labkey-tchad]

Successful run: https://teamcity.labkey.org/buildConfiguration/LabKey_Trunk_External_Wnprc_WnprcEhrPostgres_2/3296854