Skip to content

Commit 91ca04a

Browse files
committed
Replace awaitility with CountDownLatch to try and fix the test on CI
1 parent fc52a26 commit 91ca04a

File tree

1 file changed

+8
-6
lines changed

1 file changed

+8
-6
lines changed

sentry/src/test/java/io/sentry/transport/RateLimiterTest.kt

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,8 @@ import org.mockito.kotlin.verifyNoMoreInteractions
3939
import org.mockito.kotlin.whenever
4040
import java.io.File
4141
import java.util.UUID
42+
import java.util.concurrent.CountDownLatch
43+
import java.util.concurrent.TimeUnit
4244
import java.util.concurrent.atomic.AtomicBoolean
4345
import kotlin.test.Test
4446
import kotlin.test.assertEquals
@@ -363,18 +365,18 @@ class RateLimiterTest {
363365
val rateLimiter = fixture.getSUT()
364366
whenever(fixture.currentDateProvider.currentTimeMillis).thenReturn(0, 1, 2001)
365367

366-
val applied = AtomicBoolean(true)
368+
val applied = CountDownLatch(1)
369+
var activeForReplay = false
367370
rateLimiter.addRateLimitObserver {
368-
applied.set(rateLimiter.isActiveForCategory(Replay))
371+
applied.countDown()
372+
activeForReplay = rateLimiter.isActiveForCategory(Replay)
369373
}
370374

371375
rateLimiter.updateRetryAfterLimits("1:replay:key", null, 1)
372376
rateLimiter.close()
373377

374378
// If rate limit didn't already change, wait for 1.5s to ensure the timer has run after 1s
375-
if (!applied.get()) {
376-
await.untilTrue(applied)
377-
}
378-
assertTrue(applied.get())
379+
applied.await(2, TimeUnit.SECONDS)
380+
assertTrue(activeForReplay)
379381
}
380382
}

0 commit comments

Comments
 (0)