# HG changeset patch
# User Jaroslav Tulach <jtulach@netbeans.org>
# Date 1446393582 -3600
# Node ID 665b10c62f3d777f61622f660c297080857ad22c
# Parent  8a88dc9baa8705e3dcd69dd8449bb468a05c96cd
Make the test more robust by counting down the latch only when the then to be tested property is set

diff -r 8a88dc9baa87 -r 665b10c62f3d boot-fx/src/test/java/net/java/html/boot/fx/FXBrowsersTest.java
--- a/boot-fx/src/test/java/net/java/html/boot/fx/FXBrowsersTest.java	Sat Oct 31 18:06:19 2015 +0100
+++ b/boot-fx/src/test/java/net/java/html/boot/fx/FXBrowsersTest.java	Sun Nov 01 16:59:42 2015 +0100
@@ -60,7 +60,8 @@
  * @author Jaroslav Tulach
  */
 public class FXBrowsersTest {
-    
+    private static CountDownLatch PROPERTY_SET;
+
     public FXBrowsersTest() {
     }
     
@@ -85,7 +86,6 @@
     @Test
     public void behaviorOfTwoWebViewsAtOnce() throws Throwable {
         class R implements Runnable {
-            CountDownLatch DONE = new CountDownLatch(1);
             Throwable t;
 
             @Override
@@ -94,8 +94,6 @@
                     doTest();
                 } catch (Throwable ex) {
                     t = ex;
-                } finally {
-                    DONE.countDown();
                 }
             }
             
@@ -103,22 +101,12 @@
                 URL u = FXBrowsersTest.class.getResource("/org/netbeans/html/boot/fx/empty.html");
                 assertNotNull(u, "URL found");
                 FXBrowsers.load(App.getV1(), u, OnPages.class, "first");
-                
             }
         }
         R run = new R();
+        PROPERTY_SET = new CountDownLatch(2);
         Platform.runLater(run);
-        run.DONE.await();
-        for (int i = 0; i < 100; i++) {
-            if (run.t != null) {
-                throw run.t;
-            }
-            if (System.getProperty("finalSecond") == null) {
-                Thread.sleep(100);
-            }
-        }
-        
-        
+        PROPERTY_SET.await();
         
         assertEquals(Integer.getInteger("finalFirst"), Integer.valueOf(3), "Three times in view one");
         assertEquals(Integer.getInteger("finalSecond"), Integer.valueOf(2), "Two times in view one");
@@ -161,6 +149,7 @@
             
             assertEquals(increment(), 2, "Now it is two and not influenced by second view");
             System.setProperty("finalFirst", "" + increment());
+            PROPERTY_SET.countDown();
         }
         
         public static void second(String... args) {
@@ -175,6 +164,7 @@
             
             assertEquals(increment(), 1, "Counting starts from zero");
             System.setProperty("finalSecond", "" + increment());
+            PROPERTY_SET.countDown();
         }
         
         @JavaScriptBody(args = {}, body = "return window;")