JAL-2759 Update to getHiddenBoundaryRight + unit test after review

diff --git a/src/jalview/datamodel/HiddenColumns.java b/src/jalview/datamodel/HiddenColumns.java
index 3b1a574..6a2ca1d 100644 (file)
--- a/src/jalview/datamodel/HiddenColumns.java
+++ b/src/jalview/datamodel/HiddenColumns.java
@@ -868,6 +868,8 @@ public class HiddenColumns
    * @param alPos
    *          the absolute (visible) alignmentPosition to find the next hidden
    *          column for
+   * @return the index of the next hidden column, or alPos if there is no next
+   *         hidden column
    */
   public int getHiddenBoundaryRight(int alPos)
   {

diff --git a/test/jalview/datamodel/HiddenColumnsTest.java b/test/jalview/datamodel/HiddenColumnsTest.java
index 5d57ed2..b4bab22 100644 (file)
--- a/test/jalview/datamodel/HiddenColumnsTest.java
+++ b/test/jalview/datamodel/HiddenColumnsTest.java
@@ -1060,6 +1060,9 @@ public class HiddenColumnsTest
     h.hideColumns(15, 20);
     assertEquals(15, h.getHiddenBoundaryRight(7));
     assertEquals(15, h.getHiddenBoundaryRight(14));
+
+    // returns same value if there is no next hidden column
+    assertEquals(22, h.getHiddenBoundaryRight(22));
   }
 
   @Test(groups = "Functional")