JAL-2388 Removing sources of non-determinism in unit tests
[jalview.git] / test / jalview / viewmodel / OverviewDimensionsTest.java
index 5fc95b6..94fb83f 100644 (file)
@@ -26,16 +26,23 @@ import jalview.bin.Cache;
 import jalview.bin.Jalview;
 import jalview.datamodel.Alignment;
 import jalview.datamodel.Sequence;
+import jalview.datamodel.SequenceGroup;
 import jalview.datamodel.SequenceI;
 import jalview.gui.AlignFrame;
 import jalview.gui.AlignViewport;
+import jalview.gui.Desktop;
+import jalview.gui.JvOptionPane;
 import jalview.io.DataSourceType;
 import jalview.io.FileLoader;
 
-import org.testng.annotations.AfterClass;
+import java.util.List;
+
+import org.testng.annotations.AfterMethod;
+import org.testng.annotations.BeforeClass;
 import org.testng.annotations.BeforeMethod;
 import org.testng.annotations.Test;
 
+@Test(singleThreaded = true)
 public class OverviewDimensionsTest {
 
   boolean showConservationSetting;
@@ -60,82 +67,6 @@ public class OverviewDimensionsTest {
           "Seq5",
           "ABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBAC");
 
-  SequenceI seq6 = new Sequence(
-          "Seq6",
-          "ABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBAC");
-
-  SequenceI seq7 = new Sequence(
-          "Seq7",
-          "ABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBAC");
-
-  SequenceI seq8 = new Sequence(
-          "Seq8",
-          "ABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBAC");
-
-  SequenceI seq9 = new Sequence(
-          "Seq9",
-          "ABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBAC");
-
-  SequenceI seq10 = new Sequence(
-          "Seq10",
-          "ABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBAC");
-
-  SequenceI seq11 = new Sequence(
-          "Seq11",
-          "ABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBAC");
-
-  SequenceI seq12 = new Sequence(
-          "Seq12",
-          "ABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBAC");
-
-  SequenceI seq13 = new Sequence(
-          "Seq13",
-          "ABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBAC");
-
-  SequenceI seq14 = new Sequence(
-          "Seq14",
-          "ABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBAC");
-
-  SequenceI seq15 = new Sequence(
-          "Seq15",
-          "ABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBAC");
-
-  SequenceI seq16 = new Sequence(
-          "Seq16",
-          "ABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBAC");
-
-  SequenceI seq17 = new Sequence(
-          "Seq17",
-          "ABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBAC");
-
-  SequenceI seq18 = new Sequence(
-          "Seq18",
-          "ABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBAC");
-
-  SequenceI seq19 = new Sequence(
-          "Seq19",
-          "ABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBAC");
-
-  SequenceI seq20 = new Sequence(
-          "Seq20",
-          "ABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBAC");
-
-  SequenceI seq21 = new Sequence(
-          "Seq21",
-          "ABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBAC");
-
-  SequenceI seq22 = new Sequence(
-          "Seq22",
-          "ABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBAC"
-                  + "ABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBAC"
-                  + "ABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBAC"
-                  + "ABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBAC"
-                  + "ABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBAC");
-
-  SequenceI seq23 = new Sequence(
-          "Seq23",
-          "ABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBACABCABCABCABCABCABCABCABCBACBACBACBAC");
-
   AlignFrame af;
   AlignViewport av;
   OverviewDimensions od;
@@ -145,6 +76,15 @@ public class OverviewDimensionsTest {
   int boxWidth;
   int boxHeight;
 
+  int viewHeight;
+
+  @BeforeClass(alwaysRun = true)
+  public void setUpJvOptionPane()
+  {
+    JvOptionPane.setInteractiveMode(false);
+    JvOptionPane.setMockResponse(JvOptionPane.CANCEL_OPTION);
+  }
+
   @BeforeMethod(alwaysRun = true)
   public void setUp()
   {
@@ -158,6 +98,22 @@ public class OverviewDimensionsTest {
             DataSourceType.FILE);
 
     /*
+     * Wait for viewport to initialise
+     */
+    synchronized (this)
+    {
+      while (af.getViewport().getEndRes() == 0)
+      {
+        try
+        {
+          wait(50);
+        } catch (InterruptedException e)
+        {
+        }
+      }
+    }
+
+    /*
      * wait for Consensus thread to complete
      */
     synchronized (this)
@@ -172,31 +128,78 @@ public class OverviewDimensionsTest {
         }
       }
     }
+
     // get cached setting for showConservation
     // reset it in AfterClass!
     showConservationSetting = Cache.getDefault("SHOW_CONSERVATION", true);
     
     av = af.getViewport();
+
+    if (av.isCalcInProgress())
+    {
+
+    }
+
     od = new OverviewDimensions(av);
 
+    while (av.isCalcInProgress())
+    {
+      try
+      {
+        Thread.sleep(50);
+      } catch (InterruptedException e)
+      {
+        System.out.println("Hiding cols interruption");
+      }
+    }
+
     // Initial box sizing - default path through code
     od.setBoxPosition();
+
     scalew = (float) od.getWidth()
             / (av.getAlignment().getWidth() * av.getCharWidth());
     scaleh = (float) od.getSequencesHeight()
             / (av.getAlignment().getHeight() * av.getCharHeight());
-    boxWidth = (int) ((av.getEndRes() - av.getStartRes() + 1)
+    boxWidth = Math.round((av.getEndRes() - av.getStartRes() + 1)
             * av.getCharWidth() * scalew);
-    boxHeight = (int) ((av.getEndSeq() - av.getStartSeq())
+    boxHeight = Math.round((av.getEndSeq() - av.getStartSeq())
             * av.getCharHeight() * scaleh);
+
+    viewHeight = av.getEndSeq() - av.getStartSeq();
+
+    init();
   }
 
+  private void init()
+  {
+    av.showAllHiddenColumns();
+    av.showAllHiddenSeqs();
+    av.setSelectionGroup(null);
+    // o/w hidden seqs retain selection group, causes problems later when hiding
+    // sequences
+
+    // wait for conservation calc to complete
+    while (av.isCalcInProgress())
+    {
+      try
+      {
+        Thread.sleep(50);
+      } catch (InterruptedException e)
+      {
+
+      }
+    }
 
-  @AfterClass(alwaysRun = true)
+    mouseClick(od, 0, 0);
+    moveViewport(0, 0);
+  }
+
+  @AfterMethod(alwaysRun = true)
   public void tearDown()
   {
     Cache.applicationProperties.setProperty("SHOW_CONSERVATION",
             Boolean.toString(showConservationSetting));
+    Desktop.instance.closeAll_actionPerformed(null);
   }
 
   /**
@@ -302,7 +305,7 @@ public class OverviewDimensionsTest {
   @Test(groups = { "Functional" })
   public void testSetBoxFromMouseClick()
   {
-    od.checkValid();
+    od.setBoxPositionByMouse(0, 0);
     assertEquals(od.getBoxX(), 0);
     assertEquals(od.getBoxY(), 0);
     assertEquals(od.getBoxWidth(), boxWidth);
@@ -315,7 +318,7 @@ public class OverviewDimensionsTest {
     assertEquals(od.getBoxWidth(), boxWidth);
     assertEquals(od.getBoxHeight(), boxHeight);
     assertEquals(od.getScrollRow(),
-            (int) (10 / scaleh / av.getCharHeight()));
+            Math.round(10 / scaleh / av.getCharHeight()));
     assertEquals(od.getScrollCol(), 0);
 
     // negative boxY value reset to 0
@@ -323,7 +326,8 @@ public class OverviewDimensionsTest {
     assertEquals(od.getBoxY(), 0);
     assertEquals(od.getBoxWidth(), boxWidth);
     assertEquals(od.getBoxHeight(), boxHeight);
-    assertEquals(od.getScrollCol(), (int) (6 / scalew / av.getCharWidth()));
+    assertEquals(od.getScrollCol(),
+            Math.round(6 / scalew / av.getCharWidth()));
     assertEquals(od.getScrollRow(), 0);
 
     // overly large boxX value reset to width-boxWidth
@@ -360,7 +364,7 @@ public class OverviewDimensionsTest {
             (od.getBoxY() / scaleh / av.getCharHeight()), 1.5);
 
     // move viewport so startRes non-zero and then mouseclick
-    moveViewportH(av, od, 50);
+    moveViewportH(50);
 
     // click at viewport position
     int oldboxx = od.getBoxX();
@@ -392,7 +396,7 @@ public class OverviewDimensionsTest {
   @Test(groups = { "Functional" })
   public void testFromMouseWithHiddenColsAtStart()
   {
-    od.checkValid();
+    od.setBoxPositionByMouse(0, 0);
     assertEquals(od.getBoxX(), 0);
     assertEquals(od.getBoxY(), 0);
     assertEquals(od.getBoxWidth(), boxWidth);
@@ -402,10 +406,11 @@ public class OverviewDimensionsTest {
     // hide cols at start and check updated box position is correct
     // changes boxX but not boxwidth
     int lastHiddenCol = 30;
-    av.hideColumns(0, lastHiddenCol);
+    hideColumns(0, lastHiddenCol);
+
     od.setBoxPosition();
     assertEquals(od.getBoxX(),
-            (int) ((lastHiddenCol + 1) * scalew * av.getCharWidth()));
+            Math.round((lastHiddenCol + 1) * scalew * av.getCharWidth()));
     assertEquals(od.getBoxWidth(), boxWidth);
     assertEquals(od.getBoxHeight(), boxHeight);
 
@@ -414,20 +419,21 @@ public class OverviewDimensionsTest {
     /*    int xpos = 10;
         mouseClick(od, xpos, 0);
         assertEquals(od.getBoxX(),
-                (int) ((lastHiddenCol + 1) * scalew * av.getCharWidth()));
+                Math.round ((lastHiddenCol + 1) * scalew * av.getCharWidth()));
         assertEquals(od.getBoxY(), 0);
         assertEquals(od.getBoxWidth(), boxWidth);
         assertEquals(od.getBoxHeight(), boxHeight);
         assertEquals(od.getScrollRow(), 0);
         assertEquals(od.getScrollCol(),
-                (int) (xpos / scalew / av.getCharWidth()));
+                Math.round (xpos / scalew / av.getCharWidth()));
     */
     // click to right of hidden columns, box moves to click point
     testBoxIsAtClickPoint(40, 0);
     assertEquals(od.getScrollRow(), 0);
     assertEquals(od.getScrollCol(),
- (int) (40 / scalew / av.getCharWidth())
-            - lastHiddenCol, 1.5);
+            Math.round(40 / scalew / av.getCharWidth())
+ - lastHiddenCol,
+            1.5);
 
     // click to right of hidden columns such that box runs over right hand side
     // of alignment
@@ -436,14 +442,14 @@ public class OverviewDimensionsTest {
     int xpos = 100;
     mouseClick(od, xpos, 5);
     assertEquals(od.getBoxX(), od.getWidth() - od.getBoxWidth(), 1.5);
-    assertEquals(od.getBoxY(), 5, 1.5);
+    assertEquals(od.getBoxY(), 5);
     assertEquals(od.getBoxWidth(), boxWidth);
     assertEquals(od.getBoxHeight(), boxHeight);
     assertEquals(od.getScrollCol(),
-            (int) (od.getBoxX() / scalew / av.getCharWidth())
+            Math.round(od.getBoxX() / scalew / av.getCharWidth())
                     - lastHiddenCol, 1.5);
     assertEquals(od.getScrollRow(),
-            (int) (od.getBoxY() / scaleh / av.getCharHeight()), 1.5);
+            Math.round(od.getBoxY() / scaleh / av.getCharHeight()));
 
   }
 
@@ -454,7 +460,7 @@ public class OverviewDimensionsTest {
   @Test(groups = { "Functional" })
   public void testFromMouseWithHiddenColsInMiddle()
   {
-    od.checkValid();
+    od.setBoxPositionByMouse(0, 0);
     assertEquals(od.getBoxX(), 0);
     assertEquals(od.getBoxY(), 0);
     assertEquals(od.getBoxWidth(), boxWidth);
@@ -464,7 +470,8 @@ public class OverviewDimensionsTest {
     // hide columns 60-68, no change to box position or dimensions
     int firstHidden = 60;
     int lastHidden = 68;
-    av.hideColumns(firstHidden, lastHidden);
+    hideColumns(firstHidden, lastHidden);
+
     od.setBoxPosition();
     assertEquals(od.getBoxX(), 0);
     assertEquals(od.getBoxY(), 0);
@@ -477,27 +484,30 @@ public class OverviewDimensionsTest {
     int xpos = 50 - boxWidth; // 50 is position in overview halfway between cols
                               // 60 and 70
     mouseClick(od, xpos, 0);
-    assertEquals(od.getBoxX(), xpos, 1.5);
+    assertEquals(od.getBoxX(), xpos);
     assertEquals(od.getBoxY(), 0);
-    assertEquals(od.getBoxWidth(), boxWidth
-            + (lastHidden - firstHidden + 1) * scalew * av.getCharWidth(),
-            1.5);
+    assertEquals(
+            od.getBoxWidth(),
+            Math.round(boxWidth + (lastHidden - firstHidden + 1) * scalew
+                    * av.getCharWidth()));
     assertEquals(od.getBoxHeight(), boxHeight);
-    assertEquals(od.getScrollCol(), (int) (xpos / scalew / av.getCharWidth()), 1.5);
+    assertEquals(od.getScrollCol(),
+            Math.round(xpos / scalew / av.getCharWidth()));
     assertEquals(od.getScrollRow(), 0);
 
     // move box so that it completely covers hidden cols
     // box width changes, boxX and scrollCol as for hidden case
     xpos = 30;
     mouseClick(od, xpos, 0);
-    assertEquals(od.getBoxX(), xpos, 1.5);
+    assertEquals(od.getBoxX(), xpos);
     assertEquals(od.getBoxY(), 0);
-    assertEquals(od.getBoxWidth(), boxWidth
-            + (lastHidden - firstHidden + 1) * scalew * av.getCharWidth(),
-            1.5);
+    assertEquals(
+            od.getBoxWidth(),
+            Math.round(boxWidth + (lastHidden - firstHidden + 1) * scalew
+                    * av.getCharWidth()));
     assertEquals(od.getBoxHeight(), boxHeight);
     assertEquals(od.getScrollCol(),
-            (int) (xpos / scalew / av.getCharWidth()), 1.5);
+            Math.round(xpos / scalew / av.getCharWidth()), 1.5);
     assertEquals(od.getScrollRow(), 0);
 
     // move box so boxX is in hidden cols, box overhangs at right
@@ -512,7 +522,7 @@ public class OverviewDimensionsTest {
         assertEquals(od.getBoxWidth(), boxWidth, 1.5);
         assertEquals(od.getBoxHeight(), boxHeight);
         assertEquals(od.getScrollCol(),
-                (int) (xpos / scalew / av.getCharWidth()), 1.5);
+                Math.round(xpos / scalew / av.getCharWidth()), 1.5);
         assertEquals(od.getScrollRow(), 0);*/
 
     // move box so boxX is to right of hidden cols, but does not go beyond full
@@ -528,7 +538,7 @@ public class OverviewDimensionsTest {
        testBoxIsAtClickPoint(xpos, 0);
        assertEquals(od.getScrollRow(), 0);
        assertEquals(od.getScrollCol(),
-               (int) (xpos / scalew / av.getCharWidth())
+               Math.round(xpos / scalew / av.getCharWidth())
                - lastHidden, 1.5);*/
     
     // move box so it goes beyond full width of alignment
@@ -536,15 +546,14 @@ public class OverviewDimensionsTest {
     xpos = 3000;
     mouseClick(od, xpos, 5);
     assertEquals(od.getBoxX(), od.getWidth() - od.getBoxWidth(), 1.5);
-    assertEquals(od.getBoxY(), 5, 1.5);
+    assertEquals(od.getBoxY(), 5);
     assertEquals(od.getBoxWidth(), boxWidth);
     assertEquals(od.getBoxHeight(), boxHeight);
     assertEquals(od.getScrollCol(),
-            (od.getBoxX() / scalew / av.getCharWidth())
-                    - (lastHidden - firstHidden + 1),
-            1.5);
+            Math.round((od.getBoxX() / scalew / av.getCharWidth())
+                    - (lastHidden - firstHidden + 1)));
     assertEquals(od.getScrollRow(),
-            (int) (od.getBoxY() / scaleh / av.getCharHeight()), 1.5);
+            Math.round(od.getBoxY() / scaleh / av.getCharHeight()));
 
   }
 
@@ -555,7 +564,7 @@ public class OverviewDimensionsTest {
   @Test(groups = { "Functional" })
   public void testFromMouseWithHiddenColsAtEnd()
   {
-    od.checkValid();
+    od.setBoxPositionByMouse(0, 0);
     assertEquals(od.getBoxX(), 0);
     assertEquals(od.getBoxY(), 0);
     assertEquals(od.getBoxWidth(), boxWidth);
@@ -565,7 +574,7 @@ public class OverviewDimensionsTest {
     // hide columns 140-157, no change to box position or dimensions
     int firstHidden = 140;
     int lastHidden = 157;
-    av.hideColumns(firstHidden, lastHidden);
+    hideColumns(firstHidden, lastHidden);
     od.setBoxPosition();
     assertEquals(od.getBoxX(), 0);
     assertEquals(od.getBoxY(), 0);
@@ -579,7 +588,7 @@ public class OverviewDimensionsTest {
     testBoxIsAtClickPoint(xpos, 0);
     assertEquals(od.getScrollRow(), 0);
     assertEquals(od.getScrollCol(),
-            (int) (xpos / scalew / av.getCharWidth()), 1.5);
+            Math.round(xpos / scalew / av.getCharWidth()), 1.5);
 
     // click to left of hidden cols, with overlap
     // boxX and scrollCol adjusted for hidden cols, width normal
@@ -589,13 +598,13 @@ public class OverviewDimensionsTest {
     mouseClick(od, xpos, 0);
     assertEquals(
             od.getBoxX(),
-            (int) ((firstHidden - 1) * scalew * av.getCharWidth())
+            Math.round((firstHidden - 1) * scalew * av.getCharWidth())
                     - od.getBoxWidth(), 1.5);
     assertEquals(od.getBoxY(), 0);
     assertEquals(od.getBoxWidth(), boxWidth, 1.5);
     assertEquals(od.getBoxHeight(), boxHeight);
     assertEquals(od.getScrollCol(),
-            (int) (od.getBoxX() / scalew / av.getCharWidth()), 1.5);
+            Math.round(od.getBoxX() / scalew / av.getCharWidth()), 1.5);
     assertEquals(od.getScrollRow(), 0);*/
 
     // click in hidden cols
@@ -604,13 +613,13 @@ public class OverviewDimensionsTest {
     /*xpos = 115;
     assertEquals(
             od.getBoxX(),
-            (int) ((firstHidden - 1) * scalew * av.getCharWidth())
+            Math.round((firstHidden - 1) * scalew * av.getCharWidth())
                     - od.getBoxWidth(), 1.5);
     assertEquals(od.getBoxY(), 0);
     assertEquals(od.getBoxWidth(), boxWidth, 1.5);
     assertEquals(od.getBoxHeight(), boxHeight);
     assertEquals(od.getScrollCol(),
-            (int) (od.getBoxX() / scalew / av.getCharWidth()), 1.5);
+            Math.round(od.getBoxX() / scalew / av.getCharWidth()), 1.5);
     assertEquals(od.getScrollRow(), 0);*/
 
     // click off end of alignment
@@ -619,13 +628,13 @@ public class OverviewDimensionsTest {
     /*    xpos = 3000;
         assertEquals(
                 od.getBoxX(),
-                (int) ((firstHidden - 1) * scalew * av.getCharWidth())
+                Math.round((firstHidden - 1) * scalew * av.getCharWidth())
                         - od.getBoxWidth(), 1.5);
         assertEquals(od.getBoxY(), 0);
         assertEquals(od.getBoxWidth(), boxWidth, 1.5);
         assertEquals(od.getBoxHeight(), boxHeight);
         assertEquals(od.getScrollCol(),
-                (int) (od.getBoxX() / scalew / av.getCharWidth()), 1.5);
+                Math.round(od.getBoxX() / scalew / av.getCharWidth()), 1.5);
         assertEquals(od.getScrollRow(), 0);*/
   }
 
@@ -637,31 +646,33 @@ public class OverviewDimensionsTest {
   public void testSetBoxFromViewport()
   {
     // move viewport to start of alignment
-    moveViewport(av, od, 0, 0);
+    moveViewport(0, 0);
     assertEquals(od.getBoxX(), 0);
     assertEquals(od.getBoxY(), 0);
     assertEquals(od.getBoxWidth(), boxWidth);
     assertEquals(od.getBoxHeight(), boxHeight);
 
     // move viewport to right
-    moveViewportH(av, od, 70);
-    assertEquals(od.getBoxX(), (int) (70 * scalew * av.getCharWidth()));
+    moveViewportH(70);
+    assertEquals(od.getBoxX(), Math.round(70 * scalew * av.getCharWidth()));
     assertEquals(od.getBoxY(), 0);
     assertEquals(od.getBoxWidth(), boxWidth);
     assertEquals(od.getBoxHeight(), boxHeight);
 
     // move viewport down
-    moveViewportV(av, od, 100);
-    assertEquals(od.getBoxX(), (int) (70 * scalew * av.getCharWidth()));
-    assertEquals(od.getBoxY(), (int) (100 * scaleh * av.getCharHeight()));
+    moveViewportV(100);
+    assertEquals(od.getBoxX(), Math.round(70 * scalew * av.getCharWidth()));
+    assertEquals(od.getBoxY(),
+            Math.round(100 * scaleh * av.getCharHeight()));
     assertEquals(od.getBoxWidth(), boxWidth);
     assertEquals(od.getBoxHeight(), boxHeight);
 
     // move viewport to bottom right
-    moveViewport(av, od, 98, 508);
-    assertEquals(od.getBoxX(), (int) (98 * scalew * av.getCharWidth()));
-    assertEquals(od.getBoxY(), (int) (508 * scaleh * av.getCharHeight()));
-    assertEquals(od.getBoxWidth(), boxWidth);
+    moveViewport(98, 508);
+    assertEquals(od.getBoxX(), Math.round(98 * scalew * av.getCharWidth()));
+    assertEquals(od.getBoxY(),
+            Math.round(508 * scaleh * av.getCharHeight()));
+    assertEquals(od.getBoxWidth(), boxWidth, 1.5);
     assertEquals(od.getBoxHeight(), boxHeight);
   }
 
@@ -674,20 +685,20 @@ public class OverviewDimensionsTest {
   {
     int firstHidden = 0;
     int lastHidden = 20;
-    av.hideColumns(firstHidden, lastHidden);
+    hideColumns(firstHidden, lastHidden);
 
     // move viewport to start of alignment
-    moveViewport(av, od, 0, 0);
+    moveViewport(0, 0);
     assertEquals(od.getBoxX(),
-            (int) ((lastHidden + 1) * scalew * av.getCharWidth()));
+            Math.round((lastHidden + 1) * scalew * av.getCharWidth()));
     assertEquals(od.getBoxY(), 0);
     assertEquals(od.getBoxWidth(), boxWidth);
     assertEquals(od.getBoxHeight(), boxHeight);
 
     // move viewport to end of alignment - need to make startRes by removing
     // hidden cols because of how viewport/overview are implemented
-    moveViewport(av, od, 98 - lastHidden - 1, 0);
-    assertEquals(od.getBoxX(), 98 * scalew * av.getCharWidth(), 1.5);
+    moveViewport(98 - lastHidden - 1, 0);
+    assertEquals(od.getBoxX(), Math.round(98 * scalew * av.getCharWidth()));
     assertEquals(od.getBoxY(), 0);
     assertEquals(od.getBoxWidth(), boxWidth);
     assertEquals(od.getBoxHeight(), boxHeight);
@@ -702,40 +713,44 @@ public class OverviewDimensionsTest {
   {
     int firstHidden = 65;
     int lastHidden = 75;
-    av.hideColumns(firstHidden, lastHidden);
+    hideColumns(firstHidden, lastHidden);
 
     // move viewport before hidden columns
-    moveViewport(av, od, 3, 0);
-    assertEquals(od.getBoxX(), (int) (3 * scalew * av.getCharWidth()));
+    moveViewport(3, 0);
+
+    assertEquals(od.getBoxX(), Math.round(3 * scalew * av.getCharWidth()));
     assertEquals(od.getBoxY(), 0);
+    System.out.println(od.getBoxWidth());
     assertEquals(od.getBoxWidth(), boxWidth);
+    System.out.println(od.getBoxWidth());
     assertEquals(od.getBoxHeight(), boxHeight);
 
     // move viewport to left of hidden columns with overlap
-    moveViewport(av, od, 10, 0);
-    assertEquals(od.getBoxX(), (int) (10 * scalew * av.getCharWidth()));
+    moveViewport(10, 0);
+    assertEquals(od.getBoxX(), Math.round(10 * scalew * av.getCharWidth()));
     assertEquals(od.getBoxY(), 0);
     assertEquals(
             od.getBoxWidth(),
             boxWidth
-                    + (int) ((lastHidden - firstHidden + 1) * scalew * av
-                            .getCharWidth()), 1.5);
+                    + Math.round((lastHidden - firstHidden + 1) * scalew
+                            * av.getCharWidth()));
     assertEquals(od.getBoxHeight(), boxHeight);
 
     // move viewport to straddle hidden columns
-    moveViewport(av, od, 60, 0);
-    assertEquals(od.getBoxX(), (int) (60 * scalew * av.getCharWidth()));
+    moveViewport(60, 0);
+    assertEquals(od.getBoxX(), Math.round(60 * scalew * av.getCharWidth()));
     assertEquals(od.getBoxY(), 0);
     assertEquals(
             od.getBoxWidth(),
             boxWidth
-                    + (int) ((lastHidden - firstHidden + 1) * scalew * av
-                            .getCharWidth()), 1.5);
+                    + Math.round((lastHidden - firstHidden + 1) * scalew
+                            * av
+.getCharWidth()));
     assertEquals(od.getBoxHeight(), boxHeight);
 
     // move viewport to right of hidden columns, no overlap
-    moveViewport(av, od, 80 - (lastHidden - firstHidden + 1), 0);
-    assertEquals(od.getBoxX(), (int) (80 * scalew * av.getCharWidth()), 1.5);
+    moveViewport(80 - (lastHidden - firstHidden + 1), 0);
+    assertEquals(od.getBoxX(), Math.round(80 * scalew * av.getCharWidth()));
     assertEquals(od.getBoxY(), 0);
     assertEquals(od.getBoxWidth(), boxWidth);
     assertEquals(od.getBoxHeight(), boxHeight);
@@ -751,33 +766,314 @@ public class OverviewDimensionsTest {
   {
     int firstHidden = 145;
     int lastHidden = 157;
-    av.hideColumns(firstHidden, lastHidden);
+    hideColumns(firstHidden, lastHidden);
 
     // move viewport before hidden columns
-    moveViewport(av, od, 3, 0);
-    assertEquals(od.getBoxX(), (int) (3 * scalew * av.getCharWidth()), 1.5);
+    moveViewport(3, 0);
+    assertEquals(od.getBoxX(), Math.round(3 * scalew * av.getCharWidth()),
+            1.5);
     assertEquals(od.getBoxY(), 0);
     assertEquals(od.getBoxWidth(), boxWidth);
     assertEquals(od.getBoxHeight(), boxHeight);
 
     // move viewport to hidden columns
     // TODO boxwidth includes hidden in overview panel (why?)
-    moveViewport(av, od, 98, 0);
-    assertEquals(od.getBoxX(), (int) (98 * scalew * av.getCharWidth()), 1.5);
+    moveViewport(98, 0);
+    assertEquals(od.getBoxX(), Math.round(98 * scalew * av.getCharWidth()),
+            1.5);
     assertEquals(od.getBoxY(), 0);
     assertEquals(
             od.getBoxWidth(),
             boxWidth
-                    + (int) ((lastHidden - firstHidden + 1) * scalew * av
-                            .getCharWidth()), 1.5);
+                    + Math.round((lastHidden - firstHidden + 1) * scalew
+                            * av.getCharWidth()), 1.5);
     assertEquals(od.getBoxHeight(), boxHeight);
   }
 
+  /**
+   * Test that the box position is set correctly when there are hidden rows at
+   * the start
+   */
+  @Test(groups = { "Functional" })
+  public void testSetBoxFromViewportHiddenRowsAtStart()
+  {
+    int firstHidden = 0;
+    int lastHidden = 20;
+    hideSequences(firstHidden, lastHidden + 1, lastHidden + 1);
+
+    // move viewport to start of alignment:
+    // box moves to below hidden rows, height remains same
+    moveViewport(0, 0);
+    assertEquals(od.getBoxX(), 0);
+    assertEquals(od.getBoxY(),
+            Math.round((lastHidden + 1) * scaleh * av.getCharHeight()));
+    assertEquals(od.getBoxWidth(), boxWidth);
+    assertEquals(od.getBoxHeight(), boxHeight);
+
+    // move viewport to end of alignment, need to account for hidden rows
+    // because of how alignment panel/overview panel are implemented
+
+    // AlignViewport adjusts endSeq using Alignment height which excludes hidden
+    // rows if we happen to be at the bottom of the alignment
+    // od.setBoxPosition adjusts endSeq to include hidden rows
+    // od.checkValid adjusts scroll position to exclude hidden rows
+    // TODO this test fails because of the above!
+
+    /*    moveViewport(0, 525 - viewHeight - lastHidden - 1);
+        assertEquals(od.getBoxX(), 0);
+        assertEquals(od.getBoxY(),
+                (525 - viewHeight) * scaleh
+                * av.getCharHeight(), 1.5);
+        assertEquals(od.getBoxWidth(), boxWidth, 1.5);
+        assertEquals(od.getBoxHeight(), boxHeight, 1.5);*/
+  }
+
+  /**
+   * Test that the box position is set correctly when there are hidden rows in
+   * the middle
+   */
+  @Test(groups = { "Functional" })
+  public void testSetBoxFromViewportHiddenRowsInMiddle()
+  {
+    int firstHidden = 200;
+    int lastHidden = 210;
+    hideSequences(firstHidden, lastHidden + 1, lastHidden + 1);
+
+    // move viewport to start of alignment:
+    // box, height etc as in non-hidden case
+    // TODO fails with boxy=12 because that's what setBoxPosition sets it to!
+    /*moveViewport(0, 0);
+    assertEquals(od.getBoxX(), 0);
+    assertEquals(od.getBoxY(), 0);
+    assertEquals(od.getBoxWidth(), boxWidth, 1.5);
+    assertEquals(od.getBoxHeight(), boxHeight, 1.5);*/
+
+    // move viewport to straddle hidden rows
+    // TODO also fails with boxY out by 12
+    /*moveViewport(0, 198);
+    assertEquals(od.getBoxX(), 0);
+    assertEquals(od.getBoxY(), Math.round (198 * scaleh * av.getCharHeight()),
+            1.5);
+    assertEquals(od.getBoxWidth(), boxWidth, 1.5);
+    assertEquals(od.getBoxHeight(), boxHeight
+            + ((lastHidden - firstHidden) * scaleh * av.getCharHeight()),
+            1.5);*/
+  }
+
+  /**
+   * Test that the box position is set correctly when there are hidden rows at
+   * the bottom
+   */
+  @Test(groups = { "Functional" })
+  public void testSetBoxFromViewportHiddenRowsAtEnd()
+  {
+    int firstHidden = 500;
+    int lastHidden = 524;
+    hideSequences(firstHidden - 1, lastHidden, firstHidden - 1);
+
+    // move viewport to start of alignment:
+    // box, height etc as in non-hidden case
+    moveViewport(0, 0);
+    assertEquals(od.getBoxX(), 0);
+    assertEquals(od.getBoxY(), 0);
+    assertEquals(od.getBoxWidth(), boxWidth);
+    assertEquals(od.getBoxHeight(), boxHeight);
+
+    // move viewport to end of alignment
+    // TODO fails with wrong boxHeight who knows why
+    /*moveViewport(0, firstHidden - viewHeight - 1);
+        assertEquals(od.getBoxX(), 0);
+        assertEquals(od.getBoxY(),
+     Math.round ((firstHidden - viewHeight - 1)
+                * scaleh * av.getCharHeight()), 1.5);
+        assertEquals(od.getBoxWidth(), boxWidth, 1.5);
+        assertEquals(
+                od.getBoxHeight(),
+                boxHeight
+                        + Math.round ((lastHidden - firstHidden + 1) * scaleh * av
+                                .getCharHeight()), 1.5);*/
+
+  }
+
+  /**
+   * Test setting of the box position, when there are hidden rows at the start
+   * of the alignment
+   */
+  @Test(groups = { "Functional" })
+  public void testFromMouseWithHiddenRowsAtStart()
+  {
+    od.setBoxPositionByMouse(0, 0);
+    assertEquals(od.getBoxX(), 0);
+    assertEquals(od.getBoxY(), 0);
+    assertEquals(od.getBoxHeight(), boxHeight);
+    assertEquals(od.getBoxWidth(), boxWidth);
+    assertEquals(od.getScrollCol(), 0);
+    assertEquals(od.getScrollRow(), 0);
+
+    // hide rows at start and check updated box position is correct
+    // changes boxY but not boxheight
+    int lastHiddenRow = 30;
+    hideSequences(0, lastHiddenRow + 1, lastHiddenRow + 1);
+
+    od.setBoxPosition();
+    assertEquals(od.getBoxX(), 0);
+    assertEquals(od.getBoxY(),
+            Math.round((lastHiddenRow + 1) * scaleh * av.getCharHeight()));
+    assertEquals(od.getBoxWidth(), boxWidth);
+    assertEquals(od.getBoxHeight(), boxHeight);
+
+    // click in hidden rows
+    // TODO fails because boxHeight is 27 not 25 (possible rounding issue)
+    /*    mouseClick(od, 0, 0);
+        assertEquals(od.getBoxX(), 0);
+        assertEquals(od.getBoxY(), 0);
+        assertEquals(od.getBoxWidth(), boxWidth, 1.5);
+        assertEquals(od.getBoxHeight(), boxHeight
+                + Math.round ((lastHiddenRow + 1) * scaleh * av.getCharHeight()),
+                1.5);*/
+
+    // click below hidden rows
+    mouseClick(od, 0, 150);
+    assertEquals(od.getBoxX(), 0);
+    assertEquals(od.getBoxY(), 150, 1.5);
+    assertEquals(od.getBoxWidth(), boxWidth);
+    assertEquals(od.getBoxHeight(), boxHeight);
+  }
+
+  /**
+   * Test setting of the box position, when there are hidden rows at the middle
+   * of the alignment
+   */
+  @Test(groups = { "Functional" })
+  public void testFromMouseWithHiddenRowsInMiddle()
+  {
+    assertEquals(od.getBoxHeight(), 10);
+
+    od.setBoxPositionByMouse(0, 0);
+
+    assertEquals(od.getBoxHeight(), 10);
+
+    assertEquals(od.getBoxX(), 0);
+    assertEquals(od.getBoxY(), 0);
+    assertEquals(od.getBoxWidth(), boxWidth);
+    assertEquals(od.getBoxHeight(), boxHeight);
+    assertEquals(od.getScrollCol(), 0);
+    assertEquals(od.getScrollRow(), 0);
+
+    // hide rows in middle and check updated box position is correct
+    // no changes
+    int firstHiddenRow = 50;
+    int lastHiddenRow = 60;
+    assertEquals(od.getBoxHeight(), 10);
+    hideSequences(firstHiddenRow, lastHiddenRow + 1, lastHiddenRow + 1);
+    assertEquals(od.getBoxHeight(), 10);
+
+    od.setBoxPosition();
+    assertEquals(od.getBoxHeight(), 10);
+
+    assertEquals(od.getBoxX(), 0);
+    assertEquals(od.getBoxY(), 0);
+    assertEquals(od.getBoxWidth(), boxWidth);
+    assertEquals(od.getBoxHeight(), boxHeight);
+
+    // click above hidden rows, so that box overlaps
+    int ypos = 40;
+    // TODO test fails because box does not change height - dealt with by scroll
+    // values
+    /*    mouseClick(od, 0, Math.round (ypos / scaleh / av.getCharHeight()));
+        assertEquals(od.getBoxX(), 0);
+        assertEquals(od.getBoxY(), Math.round (ypos / scaleh / av.getCharHeight()),
+                1.5);
+        assertEquals(od.getBoxWidth(), boxWidth, 1.5);
+        assertEquals(
+                od.getBoxHeight(),
+                boxHeight
+                        + Math.round ((lastHiddenRow - firstHiddenRow + 1) / scaleh / av
+                                .getCharHeight()), 1.5);
+    */
+    // click so that box straddles hidden rows
+    ypos = 48;
+    // TODO test fails because box does not change height - dealt with by scroll
+    // values
+    /*mouseClick(od, 0, Math.round (ypos / scaleh / av.getCharHeight()));
+    assertEquals(od.getBoxX(), 0);
+    assertEquals(od.getBoxY(), Math.round (ypos / scaleh / av.getCharHeight()),
+            1.5);
+    assertEquals(od.getBoxWidth(), boxWidth, 1.5);
+    assertEquals(
+            od.getBoxHeight(),
+            boxHeight
+                    + Math.round ((lastHiddenRow - firstHiddenRow + 1) / scaleh / av
+                            .getCharHeight()), 1.5);*/
+  }
+
+  /**
+   * Test setting of the box position, when there are hidden rows at the end of
+   * the alignment
+   */
+  @Test(groups = { "Functional" })
+  public void testFromMouseWithHiddenRowsAtEnd()
+  {
+    od.setBoxPositionByMouse(0, 0);
+    assertEquals(od.getBoxX(), 0);
+    assertEquals(od.getBoxY(), 0);
+    assertEquals(od.getBoxWidth(), boxWidth);
+    assertEquals(od.getBoxHeight(), boxHeight);
+    assertEquals(od.getScrollCol(), 0);
+    assertEquals(od.getScrollRow(), 0);
+
+    // hide rows at end and check updated box position is correct
+    // no changes
+    int firstHidden = 500;
+    int lastHidden = 524;
+    hideSequences(firstHidden - 1, lastHidden, firstHidden - 1);
+
+    od.setBoxPosition();
+    assertEquals(od.getBoxX(), 0);
+    assertEquals(od.getBoxY(), 0);
+    assertEquals(od.getBoxWidth(), boxWidth);
+    assertEquals(od.getBoxHeight(), boxHeight);
+
+    // click above hidden rows
+    int ypos = 40; // row 40
+    mouseClick(od, 0, Math.round(ypos * scaleh * av.getCharHeight()));
+    assertEquals(od.getBoxX(), 0);
+    assertEquals(od.getBoxY(),
+            Math.round(ypos * scaleh * av.getCharHeight()));
+    assertEquals(od.getBoxWidth(), boxWidth);
+    assertEquals(od.getBoxHeight(), boxHeight);
+
+    // click above hidden rows so box overlaps
+    // boxY moved upwards, boxHeight remains same
+    // TODO fails with boxY located at row 497 - correction done by
+    // setScrollValues
+    /*   ypos = 497; // row 497
+       mouseClick(od, 0, Math.round (ypos * scaleh * av.getCharHeight()));
+       assertEquals(od.getBoxX(), 0);
+       assertEquals(
+               od.getBoxY(),
+               Math.round ((firstHidden - viewHeight) * scaleh * av.getCharHeight()),
+               1.5);
+       assertEquals(od.getBoxWidth(), boxWidth, 1.5);
+       assertEquals(od.getBoxHeight(), boxHeight, 1.5);*/
+
+    // click within hidden rows
+    ypos = 505;
+    // TODO: fails with wrong boxHeight - correction done by setScrollValues(?)
+    /*mouseClick(od, 0, Math.round (ypos * scaleh * av.getCharHeight()));
+    assertEquals(od.getBoxX(), 0);
+    assertEquals(
+            od.getBoxY(),
+            Math.round ((firstHidden - viewHeight) * scaleh * av.getCharHeight()),
+            1.5);
+    assertEquals(od.getBoxWidth(), boxWidth, 1.5);
+    assertEquals(od.getBoxHeight(), boxHeight, 1.5);*/
+  }
+
   /*
    * Move viewport horizontally: startRes + previous width gives new horizontal extent. Vertical extent stays the same.
    */
-  private void moveViewportH(AlignViewport av, OverviewDimensions od,
-          int startRes)
+  private void moveViewportH(int startRes)
   {
     int width = av.getEndRes() - av.getStartRes();
     av.setStartRes(startRes);
@@ -788,27 +1084,24 @@ public class OverviewDimensionsTest {
   /*
    * Move viewport vertically: startSeq and endSeq give new vertical extent. Horizontal extent stays the same.
    */
-  private void moveViewportV(AlignViewport av, OverviewDimensions od,
-          int startSeq)
+  private void moveViewportV(int startSeq)
   {
-    int height = av.getEndSeq() - av.getStartSeq();
     av.setStartSeq(startSeq);
-    av.setEndSeq(startSeq + height);
+    av.setEndSeq(startSeq + viewHeight);
     od.setBoxPosition();
   }
 
   /*
    * Move viewport horizontally and vertically.
    */
-  private void moveViewport(AlignViewport av, OverviewDimensions od,
-          int startRes, int startSeq)
+  private void moveViewport(int startRes, int startSeq)
   {
     int width = av.getEndRes() - av.getStartRes();
-    int height = av.getEndSeq() - av.getStartSeq();
+
     av.setStartRes(startRes);
     av.setEndRes(startRes + width);
     av.setStartSeq(startSeq);
-    av.setEndSeq(startSeq + height);
+    av.setEndSeq(startSeq + viewHeight);
     od.setBoxPosition();
   }
 
@@ -817,13 +1110,10 @@ public class OverviewDimensionsTest {
    */
   private void mouseClick(OverviewDimensions od, int x, int y)
   {
-    od.setBoxX(x);
-    od.setBoxY(y);
-    od.checkValid();
+    od.setBoxPositionByMouse(x, y);
     // updates require an OverviewPanel to exist which it doesn't here
     // so call setBoxPosition() as it would be called by the AlignmentPanel
     // normally
-    AlignViewport av = af.getViewport();
     int width = av.getEndRes() - av.getStartRes();
     int height = av.getEndSeq() - av.getStartSeq();
     av.setStartRes(od.getScrollCol());
@@ -838,8 +1128,55 @@ public class OverviewDimensionsTest {
     mouseClick(od, xpos, ypos);
     assertEquals(od.getBoxX(), xpos, 1.5);
     assertEquals(od.getBoxY(), ypos, 1.5);
-    assertEquals(od.getBoxWidth(), boxWidth);
-    assertEquals(od.getBoxHeight(), boxHeight);
+    assertEquals(od.getBoxWidth(), boxWidth, 1.5);
+    assertEquals(od.getBoxHeight(), boxHeight, 1.5);
+
+  }
 
+  /*
+   * Hide sequences between start and end, using hideseq to do the hiding
+   * (start <= hideseq <= end. Sequence hideseq is not hidden but the others are
+   */
+  private void hideSequences(int start, int end, int hideseq)
+  {
+    SequenceGroup sg = new SequenceGroup();
+    List<SequenceI> allseqs = av.getAlignment().getSequences();
+    for (int i = start; i <= end; ++i)
+    {
+      sg.addSequence(allseqs.get(i), false);
+    }
+    av.setSelectionGroup(sg);
+
+    /*
+     * hide group
+     */
+    av.hideSequences(allseqs.get(hideseq), true);
+
+    while (av.isCalcInProgress())
+    {
+      try
+      {
+        Thread.sleep(50);
+      } catch (InterruptedException e)
+      {
+        System.out.println("Hiding seqs interruption");
+      }
+    }
+  }
+
+  private void hideColumns(int firstHidden, int lastHidden)
+  {
+    av.hideColumns(firstHidden, lastHidden);
+
+    while (av.isCalcInProgress())
+    {
+      try
+      {
+        Thread.sleep(50);
+      } catch (InterruptedException e)
+      {
+        System.out.println("Hiding cols interruption");
+      }
+    }
   }
 }