diff --git a/redisson/src/test/java/org/redisson/RedissonSpinLockTest.java b/redisson/src/test/java/org/redisson/RedissonSpinLockTest.java index 2c238bea8..7447655df 100644 --- a/redisson/src/test/java/org/redisson/RedissonSpinLockTest.java +++ b/redisson/src/test/java/org/redisson/RedissonSpinLockTest.java @@ -85,7 +85,7 @@ public class RedissonSpinLockTest extends BaseConcurrentTest { RLock lock = redisson.getSpinLock("lock"); - Awaitility.await().between(Duration.ofMillis(3000), Duration.ofMillis(3200)).untilAsserted(() -> { + Awaitility.await().between(Duration.ofMillis(3000), Duration.ofMillis(3500)).untilAsserted(() -> { lock.tryLock(3, TimeUnit.SECONDS); }); } diff --git a/redisson/src/test/java/org/redisson/RedissonTopicTest.java b/redisson/src/test/java/org/redisson/RedissonTopicTest.java index 03e64feaa..911402c7a 100644 --- a/redisson/src/test/java/org/redisson/RedissonTopicTest.java +++ b/redisson/src/test/java/org/redisson/RedissonTopicTest.java @@ -836,19 +836,20 @@ public class RedissonTopicTest { AtomicBoolean exceptionDetected = new AtomicBoolean(false); + Deque status = new ConcurrentLinkedDeque<>(); Runnable rLockPayload = () -> { try { Integer randomLock = ThreadLocalRandom.current().nextInt(100); RLock lock = redisson.getLock(randomLock.toString()); - try { - lock.lock(10, TimeUnit.SECONDS); - } finally { - if (lock != null && lock.isHeldByCurrentThread()) { - lock.unlock(); - } - } + lock.lock(10, TimeUnit.SECONDS); + lock.unlock(); + status.add("ok"); } catch (Exception e) { + status.add("failed"); + if (e.getCause().getMessage().contains("slaves were synced")) { + return; + } e.printStackTrace(); exceptionDetected.set(true); } @@ -883,8 +884,9 @@ public class RedissonTopicTest { Thread.sleep(java.time.Duration.ofSeconds(10).toMillis()); assertThat(exceptionDetected.get()).isFalse(); + assertThat(status.peekLast()).isEqualTo("ok"); - executor1.shutdownNow(); + executor1.shutdown(); redisson.shutdown(); sentinel1.stop();