X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=test%2Fjalview%2Fschemes%2FPIDColourSchemeTest.java;h=59e626db991b57bb068a2f8f37a19ca0ddbeef7e;hb=57738a1f3c19b1c3a00bd3ac5108f8cd0af32f99;hp=fa4b5d922614ae6f7d0a6d821515aa6d63d0bdd9;hpb=301bcabf2b63d653ab9f5ce7102c1ea8951d6d75;p=jalview.git

diff --git a/test/jalview/schemes/PIDColourSchemeTest.java b/test/jalview/schemes/PIDColourSchemeTest.java
index fa4b5d9..59e626d 100644
--- a/test/jalview/schemes/PIDColourSchemeTest.java
+++ b/test/jalview/schemes/PIDColourSchemeTest.java
@@ -2,16 +2,16 @@ package jalview.schemes;
 
 import static org.testng.Assert.assertEquals;
 
+import java.awt.Color;
+
+import org.testng.annotations.Test;
+
 import jalview.datamodel.SequenceI;
 import jalview.gui.AlignFrame;
 import jalview.gui.AlignViewport;
 import jalview.io.DataSourceType;
 import jalview.io.FileLoader;
 
-import java.awt.Color;
-
-import org.testng.annotations.Test;
-
 public class PIDColourSchemeTest
 {
   static final Color white = Color.white;
@@ -85,18 +85,15 @@ public class PIDColourSchemeTest
             DataSourceType.PASTE);
     AlignViewport viewport = af.getViewport();
     viewport.setIgnoreGapsConsensus(false, af.alignPanel);
-    while (viewport.getConsensusSeq() == null)
+    do
     {
-      synchronized (this)
+      try
+      {
+        Thread.sleep(50);
+      } catch (InterruptedException x)
       {
-        try
-        {
-          wait(50);
-        } catch (InterruptedException e)
-        {
-        }
       }
-    }
+    } while (af.getViewport().getCalcManager().isWorking());
     af.changeColour_actionPerformed(JalviewColourScheme.PID.toString());
 
     SequenceI seq = viewport.getAlignment().getSequenceAt(0);
@@ -104,16 +101,14 @@ public class PIDColourSchemeTest
     /*
      * including gaps, A should be coloured for 50% consensus
      */
-    Color c = viewport
-            .getResidueShading().findColour('A', 0, seq);
+    Color c = viewport.getResidueShading().findColour('A', 0, seq);
     assertEquals(c, over40);
 
     /*
      * now choose to ignore gaps; colour should be for 67%
      */
     viewport.setIgnoreGapsConsensus(true, af.alignPanel);
-    c = viewport
-            .getResidueShading().findColour('A', 0, seq);
+    c = viewport.getResidueShading().findColour('A', 0, seq);
     assertEquals(c, over60);
   }
 }