Removed redundant checks associated with pack command

Author: vladvildanov

redis/connection.py

@@ -38,7 +38,6 @@
 from .utils import (
     CRYPTOGRAPHY_AVAILABLE,
     HIREDIS_AVAILABLE,
-    HIREDIS_PACK_AVAILABLE,
     SSL_AVAILABLE,
     compare_versions,
     ensure_string,
@@ -314,8 +313,6 @@ def __del__(self):
     def _construct_command_packer(self, packer):
         if packer is not None:
             return packer
-        elif HIREDIS_PACK_AVAILABLE:
-            return HiredisRespSerializer()
         else:
             return PythonRespSerializer(self._buffer_cutoff, self.encoder.encode)
 

redis/utils.py

@@ -12,7 +12,6 @@
         raise ImportError("hiredis package should be >= 3.0.0")
 except ImportError:
     HIREDIS_AVAILABLE = False
-    HIREDIS_PACK_AVAILABLE = False
 
 try:
     import ssl  # noqa

tests/test_encoding.py

@@ -1,7 +1,6 @@
 import pytest
 import redis
 from redis.connection import Connection
-from redis.utils import HIREDIS_PACK_AVAILABLE
 
 from .conftest import _get_client
 
@@ -75,22 +74,6 @@ def test_replace(self, request):
         assert r.get("a") == "foo\ufffd"
 
 
-@pytest.mark.skipif(
-    HIREDIS_PACK_AVAILABLE,
-    reason="Packing via hiredis does not preserve memoryviews",
-)
-class TestMemoryviewsAreNotPacked:
-    def test_memoryviews_are_not_packed(self):
-        c = Connection()
-        arg = memoryview(b"some_arg")
-        arg_list = ["SOME_COMMAND", arg]
-        cmd = c.pack_command(*arg_list)
-        assert cmd[1] is arg
-        cmds = c.pack_commands([arg_list, arg_list])
-        assert cmds[1] is arg
-        assert cmds[3] is arg
-
-
 class TestCommandsAreNotEncoded:
     @pytest.fixture()
     def r(self, request):