Refactor JBrowseTest to streamline variant handling

Removed redundant variant waits and adjusted click action.

Author: bbimber

jbrowse/test/src/org/labkey/test/tests/external/labModules/JBrowseTest.java

@@ -438,11 +438,6 @@ private void testSessionCardDisplay()
         beginAt("/" + getProjectName() + "/jbrowse-jbrowse.view?session=mgap&location=1:8328..8842");
         waitForJBrowseToLoad();
 
-        // This will wait for them, allowing the full browser to lod:
-        getVariantWithinTrack("mgap_hg38", "SNV C -> T");
-        getVariantWithinTrack("mgap_hg38", "SNV G -> C");
-
-
         Actions actions = new Actions(getDriver());
         WebElement toClick = getDriver().findElements(getVariantWithinTrack("mgap_hg38", "SNV A -> G")).stream().filter(WebElement::isDisplayed).collect(JBrowseTestHelper.toSingleton());
         actions.click(toClick).perform();
@@ -456,9 +451,12 @@ private void testTitleMapping()
         beginAt("/" + getProjectName() + "/jbrowse-jbrowse.view?session=mgap&location=1:104..275");
         waitForJBrowseToLoad();
 
+        // This will wait for them, allowing the full browser to lod:
+        getVariantWithinTrack("mgap_hg38", "SNV C -> T");
+        getVariantWithinTrack("mgap_hg38", "SNV G -> C");
+
         Actions actions = new Actions(getDriver());
         WebElement toClick = getDriver().findElements(getVariantWithinTrack("mgap_hg38", "SNV T -> C")).stream().filter(WebElement::isDisplayed).collect(JBrowseTestHelper.toSingleton()); // 1:137..137
-        WebDriverWrapper.sleep(100); // This allows a short window for rendering prior to clicking variants
         actions.click().perform();        
 
         waitForElement(Locator.tagWithText("div", "1:137"));