Fixed - RBitSet.set(long bitIndex, boolean value) should return boolean. #1410

redisson/src/main/java/org/redisson/RedissonBitSet.java

@@ -205,8 +205,8 @@ public class RedissonBitSet extends RedissonExpirable implements RBitSet {
     }
 
     @Override
-    public void set(long bitIndex, boolean value) {
-        get(setAsync(bitIndex, value));
+    public boolean set(long bitIndex, boolean value) {
+        return get(setAsync(bitIndex, value));
     }
 
     @Override

redisson/src/main/java/org/redisson/api/RBitSet.java

@@ -214,9 +214,11 @@ public interface RBitSet extends RExpirable, RBitSetAsync {
      * 
      * @param bitIndex - index of bit
      * @param value true = 1, false = 0
-     * 
+     * @return <code>true</code> - if previous value was true,
+     * <code>false</code> - if previous value was false
+     *
     */
-    void set(long bitIndex, boolean value);
+    boolean set(long bitIndex, boolean value);
 
     byte[] toByteArray();
 

redisson/src/test/java/org/redisson/RedissonBitSetTest.java

@@ -114,8 +114,8 @@ public class RedissonBitSetTest extends BaseTest {
         assertThat(bitset.cardinality()).isZero();
         assertThat(bitset.size()).isZero();
 
-        bitset.set(10, true);
-        bitset.set(31, true);
+        assertThat(bitset.set(10, true)).isFalse();
+        assertThat(bitset.set(31, true)).isFalse();
         assertThat(bitset.get(0)).isFalse();
         assertThat(bitset.get(31)).isTrue();
         assertThat(bitset.get(10)).isTrue();