JAL-4513 add a SwingUtilities.invokeAndWait() to see what happens
authorBen Soares <b.soares@dundee.ac.uk>
Tue, 18 Jul 2023 13:19:15 +0000 (14:19 +0100)
committerBen Soares <b.soares@dundee.ac.uk>
Tue, 18 Jul 2023 13:19:15 +0000 (14:19 +0100)
test/jalview/io/cache/JvCacheableInputBoxTest.java

index 9946f4c..54152a6 100644 (file)
  */
 package jalview.io.cache;
 
+import java.lang.reflect.InvocationTargetException;
 import java.util.LinkedHashSet;
 
+import javax.swing.SwingUtilities;
+
 import org.junit.Assert;
 import org.testng.annotations.BeforeClass;
 import org.testng.annotations.Test;
@@ -54,11 +57,19 @@ public class JvCacheableInputBoxTest
 
     try
     {
-      // This delay is essential to prevent the
-      // assertion below from executing before
-      // swing thread finishes updating the combo-box
-      Thread.sleep(350);
-    } catch (InterruptedException e)
+      SwingUtilities.invokeAndWait(() -> {
+        try
+        {
+          // This delay is essential to prevent the
+          // assertion below from executing before
+          // swing thread finishes updating the combo-box
+          Thread.sleep(10);
+        } catch (InterruptedException e)
+        {
+          e.printStackTrace();
+        }
+      });
+    } catch (InvocationTargetException | InterruptedException e)
     {
       e.printStackTrace();
     }
@@ -73,12 +84,21 @@ public class JvCacheableInputBoxTest
     cacheBox.addItem(testInput);
     cacheBox.setSelectedItem(testInput);
     cacheBox.updateCache();
+
     try
     {
-      // This delay is to let
-      // cacheBox.updateCache() finish updating the cache
-      Thread.sleep(350);
-    } catch (InterruptedException e)
+      SwingUtilities.invokeAndWait(() -> {
+        try
+        {
+          // This delay is to let
+          // cacheBox.updateCache() finish updating the cache
+          Thread.sleep(10);
+        } catch (InterruptedException e)
+        {
+          e.printStackTrace();
+        }
+      });
+    } catch (InvocationTargetException | InterruptedException e)
     {
       e.printStackTrace();
     }