fix linters and failed tests

Author: tzongw

redis/connection.py

@@ -1539,7 +1539,7 @@ def read_response(
 
             # Cache only responses that still valid
             # and wasn't invalidated by another connection in meantime.
-            if cache_entry is self._current_command_cache_entry:
+            if cache_entry is not None:
                 cache_entry.status = CacheEntryStatus.VALID
                 cache_entry.cache_value = response
                 self._cache.set(cache_entry)

tests/test_connection.py

@@ -482,7 +482,6 @@ def test_read_response_returns_cached_reply(self, mock_cache, mock_connection):
 
         mock_cache.is_cachable.return_value = True
         mock_cache.get.side_effect = [
-            None,
             None,
             CacheEntry(
                 cache_key=CacheKey(
@@ -570,11 +569,6 @@ def test_read_response_returns_cached_reply(self, mock_cache, mock_connection):
                         command="GET", redis_keys=("foo",), redis_args=("GET", "foo")
                     )
                 ),
-                call(
-                    CacheKey(
-                        command="GET", redis_keys=("foo",), redis_args=("GET", "foo")
-                    )
-                ),
             ]
         )
 
@@ -730,4 +724,4 @@ def test_cache_entry_fill_between_send_and_read(
 
         assert proxy_connection.read_response() == b"bar2"
         mock_connection.send_command.assert_called_once()
-        mock_connection.read_response.assert_called_once()
+        mock_connection.read_response.assert_called_once()