Remove redundant check for 0 and add addPermit test case

Signed-off-by: maddie-boisvert <maddie.boisvert@gmail.com>
pull/4719/head
maddie-boisvert 2 years ago
parent 932c2df637
commit c169b30d01

@ -719,9 +719,6 @@ public class RedissonPermitExpirableSemaphore extends RedissonExpirable implemen
"return 0;" +
"end;" +
"local delta = tonumber(ARGV[1]) - maximum; " +
"if (delta == 0) then " +
"return 0;" +
"end;" +
"redis.call('incrby', KEYS[1], delta); " +
"redis.call('publish', KEYS[2], ARGV[1]); " +
"return delta;",

@ -247,6 +247,8 @@ public class RedissonPermitExpirableSemaphoreTest extends BaseConcurrentTest {
assertThat(s.availablePermits()).isEqualTo(15);
s.addPermits(-10);
assertThat(s.availablePermits()).isEqualTo(5);
s.addPermits(-10);
assertThat(s.availablePermits()).isEqualTo(-5);
}
@Test

Loading…
Cancel
Save