Merge pull request #30 from intaro/redis-version

Simple test for Redis compability with base class

.github/workflows/ci.yml

@@ -43,6 +43,7 @@ jobs:
           with:
             tools: flex
             php-version: "${{ matrix.php-version }}"
+            extensions: redis-5.3.7
             coverage: "${{ matrix.coverage }}"
 
       -   name: "Install dependencies"

Dockerfile

@@ -2,7 +2,7 @@ ARG PHP_IMAGE_TAG
 FROM php:${PHP_IMAGE_TAG}-cli-alpine
 
 RUN apk add autoconf gcc g++ make
-RUN pecl install redis && docker-php-ext-enable redis
+RUN pecl install redis-5.3.7 && docker-php-ext-enable redis
 
 COPY --from=composer:latest /usr/bin/composer /usr/bin/composer
 

tests/Cache/RedisTest.php

@@ -0,0 +1,21 @@
+<?php
+
+namespace Intaro\PinbaBundle\Tests\Cache;
+
+use Intaro\PinbaBundle\Cache\Redis;
+use PHPUnit\Framework\TestCase;
+
+class RedisTest extends TestCase
+{
+    /**
+     * @doesNotPerformAssertions
+     *
+     * Check that Pinba Redis class is compatible with base Redis class
+     *
+     * TODO Redis >= 6.0.0 requires typed properties in methods
+     */
+    public function testInit(): void
+    {
+        $redis = new Redis();
+    }
+}