Added abstract class for tree viewer testing, rearranged existing tests

[jalview.git] / test / jalview / ext / archaeopteryx / AptxPhylogenyTreeTest.java

diff --git a/test/jalview/ext/archaeopteryx/AptxPhylogenyTreeTest.java b/test/jalview/ext/archaeopteryx/AptxPhylogenyTreeTest.java
index 6e324d7..5e00496 100644 (file)
--- a/test/jalview/ext/archaeopteryx/AptxPhylogenyTreeTest.java
+++ b/test/jalview/ext/archaeopteryx/AptxPhylogenyTreeTest.java
@@ -3,8 +3,6 @@ package jalview.ext.archaeopteryx;
 import static org.testng.Assert.assertEquals;
 import static org.testng.Assert.assertTrue;
 
-import jalview.bin.Jalview;
-import jalview.gui.Desktop;
 import jalview.gui.JvOptionPane;
 
 import org.forester.archaeopteryx.MainFrame;
@@ -14,7 +12,7 @@ import org.forester.phylogeny.PhylogenyNode;
 import org.testng.annotations.BeforeClass;
 import org.testng.annotations.Test;
 
-public class AptxPhylogenyTreeTest
+public class AptxPhylogenyTreeTest extends TreeViewTest
 {
   final Phylogeny inputTree = new Phylogeny();
 
@@ -22,7 +20,7 @@ public class AptxPhylogenyTreeTest
 
   final PhylogenyNode ancestor1Node = new PhylogenyNode("ancestor 1");
 
-  final PhylogenyNode ancestor2Node = new PhylogenyNode("ancestor 2");
+  final PhylogenyNode ancestor2Node = new PhylogenyNode("leaf 2");
 
   final PhylogenyNode leaf1aNode = new PhylogenyNode("leaf 1a");
 
@@ -30,14 +28,14 @@ public class AptxPhylogenyTreeTest
 
   final PhylogenyNode leaf1cNode = new PhylogenyNode("leaf 1c");
 
-  TreePanel treePanel;
-
   Phylogeny tree;
 
+  TreePanel treePanel;
+
   MainFrame aptx;
 
-  Jalview jalview;
 
+  @Override
   @BeforeClass(alwaysRun = true)
   public void setUpTree()
   {
@@ -60,13 +58,12 @@ public class AptxPhylogenyTreeTest
 
   }
 
-  @BeforeClass(dependsOnMethods = { "setUpTree" })
-  public void setUpJalview()
+  @BeforeClass(dependsOnMethods = { "setUpJalview", "setUpTree" })
+  public void createTreeView()
   {
-    String[] args = new String[0];
-    Jalview.main(args);
-    aptx = ArchaeopteryxInit.createInstance(inputTree);
 
+    treeView = ArchaeopteryxInit.createInstance(inputTree);
+    aptx = (MainFrame) treeView; // pretty ugly
     treePanel = aptx.getMainPanel().getCurrentTreePanel();
     tree = treePanel.getPhylogeny();
   }
@@ -80,26 +77,6 @@ public class AptxPhylogenyTreeTest
   }
 
   @Test(groups = "Functional")
-  public void testAptxExists()
-  {
-    assertTrue(aptx != null);
-  }
-
-
-  @Test(groups = "Functional")
-  public void testAptxBoundToJalview()
-  {
-    assertTrue(aptx.getTopLevelAncestor() instanceof Desktop);
-  }
-
-  @Test(groups = "Functional")
-  public void testShowingAptx()
-  {
-    assertTrue(aptx.isShowing());
-
-  }
-
-  @Test(groups = "Functional")
   public void testMainPanelExists()
   {
     assertTrue(aptx.getMainPanel() != null);
@@ -111,12 +88,15 @@ public class AptxPhylogenyTreeTest
     assertTrue(aptx.getMainPanel().getCurrentTreePanel() != null);
   }
 
+  @Override
   @Test(groups = "Functional")
   public void testTreeLoaded()
   {
-    assertTrue(aptx.getMainPanel().getCurrentTreePanel() != null);
+    assertTrue(aptx.getMainPanel().getCurrentTreePanel()
+            .getPhylogeny() != null);
   }
 
+  @Override
   @Test(groups = "Functional")
   public void testChildNodesCount()
   {
@@ -124,22 +104,25 @@ public class AptxPhylogenyTreeTest
 
   }
 
+  @Override
   @Test(groups = "Functional")
-  public void testCorrectLeafToParentBranchLength()
+  public void testChildToParentBranchLength()
   {
     assertEquals(tree.getNode("leaf 1a").getDistanceToParent(), 2.0);
 
   }
 
+  @Override
   @Test(groups = "Functional")
-  public void testCorrectAncestorLeafToRootBranchLength()
+  public void testNodeToRootBranchLength()
   {
-    assertEquals(tree.getNode("ancestor 2").getDistanceToParent(), 42.0);
+    assertEquals(tree.getNode("leaf 2").getDistanceToParent(), 42.0);
 
   }
 
+  @Override
   @Test(groups = "Functional")
-  public void testCorrectLeafToRootBranchLength()
+  public void testDistantNodeToRootBranchLength()
   {
     assertEquals(tree.getNode("leaf 1c").calculateDistanceToRoot(),
             4.0 + 36.0);