JAL-3453 test changes needed to allow tests to proceed unheeded by

[jalview.git] / test / jalview / ws / dbsources / RemoteFormatTest.java

diff --git a/test/jalview/ws/dbsources/RemoteFormatTest.java b/test/jalview/ws/dbsources/RemoteFormatTest.java
index a9cad80..840f439 100644 (file)
--- a/test/jalview/ws/dbsources/RemoteFormatTest.java
+++ b/test/jalview/ws/dbsources/RemoteFormatTest.java
@@ -1,18 +1,31 @@
 package jalview.ws.dbsources;
 
-import static org.testng.AssertJUnit.assertFalse;
-import static org.testng.AssertJUnit.assertTrue;
+import static org.testng.Assert.assertEquals;
+import static org.testng.Assert.assertFalse;
+import static org.testng.Assert.assertNotNull;
+import static org.testng.Assert.assertTrue;
 
+import jalview.analysis.AlignSeq;
+import jalview.bin.Cache;
+import jalview.datamodel.AlignmentI;
+import jalview.datamodel.DBRefSource;
+import jalview.datamodel.SequenceI;
+import jalview.ext.ensembl.EnsemblGenomes;
 import jalview.fts.api.FTSData;
 import jalview.fts.api.FTSDataColumnI;
 import jalview.fts.api.FTSRestClientI;
 import jalview.fts.core.FTSRestRequest;
 import jalview.fts.core.FTSRestResponse;
 import jalview.fts.service.uniprot.UniProtFTSRestClient;
+import jalview.gui.Desktop;
+import jalview.ws.SequenceFetcher;
+import jalview.ws.seqfetcher.DbSourceProxy;
 
 import java.util.ArrayList;
 import java.util.List;
 
+import org.testng.annotations.BeforeTest;
+import org.testng.annotations.DataProvider;
 import org.testng.annotations.Test;
 
 /**
@@ -27,64 +40,64 @@ import org.testng.annotations.Test;
  */
 public class RemoteFormatTest
 {
-  @Test(groups = "Network")
-  public void testFetchUniprot()
-  {
-    // P00340
-  }
-
-  @Test(groups = "Network")
-  public void testFetchPdb()
-  {
-    // 1QIP
-  }
+  SequenceFetcher sf;
 
-  @Test(groups = "Network")
-  public void testFetchEmbl()
+  @BeforeTest(alwaysRun = true)
+  public void setUp() throws Exception
   {
-    // X53828
-  }
+    Desktop.getInstanceOnly();
+    Cache.loadProperties("test/jalview/io/testProps.jvprops");
+    // ensure 'add annotation from structure' is selected
+    Cache.setPropertyNoSave("STRUCT_FROM_PDB",
+            Boolean.TRUE.toString());
+    Cache.setPropertyNoSave("ADD_SS_ANN",
+            Boolean.TRUE.toString());
 
-  @Test(groups = "Network")
-  public void testFetchEmblCDS()
-  {
-    // CAA37824
+    sf = SequenceFetcher.getInstance();
   }
 
-  @Test(groups = "Network")
-  public void testFetchEnsembl()
+  @DataProvider(name = "AccessionData")
+  protected Object[][] getAccessions()
   {
-    // ENSG00000157764
+    return new Object[][] { { DBRefSource.UNIPROT, "P30419" },
+        { DBRefSource.PDB, "1QIP" }, { DBRefSource.EMBL, "X53828" },
+        { DBRefSource.EMBLCDS, "CAA37824" },
+        { DBRefSource.ENSEMBL, "ENSG00000157764" },
+        { new EnsemblGenomes().getDbSource(), "DDB_G0283883" },
+        { new PfamFull().getDbSource(), "PF03760" },
+        { new PfamSeed().getDbSource(), "PF03760" },
+        { new RfamSeed().getDbSource(), "RF00014" } };
   }
 
-  @Test(groups = "Network")
-  public void testFetchEnsemblGenomes()
+  @Test(groups = "Network", dataProvider = "AccessionData")
+  public void testFetchAccession(String dbSource, String accessionId)
+          throws Exception
   {
-    // DDB_G0283883
-  }
+    System.out.println("Fetching " + accessionId + " from " + dbSource);
+    System.err.println(
+            "BH 2019.10.06 note see JAL-2114 JUL 2016 for why this test fails.");
 
-  @Test(groups = "Network")
-  public void testFetchPfamSeed()
-  {
-    // PF03760
-  }
-
-  @Test(groups = "Network")
-  public void testFetchPfamFull()
-  {
-    // PF03760
-  }
-
-  @Test(groups = "Network")
-  public void testFetchRfamSeed()
-  {
-    // RF00014
+    List<DbSourceProxy> sps = sf.getSourceProxy(dbSource);
+    assertFalse(sps.isEmpty());
+    AlignmentI al = sps.get(0).getSequenceRecords(accessionId);
+    assertNotNull(al);
+    assertTrue(al.getHeight() > 0);
+    SequenceI sq = al.getSequenceAt(0);
+    // suppress this check as only Uniprot and PDB acquire PDB refs
+    // assertTrue(sq.getAllPDBEntries().size() > 0, "No PDBEntry on sequence.");
+    assertTrue(sq.getDBRefs().size() > 0, "No DBRef on sequence.");
+    // suppress this test as only certain databases provide 'primary' dbrefs
+    // assertFalse(sq.getPrimaryDBRefs().isEmpty());
+    int length = AlignSeq.extractGaps("-. ", sq.getSequenceAsString())
+            .length();
+    assertEquals(sq.getEnd() - sq.getStart() + 1, length,
+            "Sequence start/end doesn't match number of residues in sequence");
   }
 
   @Test(groups = { "Network" })
   public void testUniprotFreeTextSearch() throws Exception
   {
-    List<FTSDataColumnI> wantedFields = new ArrayList<FTSDataColumnI>();
+    List<FTSDataColumnI> wantedFields = new ArrayList<>();
     FTSRestClientI client = UniProtFTSRestClient.getInstance();
     wantedFields.add(client.getDataColumnByNameOrCode("id"));
     wantedFields.add(client.getDataColumnByNameOrCode("entry name"));
@@ -106,7 +119,8 @@ public class RemoteFormatTest
     assertTrue(response.getSearchSummary().size() > 20);
     // verify we successfully filtered out the header row (JAL-2485)
     FTSData header = response.getSearchSummary().iterator().next();
-    assertFalse("Failed to filter out summary header row",
-            header.getSummaryData()[0].toString().equalsIgnoreCase("Entry"));
+    assertFalse(
+            header.getSummaryData()[0].toString().equalsIgnoreCase("Entry"),
+            "Failed to filter out summary header row");
   }
 }