Pinned redis py to 4.3.5 (#349)

* tox tests longer than 10 minutes are likely hung so let github bail
* pin redis 4.3.5
* let the async_timeout occur
* Added change note.

---------

Co-authored-by: Carlton Gibson <carlton.gibson@noumenal.es>

.github/workflows/tests.yml

@@ -10,6 +10,7 @@ jobs:
   tests:
     name: Python ${{ matrix.python-version }}
     runs-on: ubuntu-latest
+    timeout-minutes: 10
     strategy:
       fail-fast: false
       matrix:

CHANGELOG.txt

@@ -1,3 +1,10 @@
+UNRELEASED
+----------
+
+* Pinned redis-py version to 4.3.5 in order to avoid difficult to diagnose
+  issues occurring with redis-py 4.4+ (to 4.5.2 currently).
+
+
 4.0.0 (2022-10-07)
 ------------------
 

setup.py

@@ -30,7 +30,7 @@
     include_package_data=True,
     python_requires=">=3.7",
     install_requires=[
-        "redis>=4.2.0",
+        "redis==4.3.5",
         "msgpack~=1.0",
         "asgiref>=3.2.10,<4",
         "channels",

tests/test_pubsub.py

@@ -19,7 +19,8 @@ async def channel_layer():
     """
     channel_layer = RedisPubSubChannelLayer(hosts=TEST_HOSTS)
     yield channel_layer
-    await channel_layer.flush()
+    async with async_timeout.timeout(1):
+        await channel_layer.flush()
 
 
 @pytest.fixture()

tests/test_pubsub_sentinel.py

@@ -25,7 +25,8 @@ async def channel_layer():
     """
     channel_layer = RedisPubSubChannelLayer(hosts=TEST_HOSTS)
     yield channel_layer
-    await channel_layer.flush()
+    async with async_timeout.timeout(1):
+        await channel_layer.flush()
 
 
 @pytest.mark.asyncio