X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=test%2Fjalview%2Futil%2FDBRefUtilsTest.java;h=963ca7c807f7296e1baca5fc685b4e094da64f7f;hb=2d8dfad8a0aa6a6febfc0c3bc0379de0b46e0cfc;hp=0ef3c2553d40b474601b12f0d65d6326d07fd021;hpb=5f4e1e4c330b045e9c8bce28ee132a0fca3834d8;p=jalview.git

diff --git a/test/jalview/util/DBRefUtilsTest.java b/test/jalview/util/DBRefUtilsTest.java
index 0ef3c25..963ca7c 100644
--- a/test/jalview/util/DBRefUtilsTest.java
+++ b/test/jalview/util/DBRefUtilsTest.java
@@ -34,6 +34,7 @@ import jalview.datamodel.Sequence;
 import jalview.datamodel.SequenceI;
 import jalview.gui.JvOptionPane;
 
+import java.util.Arrays;
 import java.util.List;
 
 import org.testng.annotations.BeforeClass;
@@ -63,25 +64,25 @@ public class DBRefUtilsTest
     DBRefEntry ref2 = new DBRefEntry("UNIPROT", "1.2", "A12346");
     // Source is converted to upper-case by this constructor!
     DBRefEntry ref3 = new DBRefEntry("Uniprot", "1.2", "A12347");
-    DBRefEntry[] dbrefs = new DBRefEntry[] { ref1, ref2, ref3 };
+    List<DBRefEntry> dbrefs = Arrays.asList(new DBRefEntry[] { ref1, ref2, ref3 });
     String[] sources = new String[] { "EMBL", "UNIPROT" };
 
-    DBRefEntry[] selected = DBRefUtils.selectRefs(dbrefs, sources);
-    assertEquals(3, selected.length);
-    assertSame(ref1, selected[0]);
-    assertSame(ref2, selected[1]);
-    assertSame(ref3, selected[2]);
+    List<DBRefEntry> selected = DBRefUtils.selectRefs(dbrefs, sources);
+    assertEquals(3, selected.size());
+    assertSame(ref1, selected.get(0));
+    assertSame(ref2, selected.get(1));
+    assertSame(ref3, selected.get(2));
 
     sources = new String[] { "EMBL" };
     selected = DBRefUtils.selectRefs(dbrefs, sources);
-    assertEquals(1, selected.length);
-    assertSame(ref1, selected[0]);
+    assertEquals(1, selected.size());
+    assertSame(ref1, selected.get(0));
 
     sources = new String[] { "UNIPROT" };
     selected = DBRefUtils.selectRefs(dbrefs, sources);
-    assertEquals(2, selected.length);
-    assertSame(ref2, selected[0]);
-    assertSame(ref3, selected[1]);
+    assertEquals(2, selected.size());
+    assertSame(ref2, selected.get(0));
+    assertSame(ref3, selected.get(1));
 
     sources = new String[] { "EMBLCDS" };
     selected = DBRefUtils.selectRefs(dbrefs, sources);
@@ -89,10 +90,10 @@ public class DBRefUtilsTest
 
     sources = new String[] { "embl", "uniprot" };
     selected = DBRefUtils.selectRefs(dbrefs, sources);
-    assertEquals(3, selected.length);
-    assertSame(ref1, selected[0]);
-    assertSame(ref2, selected[1]);
-    assertSame(ref3, selected[2]);
+    assertEquals(3, selected.size());
+    assertSame(ref1, selected.get(0));
+    assertSame(ref2, selected.get(1));
+    assertSame(ref3, selected.get(2));
   }
 
   /**
@@ -161,9 +162,9 @@ public class DBRefUtilsTest
   {
     SequenceI seq = new Sequence("Seq1", "ABCD");
     DBRefEntry ref = DBRefUtils.parseToDbRef(seq, "EMBL", "1.2", "a7890");
-    DBRefEntry[] refs = seq.getDBRefs();
-    assertEquals(1, refs.length);
-    assertSame(ref, refs[0]);
+    List<DBRefEntry> refs = seq.getDBRefs();
+    assertEquals(1, refs.size());
+    assertSame(ref, refs.get(0));
     assertEquals("EMBL", ref.getSource());
     assertEquals("1.2", ref.getVersion());
     assertEquals("a7890", ref.getAccessionId());
@@ -182,9 +183,9 @@ public class DBRefUtilsTest
     // TODO: correct PDBEntry and PDB DBRef accessions need to be generated for
     // PDB ref in Stockholm
 
-    DBRefEntry[] refs = seq.getDBRefs();
-    assertEquals(1, refs.length);
-    assertSame(ref, refs[0]);
+    List<DBRefEntry> refs = seq.getDBRefs();
+    assertEquals(1, refs.size());
+    assertSame(ref, refs.get(0));
     assertEquals("PDB", ref.getSource());
     assertEquals("1.2", ref.getVersion());
     // DBRef id is pdbId + chain code
@@ -217,8 +218,9 @@ public class DBRefUtilsTest
     ref5.setMap(new Mapping(new MapList(new int[] { 1, 1 }, new int[] { 1,
         1 }, 1, 1)));
 
-    List<DBRefEntry> matches = DBRefUtils.searchRefs(new DBRefEntry[] {
-        ref1, ref2, ref3, ref4, ref5 }, target);
+    List<DBRefEntry> matches = DBRefUtils.searchRefs(
+    		Arrays.asList(new DBRefEntry[] {
+        ref1, ref2, ref3, ref4, ref5 }), target, DBRefUtils.SEARCH_MODE_FULL);
     assertEquals(3, matches.size());
     assertSame(ref1, matches.get(0));
     assertSame(ref2, matches.get(1));
@@ -250,8 +252,9 @@ public class DBRefUtilsTest
             new int[] { 1, 1 }, 2, 2));
     ref3.setMap(map3);
 
-    List<DBRefEntry> matches = DBRefUtils.searchRefs(new DBRefEntry[] {
-        ref1, ref2, ref3 }, target);
+    List<DBRefEntry> matches = DBRefUtils.searchRefs(
+    		Arrays.asList(new DBRefEntry[] {
+        ref1, ref2, ref3 }), target, DBRefUtils.SEARCH_MODE_FULL);
     assertEquals(2, matches.size());
     assertSame(ref1, matches.get(0));
     assertSame(ref2, matches.get(1));
@@ -275,7 +278,8 @@ public class DBRefUtilsTest
     ref5.setMap(new Mapping(new MapList(new int[] { 1, 1 }, new int[] { 1,
         1 }, 1, 1)));
 
-    DBRefEntry[] dbrefs = new DBRefEntry[] { ref1, ref2, ref3, ref4, ref5 };
+    List<DBRefEntry> dbrefs = Arrays.asList(new DBRefEntry[] {
+	         ref1, ref2, ref3, ref4, ref5 });
     List<DBRefEntry> matches = DBRefUtils.searchRefs(dbrefs, "A1234");
     assertEquals(3, matches.size());
     assertSame(ref1, matches.get(0));
@@ -303,8 +307,9 @@ public class DBRefUtilsTest
     ref5.setMap(new Mapping(new MapList(new int[] { 1, 1 }, new int[] { 1,
         1 }, 1, 1)));
 
-    List<DBRefEntry> matches = DBRefUtils.searchRefs(new DBRefEntry[] {
-        ref1, ref2, ref3, ref4, ref5 }, target);
+    List<DBRefEntry> matches = DBRefUtils.searchRefs(
+    		Arrays.asList(new DBRefEntry[] {
+        ref1, ref2, ref3, ref4, ref5 }), target, DBRefUtils.SEARCH_MODE_FULL);
     assertEquals(4, matches.size());
     assertSame(ref1, matches.get(0));
     assertSame(ref2, matches.get(1));