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

@@ -19,6 +19,7 @@ private Awaitable(boolean isDone) {
      * {@link Awaitable#wakeup()} has been called before the current thread invokes this method, it will return
      * immediately.
      */
+    @SuppressWarnings("java:S2142")
     public void await() {
         if (isDone) {
             return;
@@ -28,7 +29,7 @@ public void await() {
                 try {
                     this.wait();
                 } catch (InterruptedException ignored) {
-                    Thread.currentThread().interrupt();
+                    // ignored, do not propagate the interrupted state
                 }
             }
         }

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

@@ -86,6 +86,8 @@ public Awaitable emit(final ProviderEvent event, final ProviderEventDetails deta
 
         final var awaitable = new Awaitable();
 
+        // These calls need to be executed on a different thread to prevent deadlocks when the provider initialization
+        // relies on a ready event to be emitted
         emitterExecutor.submit(() -> {
             try (var ignored = OpenFeatureAPI.lock.readLockAutoCloseable()) {
                 if (localEventProviderListener != null) {

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

@@ -10,7 +10,7 @@
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.Timeout;
 
-@Timeout(2)
+@Timeout(value=5, threadMode = Timeout.ThreadMode.SEPARATE_THREAD)
 class AwaitableTest {
     @Test
     void waitingForFinishedIsANoOp() {