JAL-3054 table tooltip follow mouse in a sensible manner

[jalview.git] / test / jalview / datamodel / features / NCListTest.java

diff --git a/test/jalview/datamodel/features/NCListTest.java b/test/jalview/datamodel/features/NCListTest.java
index 65c8c91..2c7f752 100644 (file)
--- a/test/jalview/datamodel/features/NCListTest.java
+++ b/test/jalview/datamodel/features/NCListTest.java
@@ -4,6 +4,8 @@ import static org.testng.Assert.assertFalse;
 import static org.testng.Assert.assertSame;
 import static org.testng.Assert.assertTrue;
 
+import jalview.datamodel.ContiguousI;
+import jalview.datamodel.Range;
 import jalview.datamodel.SequenceFeature;
 
 import java.util.ArrayList;
@@ -12,6 +14,8 @@ import java.util.Comparator;
 import java.util.List;
 import java.util.Random;
 
+import junit.extensions.PA;
+
 import org.testng.annotations.DataProvider;
 import org.testng.annotations.Test;
 
@@ -212,20 +216,20 @@ public class NCListTest
       assertEquals(ncl.size(), features.size());
       int toDelete = random.nextInt(features.size());
       SequenceFeature entry = features.get(toDelete);
-      String lastDeleted = "'" + entry.toString() + "'";
       assertTrue(ncl.contains(entry), String.format(
-              "NCList doesn't contain entry [%d] %s!", deleted,
-              lastDeleted));
+              "NCList doesn't contain entry [%d] '%s'!", deleted,
+              entry.toString()));
+
       ncl.delete(entry);
       assertFalse(ncl.contains(entry), String.format(
-              "NCList still contains deleted entry [%d] %s!", deleted,
-              lastDeleted));
+              "NCList still contains deleted entry [%d] '%s'!", deleted,
+              entry.toString()));
       features.remove(toDelete);
       deleted++;
 
       assertTrue(ncl.isValid(), String.format(
-              "NCList invalid after %d deletions, last deleted was %s",
-              deleted, lastDeleted));
+              "NCList invalid after %d deletions, last deleted was '%s'",
+              deleted, entry.toString()));
 
       /*
        * brute force check that deleting one entry didn't delete any others
@@ -234,8 +238,8 @@ public class NCListTest
       {
         SequenceFeature sf = features.get(i);
         assertTrue(ncl.contains(sf), String.format(
-                "NCList doesn't contain entry [%d] %s after deleting %s!",
-                i, sf.toString(), lastDeleted));
+                        "NCList doesn't contain entry [%d] %s after deleting '%s'!",
+                        i, sf.toString(), entry.toString()));
       }
     }
     assertEquals(ncl.size(), 0); // all gone
@@ -584,22 +588,22 @@ public class NCListTest
     assertEquals(ncl.toString(), "[40-50 [42-44 [43-43], 46-48]]");
     assertTrue(ncl.isValid());
 
-    r1.start = 43;
+    PA.setValue(r1, "start", 43);
     assertFalse(ncl.isValid()); // r2 not inside r1
-    r1.start = 40;
+    PA.setValue(r1, "start", 40);
     assertTrue(ncl.isValid());
 
-    r3.start = 41;
+    PA.setValue(r3, "start", 41);
     assertFalse(ncl.isValid()); // r3 should precede r2
-    r3.start = 46;
+    PA.setValue(r3, "start", 46);
     assertTrue(ncl.isValid());
 
-    r4.start = 41;
+    PA.setValue(r4, "start", 41);
     assertFalse(ncl.isValid()); // r4 not inside r2
-    r4.start = 43;
+    PA.setValue(r4, "start", 43);
     assertTrue(ncl.isValid());
 
-    r4.start = 44;
+    PA.setValue(r4, "start", 44);
     assertFalse(ncl.isValid()); // r4 has reverse range
   }