Use GitHub actions for continuous integration (CI)

.github/workflows/ci.yml

@@ -0,0 +1,24 @@
+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
+          ini-file: development
+      - run: composer install
+      - run: vendor/bin/phpunit --coverage-text

README.md

@@ -1,5 +1,7 @@
 # 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)
+
 A streaming redis protocol parser and serializer written in PHP 
 
 This parser and serializer implementation allows you to parse redis protocol

composer.json

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

phpunit.xml.dist

@@ -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>

tests/FactoryTest.php

@@ -1,5 +1,7 @@
 <?php
 
+namespace Clue\Tests\Redis\Protocol;
+
 use Clue\Redis\Protocol\Factory;
 
 class FactoryTest extends TestCase

tests/Model/AbstractModelTest.php

@@ -1,6 +1,9 @@
 <?php
 
+namespace Clue\Tests\Redis\Protocol\Model;
+
 use Clue\Redis\Protocol\Serializer\RecursiveSerializer;
+use Clue\Tests\Redis\Protocol\TestCase;
 
 abstract class AbstractModelTest extends TestCase
 {

tests/Model/BulkReplyTest.php

@@ -1,5 +1,7 @@
 <?php
 
+namespace Clue\Tests\Redis\Protocol\Model;
+
 use Clue\Redis\Protocol\Model\BulkReply;
 
 class BulkReplyTest extends AbstractModelTest

tests/Model/ErrorReplyTest.php

@@ -1,5 +1,7 @@
 <?php
 
+namespace Clue\Tests\Redis\Protocol\Model;
+
 use Clue\Redis\Protocol\Model\ErrorReply;
 
 class ErrorReplyTest extends AbstractModelTest

tests/Model/IntegerReplyTest.php

@@ -1,5 +1,7 @@
 <?php
 
+namespace Clue\Tests\Redis\Protocol\Model;
+
 use Clue\Redis\Protocol\Model\IntegerReply;
 
 class IntegerReplyTest extends AbstractModelTest

tests/Model/MultiBulkReplyTest.php

@@ -1,5 +1,7 @@
 <?php
 
+namespace Clue\Tests\Redis\Protocol\Model;
+
 use Clue\Redis\Protocol\Model\MultiBulkReply;
 use Clue\Redis\Protocol\Model\BulkReply;
 use Clue\Redis\Protocol\Model\IntegerReply;

tests/Model/RequestTest.php

@@ -1,5 +1,7 @@
 <?php
 
+namespace Clue\Tests\Redis\Protocol\Model;
+
 use Clue\Redis\Protocol\Model\Request;
 
 class RequestTest extends AbstractModelTest

tests/Model/StatusReplyTest.php

@@ -1,5 +1,7 @@
 <?php
 
+namespace Clue\Tests\Redis\Protocol\Model;
+
 use Clue\Redis\Protocol\Model\StatusReply;
 
 class StatusReplyTest extends AbstractModelTest

tests/Parser/AbstractParserTest.php

@@ -1,7 +1,10 @@
 <?php
 
+namespace Clue\Tests\Redis\Protocol\Parser;
+
 use Clue\Redis\Protocol\Parser\ParserInterface;
 use Clue\Redis\Protocol\Parser\MessageBuffer;
+use Clue\Tests\Redis\Protocol\TestCase;
 
 abstract class AbstractParserTest extends TestCase
 {

tests/Parser/RequestParserTest.php

@@ -1,5 +1,7 @@
 <?php
 
+namespace Clue\Tests\Redis\Protocol\Parser;
+
 use Clue\Redis\Protocol\Parser\RequestParser;
 
 class RequestParserTest extends AbstractParserTest

tests/Parser/ResponseParserTest.php

@@ -1,5 +1,7 @@
 <?php
 
+namespace Clue\Tests\Redis\Protocol\Parser;
+
 use Clue\Redis\Protocol\Parser\ResponseParser;
 
 class RecursiveParserTest extends AbstractParserTest

tests/Serializer/AbstractSerializerTest.php

@@ -1,8 +1,9 @@
 <?php
 
+namespace Clue\Tests\Redis\Protocol\Serializer;
+
 use Clue\Redis\Protocol\Serializer\SerializerInterface;
-use Clue\Redis\Protocol\Model\Status;
-use Clue\Redis\Protocol\Model\ErrorReplyException;
+use Clue\Tests\Redis\Protocol\TestCase;
 //use Exception;
 
 abstract class AbstractSerializerTest extends TestCase
@@ -85,10 +86,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')));
     }
 
     /**

tests/Serializer/RecursiveSerializerTest.php

@@ -1,5 +1,7 @@
 <?php
 
+namespace Clue\Tests\Redis\Protocol\Serializer;
+
 use Clue\Redis\Protocol\Serializer\RecursiveSerializer;
 
 class RecursiveSerializerTest extends AbstractSerializerTest

tests/TestCase.php

@@ -0,0 +1,7 @@
+<?php
+
+namespace Clue\Tests\Redis\Protocol;
+
+class TestCase extends \PHPUnit_Framework_TestCase
+{
+}