Disable scananddump when hiredis

Author: chayim

redis/commands/bf/commands.py

@@ -1,3 +1,7 @@
+from redis.client import NEVER_DECODE
+from redis.exceptions import ModuleError
+from redis.utils import HIREDIS_AVAILABLE
+
 BF_RESERVE = "BF.RESERVE"
 BF_ADD = "BF.ADD"
 BF_MADD = "BF.MADD"
@@ -133,7 +137,8 @@ def scandump(self, key, iter):
         This command will return successive (iter, data) pairs until (0, NULL) to indicate completion.
         For more information see `BF.SCANDUMP <https://oss.redis.com/redisbloom/master/Bloom_Commands/#bfscandump>`_.
         """  # noqa
-        from redis.client import NEVER_DECODE
+        if HIREDIS_AVAILABLE:
+            raise ModuleError("This command cannot be used when hiredis is available.")
 
         params = [key, iter]
         options = {}

tests/test_bloom.py

@@ -1,7 +1,8 @@
 import pytest
 
 import redis.commands.bf
-from redis.exceptions import RedisError
+from redis.exceptions import ModuleError, RedisError
+from redis.utils import HIREDIS_AVAILABLE
 
 
 def intlist(obj):
@@ -85,6 +86,11 @@ def do_verify():
 
     do_verify()
     cmds = []
+    if HIREDIS_AVAILABLE:
+        with pytest.raises(ModuleError):
+            cur = client.bf().scandump("myBloom", 0)
+        return
+
     cur = client.bf().scandump("myBloom", 0)
     first = cur[0]
     cmds.append(cur)