X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;ds=inline;f=test%2Fjalview%2Fio%2FAnnotatedPDBFileInputTest.java;h=1b62ce3dd79de80f3153f15455c78e29c9e0b604;hb=0ccaf35f7e6a5cc62d4f433b39d574d0aa4b2d26;hp=ef88ca5700abfbfefd055a9b9795e68458a3e457;hpb=b5893211716553a96176a37f7c6e293cd6618559;p=jalview.git
diff --git a/test/jalview/io/AnnotatedPDBFileInputTest.java b/test/jalview/io/AnnotatedPDBFileInputTest.java
index ef88ca5..1b62ce3 100644
--- a/test/jalview/io/AnnotatedPDBFileInputTest.java
+++ b/test/jalview/io/AnnotatedPDBFileInputTest.java
@@ -1,16 +1,37 @@
+/*
+ * Jalview - A Sequence Alignment Editor and Viewer ($$Version-Rel$$)
+ * Copyright (C) $$Year-Rel$$ The Jalview Authors
+ *
+ * This file is part of Jalview.
+ *
+ * Jalview is free software: you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License
+ * as published by the Free Software Foundation, either version 3
+ * of the License, or (at your option) any later version.
+ *
+ * Jalview is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty
+ * of MERCHANTABILITY or FITNESS FOR A PARTICULAR
+ * PURPOSE. See the GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with Jalview. If not, see .
+ * The Jalview Authors are detailed in the 'AUTHORS' file.
+ */
package jalview.io;
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertNotEquals;
-import static org.junit.Assert.assertNotNull;
-import static org.junit.Assert.assertTrue;
+import static org.testng.AssertJUnit.assertEquals;
+import static org.testng.AssertJUnit.assertNotNull;
+import static org.testng.AssertJUnit.assertTrue;
import java.io.File;
+import java.util.List;
-import org.junit.AfterClass;
-import org.junit.Before;
-import org.junit.BeforeClass;
-import org.junit.Test;
+import org.junit.Assert;
+import org.testng.annotations.AfterClass;
+import org.testng.annotations.BeforeClass;
+import org.testng.annotations.BeforeMethod;
+import org.testng.annotations.Test;
import jalview.bin.Cache;
import jalview.datamodel.AlignmentAnnotation;
@@ -18,21 +39,34 @@ import jalview.datamodel.AlignmentI;
import jalview.datamodel.PDBEntry;
import jalview.datamodel.SequenceFeature;
import jalview.datamodel.SequenceI;
+import jalview.datamodel.features.SequenceFeatures;
import jalview.gui.AlignFrame;
import jalview.gui.Desktop;
-import jalview.structure.StructureMapping;
-import jalview.structure.StructureSelectionManager;
+import jalview.gui.JvOptionPane;
+import jalview.structure.StructureImportSettings;
+import jalview.structure.StructureImportSettings.StructureParser;
public class AnnotatedPDBFileInputTest
{
- AlignmentI al;
+ @BeforeClass(alwaysRun = true)
+ public void setUpJvOptionPane()
+ {
+ JvOptionPane.setInteractiveMode(false);
+ JvOptionPane.setMockResponse(JvOptionPane.CANCEL_OPTION);
+ }
- String pdbStr = "examples/1gaq.txt";
+ AlignmentI al;
String pdbId;
- @Before
+ /**
+ * Ensure 'process secondary structure from PDB and add annotations' are set
+ * in preferences, and load PDB example file 1gaq
+ *
+ * @throws Exception
+ */
+ @BeforeMethod(alwaysRun = true)
public void setup() throws Exception
{
Cache.applicationProperties.setProperty("STRUCT_FROM_PDB",
@@ -40,14 +74,17 @@ public class AnnotatedPDBFileInputTest
Cache.applicationProperties.setProperty("ADD_SS_ANN",
Boolean.TRUE.toString());
FileLoader loader = new FileLoader(false);
- AlignFrame af = loader.LoadFileWaitTillLoaded(pdbStr,
- FormatAdapter.FILE);
+ AlignFrame af = loader.LoadFileWaitTillLoaded("examples/1gaq.txt",
+ DataSourceType.FILE);
al = af.getViewport().getAlignment();
- pdbId = al.getSequenceAt(0).getDatasetSequence().getPDBId()
+ pdbId = al.getSequenceAt(0).getDatasetSequence().getAllPDBEntries()
.get(0).getId();
+ StructureImportSettings.setDefaultStructureFileFormat("PDB");
+ // StructureImportSettings
+ // .setDefaultPDBFileParser(StructureParser.JALVIEW_PARSER);
}
- @Test
+ @Test(groups = { "Functional" })
public void checkNoDuplicates()
{
// not strictly a requirement, but strange things may happen if multiple
@@ -60,24 +97,28 @@ public class AnnotatedPDBFileInputTest
{
for (int q = p + 1; q < avec.length; q++)
{
- assertNotEquals(
- "Found a duplicate annotation row " + avec[p].label,
- avec[p], avec[q]);
+ assertTrue("Found a duplicate annotation row " + avec[p].label,
+ avec[p] != avec[q]);
}
}
}
- @Test
+ @Test(groups = { "Functional" }, enabled = false)
public void checkPDBannotationSource()
{
-
+ Assert.fail(
+ "This test is incorrect - does not verify that JmolParser's annotation rows can be recognised as generated by the Jmol parser.");
for (SequenceI asq : al.getSequences())
{
for (AlignmentAnnotation aa : asq.getAnnotation())
{
System.out.println("CalcId: " + aa.getCalcId());
- assertTrue(MCview.PDBfile.isCalcIdForFile(aa, pdbId));
+ if (StructureImportSettings.getDefaultPDBFileParser()
+ .equals(StructureParser.JALVIEW_PARSER))
+ {
+ assertTrue(mc_view.PDBfile.isCalcIdForFile(aa, pdbId));
+ }
}
}
}
@@ -85,46 +126,44 @@ public class AnnotatedPDBFileInputTest
/**
* Check sequence features have been added
*/
- @Test
+ @Test(groups = { "Functional" })
public void checkPDBSequenceFeatures()
{
- StructureSelectionManager ssm = StructureSelectionManager
- .getStructureSelectionManager(Desktop.instance);
- StructureMapping[] mappings = ssm.getMapping("1gaq");
- // suspect we really want to make assertions on sequence features
- // in these mappings' sequencess
/*
* 1GAQ/A
*/
- SequenceFeature[] sf = al.getSequenceAt(0).getSequenceFeatures();
- assertEquals(296, sf.length);
- assertEquals("RESNUM", sf[0].getType());
- assertEquals("GLU:19 1gaqA", sf[0].getDescription());
- assertEquals("RESNUM", sf[295].getType());
- assertEquals("TYR:314 1gaqA", sf[295].getDescription());
+ List sf = al.getSequenceAt(0).getSequenceFeatures();
+ SequenceFeatures.sortFeatures(sf, true);
+ assertEquals(296, sf.size());
+ assertEquals("RESNUM", sf.get(0).getType());
+ assertEquals("GLU: 19 1gaqA", sf.get(0).getDescription());
+ assertEquals("RESNUM", sf.get(295).getType());
+ assertEquals("TYR: 314 1gaqA", sf.get(295).getDescription());
/*
* 1GAQ/B
*/
sf = al.getSequenceAt(1).getSequenceFeatures();
- assertEquals(98, sf.length);
- assertEquals("RESNUM", sf[0].getType());
- assertEquals("ALA:1 1gaqB", sf[0].getDescription());
- assertEquals("RESNUM", sf[97].getType());
- assertEquals("ALA:98 1gaqB", sf[97].getDescription());
+ SequenceFeatures.sortFeatures(sf, true);
+ assertEquals(98, sf.size());
+ assertEquals("RESNUM", sf.get(0).getType());
+ assertEquals("ALA: 1 1gaqB", sf.get(0).getDescription());
+ assertEquals("RESNUM", sf.get(97).getType());
+ assertEquals("ALA: 98 1gaqB", sf.get(97).getDescription());
/*
* 1GAQ/C
*/
sf = al.getSequenceAt(2).getSequenceFeatures();
- assertEquals(296, sf.length);
- assertEquals("RESNUM", sf[0].getType());
- assertEquals("GLU:19 1gaqC", sf[0].getDescription());
- assertEquals("RESNUM", sf[295].getType());
- assertEquals("TYR:314 1gaqC", sf[295].getDescription());
+ SequenceFeatures.sortFeatures(sf, true);
+ assertEquals(296, sf.size());
+ assertEquals("RESNUM", sf.get(0).getType());
+ assertEquals("GLU: 19 1gaqC", sf.get(0).getDescription());
+ assertEquals("RESNUM", sf.get(295).getType());
+ assertEquals("TYR: 314 1gaqC", sf.get(295).getDescription());
}
- @Test
+ @Test(groups = { "Functional" })
public void checkAnnotationWiring()
{
assertTrue(al.getAlignmentAnnotation() != null);
@@ -144,10 +183,9 @@ public class AnnotatedPDBFileInputTest
break;
}
}
- assertTrue(
- "Couldn't find sequence associated annotation "
- + aa.label
- + " on the sequence it is associated with.\nSequence associated editing will fail.",
+ assertTrue("Couldn't find sequence associated annotation "
+ + aa.label
+ + " on the sequence it is associated with.\nSequence associated editing will fail.",
found);
}
}
@@ -156,39 +194,41 @@ public class AnnotatedPDBFileInputTest
/**
* @throws java.lang.Exception
*/
- @BeforeClass
+ @BeforeClass(alwaysRun = true)
public static void setUpBeforeClass() throws Exception
{
- jalview.bin.Jalview.main(new String[]
- { "-props", "test/src/jalview/io/testProps.jvprops" });
+ jalview.bin.Jalview
+ .main(new String[]
+ { "-props", "test/jalview/io/testProps.jvprops" });
}
/**
* @throws java.lang.Exception
*/
- @AfterClass
+ @AfterClass(alwaysRun = true)
public static void tearDownAfterClass() throws Exception
{
- jalview.gui.Desktop.instance.closeAll_actionPerformed(null);
-
+ if (Desktop.instance != null)
+ Desktop.instance.closeAll_actionPerformed(null);
}
- @Test
+ @Test(groups = { "Functional" })
public void testJalviewProjectRelocationAnnotation() throws Exception
{
String inFile = "examples/1gaq.txt";
String tfile = File.createTempFile("JalviewTest", ".jvp")
.getAbsolutePath();
- AlignFrame af = new jalview.io.FileLoader().LoadFileWaitTillLoaded(
- inFile, FormatAdapter.FILE);
+ AlignFrame af = new jalview.io.FileLoader()
+ .LoadFileWaitTillLoaded(inFile, DataSourceType.FILE);
assertTrue("Didn't read input file " + inFile, af != null);
+ af.saveAlignment(tfile, FileFormat.Jalview);
assertTrue("Failed to store as a project.",
- af.saveAlignment(tfile, "Jalview"));
+ af.isSaveAlignmentSuccessful());
af.closeMenuItem_actionPerformed(true);
af = null;
af = new jalview.io.FileLoader().LoadFileWaitTillLoaded(tfile,
- FormatAdapter.FILE);
+ DataSourceType.FILE);
assertTrue("Failed to import new project", af != null);
for (SequenceI asq : af.getViewport().getAlignment().getSequences())
{
@@ -197,22 +237,22 @@ public class AnnotatedPDBFileInputTest
{
sq = sq.getDatasetSequence();
}
- assertNotNull(sq.getPDBId());
- assertEquals("Expected only one PDB ID", sq.getPDBId().size(), 1);
- for (PDBEntry pdbentry : sq.getPDBId())
+ assertNotNull(sq.getAllPDBEntries());
+ assertEquals("Expected only one PDB ID", 1,
+ sq.getAllPDBEntries().size());
+ for (PDBEntry pdbentry : sq.getAllPDBEntries())
{
- System.err.println("PDB Entry " + pdbentry.getId() + " "
- + pdbentry.getFile());
+ System.err.println(
+ "PDB Entry " + pdbentry.getId() + " " + pdbentry.getFile());
boolean exists = false, found = false;
for (AlignmentAnnotation ana : sq.getAnnotation())
{
System.err.println("CalcId " + ana.getCalcId());
if (ana.getCalcId() != null
- && MCview.PDBfile.isCalcIdHandled(ana.getCalcId()))
+ && mc_view.PDBfile.isCalcIdHandled(ana.getCalcId()))
{
exists = true;
- if (MCview.PDBfile.isCalcIdForFile(ana,
- pdbentry.getId()))
+ if (mc_view.PDBfile.isCalcIdForFile(ana, pdbentry.getId()))
{
found = true;
}
@@ -220,8 +260,10 @@ public class AnnotatedPDBFileInputTest
}
if (exists)
{
- assertTrue("Couldn't find any annotation for " + pdbentry.getId()
- + " (file handle " + pdbentry.getFile() + ")", found);
+ assertTrue(
+ "Couldn't find any annotation for " + pdbentry.getId()
+ + " (file handle " + pdbentry.getFile() + ")",
+ found);
}
}
}