Flaky build due to a possible race condition #1449

Signed-off-by: christian.lutnik <[email protected]>

Author: chrfwow

src/main/java/dev/openfeature/sdk/Awaitable.java

@@ -1,7 +1,7 @@
 package dev.openfeature.sdk;
 
 /**
- * A class to help with synchronization
+ * A class to help with synchronization.
  */
 public class Awaitable {
     public static final Awaitable FINISHED = new Awaitable(true);
@@ -17,7 +17,7 @@ private Awaitable(boolean isDone) {
     /**
      * Lets the calling thread wait until some other thread calls {@link Awaitable#wakeup()}. If
      * {@link Awaitable#wakeup()} has been called before the current thread invokes this method, it will return
-     * immediately
+     * immediately.
      */
     public void await() {
         if (isDone) {

src/test/java/dev/openfeature/sdk/AwaitableTest.java

@@ -1,8 +1,10 @@
 package dev.openfeature.sdk;
 
+import static org.awaitility.Awaitility.await;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 import static org.junit.jupiter.api.Assertions.fail;
 
+import java.util.concurrent.TimeUnit;
 import java.util.concurrent.atomic.AtomicBoolean;
 import java.util.concurrent.atomic.AtomicInteger;
 import org.junit.jupiter.api.Test;
@@ -62,9 +64,7 @@ void callingWakeUpWakesUpAllWaitingThreads() throws InterruptedException {
             threads[i].start();
         }
 
-        while (isRunning.get() < numThreads) {
-            Thread.sleep(1);
-        }
+        await().atMost(1, TimeUnit.SECONDS).until(() -> isRunning.get() == numThreads);
 
         awaitable.wakeup();