Adding RESP3 tests support

.github/workflows/integration.yaml

@@ -16,6 +16,10 @@ on:
   schedule:
     - cron: '0 1 * * *' # nightly build
 
+concurrency:
+  group: ${{ github.event.pull_request.number || github.ref }}
+  cancel-in-progress: true
+
 permissions:
   contents: read  #  to fetch code (actions/checkout)
 
@@ -48,7 +52,7 @@ jobs:
 
    run-tests:
      runs-on: ubuntu-latest
-     timeout-minutes: 30
+     timeout-minutes: 15
      strategy:
        max-parallel: 15
        fail-fast: false
@@ -69,30 +73,63 @@ jobs:
          run: |
            pip install -U setuptools wheel
            pip install -r dev_requirements.txt
+           invoke devenv
            tox -e ${{matrix.test-type}}-${{matrix.connection-type}}
+
        - uses: actions/upload-artifact@v2
          if: success() || failure()
          with:
-           name: pytest-results-${{matrix.test-type}}
+           name: pytest-results-${{matrix.test-type}}-${{matrix.connection-type}}-${{matrix.python-version}}
            path: '${{matrix.test-type}}*results.xml'
+
        - name: Upload codecov coverage
          uses: codecov/codecov-action@v3
+         if: ${{matrix.python-version == '3.11'}}
          with:
            fail_ci_if_error: false
-      #  - name: View Test Results
-      #    uses: dorny/test-reporter@v1
-      #    if: success() || failure()
-      #    with:
-      #      name: Test Results ${{matrix.python-version}} ${{matrix.test-type}}-${{matrix.connection-type}}
-      #      path: '${{matrix.test-type}}*results.xml'
-      #      reporter: java-junit
-      #      list-suites: failed
-      #      list-tests: failed
-      #      max-annotations: 10
+
+       - name: View Test Results
+         uses: dorny/test-reporter@v1
+         if: success() || failure()
+         continue-on-error: true
+         with:
+           name: Test Results ${{matrix.python-version}} ${{matrix.test-type}}-${{matrix.connection-type}}
+           path: '*.xml'
+           reporter: java-junit
+           list-suites: all
+           list-tests: all
+           max-annotations: 10
+           fail-on-error: 'false'
+
+   resp3_tests:
+    name: RESP3 Protocol Testing
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        python-version: ['3.7', '3.11']
+        test-type: ['standalone', 'cluster']
+        connection-type: ['hiredis', 'plain']
+        protocol: ['3']
+    env:
+       ACTIONS_ALLOW_UNSECURE_COMMANDS: true
+     name: Python ${{ matrix.python-version }} ${{matrix.test-type}}-${{matrix.connection-type}} tests
+     steps:
+       - uses: actions/checkout@v3
+       - uses: actions/setup-python@v4
+         with:
+           python-version: ${{ matrix.python-version }}
+           cache: 'pip'
+       - name: run tests
+         run: |
+           pip install -U setuptools wheel
+           pip install -r dev_requirements.txt
+           invoke devenv
+           tox -e ${{matrix.test-type}}-${{matrix.connection-type}}-resp${{matrix.protocol}}
 
    build_and_test_package:
     name: Validate building and installing the package
     runs-on: ubuntu-latest
+    needs: [run-tests]
     strategy:
       matrix:
         extension: ['tar.gz', 'whl']
@@ -107,6 +144,7 @@ jobs:
 
    install_package_from_commit:
     name: Install package from commit hash
+    needs: [build_and_test_package]
     runs-on: ubuntu-latest
     strategy:
       fail-fast: false

CONTRIBUTING.md

@@ -38,8 +38,9 @@ Here's how to get started with your code contribution:
         a.  python -m venv .venv
         b.  source .venv/bin/activate
         c.  pip install -r dev_requirements.txt
+        c.  pip install -r requirements.txt
 
-4.  If you need a development environment, run `invoke devenv`
+4.  If you need a development environment, run `invoke devenv`. Note: this relies on docker-compose to build environments, and assumes that you have a version supporting [docker profiles](https://docs.docker.com/compose/profiles/).
 5.  While developing, make sure the tests pass by running `invoke tests`
 6.  If you like the change and think the project could use it, send a
     pull request
@@ -59,7 +60,6 @@ can execute docker and its various commands.
 -   Three sentinel Redis nodes
 -   A redis cluster
 -   An stunnel docker, fronting the master Redis node
--   A Redis node, running unstable - the latest redis
 
 The replica node, is a replica of the master node, using the
 [leader-follower replication](https://redis.io/topics/replication)

dev_requirements.txt

@@ -9,7 +9,6 @@ pytest==7.2.0
 pytest-timeout==2.0.1
 pytest-asyncio>=0.20.2
 tox==3.27.1
-tox-docker==3.1.0
 invoke==1.7.3
 pytest-cov>=4.0.0
 vulture>=2.3.0

docker-compose.yml

@@ -0,0 +1,103 @@
+---
+
+version: "3.8"
+
+services:
+
+  redis:
+    image: redis/redis-stack-server:edge
+    ports:
+      - 6379:6379
+    environment:
+      - "REDIS_ARGS=--enable-debug-command yes --enable-module-command yes"
+    profiles:
+      - standalone
+      - sentinel
+      - replica
+      - all
+
+  replica:
+    image: redis/redis-stack-server:edge
+    depends_on:
+      - redis
+    environment:
+      - "REDIS_ARGS=--replicaof redis 6379"
+    ports:
+      - 6380:6379
+    profiles:
+      - replica
+      - all
+
+  cluster:
+    build:
+      context: .
+      dockerfile: dockers/Dockerfile.cluster
+    ports:
+      - 16379:16379
+      - 16380:16380
+      - 16381:16381
+      - 16382:16382
+      - 16383:16383
+      - 16384:16384
+    volumes:
+      - "./dockers/cluster.redis.conf:/redis.conf:ro"
+    profiles:
+      - cluster
+      - all
+
+  stunnel:
+    image: redisfab/stunnel:latest
+    depends_on:
+      - redis
+    ports:
+      - 6666:6666
+    profiles:
+      - all
+      - standalone
+      - ssl
+    volumes:
+      - "./dockers/stunnel/conf:/etc/stunnel/conf.d:ro"
+      - "./dockers/stunnel/keys:/etc/stunnel/keys:ro"
+
+  sentinel:
+    image: redis/redis-stack-server:edge
+    depends_on:
+      - redis
+    environment:
+      - "REDIS_ARGS=--port 26379"
+    entrypoint: "/opt/redis-stack/bin/redis-sentinel /redis.conf --port 26379"
+    ports:
+      - 26379:26379
+    volumes:
+      - "./dockers/sentinel.conf:/redis.conf"
+    profiles:
+      - sentinel
+      - all
+
+  sentinel2:
+    image: redis/redis-stack-server:edge
+    depends_on:
+      - redis
+    environment:
+      - "REDIS_ARGS=--port 26380"
+    entrypoint: "/opt/redis-stack/bin/redis-sentinel /redis.conf --port 26380"
+    ports:
+      - 26380:26380
+    volumes:
+      - "./dockers/sentinel.conf:/redis.conf"
+    profiles:
+      - sentinel
+      - all
+
+  sentinel3:
+    image: redis/redis-stack-server:edge
+    depends_on:
+      - redis
+    entrypoint: "/opt/redis-stack/bin/redis-sentinel /redis.conf --port 26381"
+    ports:
+      - 26381:26381
+    volumes:
+      - "./dockers/sentinel.conf:/redis.conf"
+    profiles:
+      - sentinel
+      - all