test: adjust some assert logic to reduce flakiness (#127)

Signed-off-by: Nicklas Lundin <[email protected]>
Co-authored-by: Michael Beemer <[email protected]>

Author: nicklasl

android/src/test/java/dev/openfeature/sdk/DeveloperExperienceTests.kt

@@ -259,7 +259,9 @@ class DeveloperExperienceTests {
             assertEquals(OpenFeatureStatus.NotReady, OpenFeatureAPI.getStatus())
         }
         testScheduler.advanceTimeBy(1)
-        assertEquals(OpenFeatureStatus.NotReady, OpenFeatureAPI.getStatus())
+        waitAssert {
+            assertEquals(OpenFeatureStatus.NotReady, OpenFeatureAPI.getStatus())
+        }
         testScheduler.advanceTimeBy(healDelayMillis)
         waitAssert {
             assertEquals(OpenFeatureStatus.Ready, OpenFeatureAPI.getStatus())

android/src/test/java/dev/openfeature/sdk/StatusTests.kt

@@ -83,7 +83,6 @@ class StatusTests {
         }
         job.cancelAndJoin()
 
-        assertEquals(7, statuses.size)
         assertEquals(
             listOf(
                 OpenFeatureStatus.NotReady,
@@ -100,13 +99,15 @@ class StatusTests {
 }
 
 @OptIn(ExperimentalCoroutinesApi::class)
-suspend fun TestScope.waitAssert(function: () -> Unit) {
-    while (true) {
+suspend fun TestScope.waitAssert(timeoutMs: Long = 5000, function: () -> Unit) {
+    var timeWaited = 0L
+    while (timeWaited < timeoutMs) {
         try {
             function()
             return
         } catch (e: Throwable) {
-            delay(100)
+            delay(10)
+            timeWaited += 10
             advanceUntilIdle()
         }
     }