Use GitHub actions for continuous integration (CI)

Author: SimonFrings

.github/workflows/ci.yml

@@ -0,0 +1,23 @@
+name: CI
+
+on:
+  push:
+  pull_request:
+
+jobs:
+  PHPUnit:
+    name: PHPUnit (PHP ${{ matrix.php }})
+    runs-on: ubuntu-22.04
+    strategy:
+      matrix:
+        php:
+          - 5.4
+          - 5.3
+    steps:
+      - uses: actions/checkout@v4
+      - uses: shivammathur/setup-php@v2
+        with:
+          php-version: ${{ matrix.php }}
+          coverage: xdebug
+      - run: composer install
+      - run: vendor/bin/phpunit --coverage-text

.travis.yml

@@ -1,5 +1,8 @@
 # clue/redis-protocol [![Build Status](https://travis-ci.org/clue/php-redis-protocol.png?branch=master)](https://travis-ci.org/clue/php-redis-protocol)
 
+[![CI status](https://github.com/clue/php-redis-protocol/actions/workflows/ci.yml/badge.svg)](https://github.com/clue/php-redis-protocol/actions)
+[![installs on Packagist](https://img.shields.io/packagist/dt/clue/redis-protocol?color=blue&label=installs%20on%20Packagist)](https://packagist.org/packages/clue/redis-protocol)
+
 A streaming redis protocol parser and serializer written in PHP 
 
 This parser and serializer implementation allows you to parse redis protocol

README.md

@@ -13,7 +13,12 @@
     "require": {
         "php": ">=5.3"
     },
+    "require-dev": {
+        "phpunit/phpunit": "^4.8.36"
+    },
     "autoload": {
-        "psr-0": { "Clue\\Redis\\Protocol": "src" }
+        "psr-0": {
+            "Clue\\Redis\\Protocol": "src/"
+        }
     }
 }

composer.json

@@ -1,11 +1,10 @@
 <?xml version="1.0" encoding="UTF-8"?>
 
-<phpunit bootstrap="tests/bootstrap.php"
-         colors="true"
-         convertErrorsToExceptions="true"
-         convertNoticesToExceptions="true"
-         convertWarningsToExceptions="true"
->
+<!-- PHPUnit configuration file with old format for legacy PHPUnit -->
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/4.8/phpunit.xsd"
+         bootstrap="vendor/autoload.php"
+         colors="true">
     <testsuites>
         <testsuite name="Redis Protocol Test Suite">
             <directory>./tests/</directory>
@@ -16,4 +15,7 @@
             <directory>./src/</directory>
         </whitelist>
     </filter>
-</phpunit>
+    <php>
+        <ini name="error_reporting" value="-1" />
+    </php>
+</phpunit>

phpunit.xml.dist

@@ -2,7 +2,7 @@
 
 use Clue\Redis\Protocol\Serializer\RecursiveSerializer;
 
-abstract class AbstractModelTest extends TestCase
+abstract class AbstractModelTest extends \TestCase
 {
     protected $serializer;
 

tests/Model/AbstractModelTest.php

@@ -3,7 +3,7 @@
 use Clue\Redis\Protocol\Parser\ParserInterface;
 use Clue\Redis\Protocol\Parser\MessageBuffer;
 
-abstract class AbstractParserTest extends TestCase
+abstract class AbstractParserTest extends \TestCase
 {
     /**
      *

tests/Parser/AbstractParserTest.php

@@ -2,7 +2,7 @@
 
 use Clue\Redis\Protocol\Parser\ResponseParser;
 
-class RecursiveParserTest extends AbstractParserTest
+class ResponseParserTest extends AbstractParserTest
 {
     protected function createParser()
     {

tests/Parser/ResponseParserTest.php

@@ -1,11 +1,9 @@
 <?php
 
 use Clue\Redis\Protocol\Serializer\SerializerInterface;
-use Clue\Redis\Protocol\Model\Status;
-use Clue\Redis\Protocol\Model\ErrorReplyException;
 //use Exception;
 
-abstract class AbstractSerializerTest extends TestCase
+abstract class AbstractSerializerTest extends \TestCase
 {
     /**
      * @return SerializerInterface
@@ -85,10 +83,10 @@ public function testArrayMultiBulkReply()
 
     public function testErrorReply()
     {
-        $model = $this->serializer->createReplyModel(new Exception('ERR failure'));
+        $model = $this->serializer->createReplyModel(new \Exception('ERR failure'));
         $this->assertInstanceOf('Clue\Redis\Protocol\Model\ErrorReply', $model);
         $this->assertEquals('ERR failure', $model->getValueNative());
-        $this->assertEquals($model->getMessageSerialized($this->serializer), $this->serializer->getReplyMessage(new Exception('ERR failure')));
+        $this->assertEquals($model->getMessageSerialized($this->serializer), $this->serializer->getReplyMessage(new \Exception('ERR failure')));
     }
 
     /**