Merge pull request #93 from SimonFrings/tests

Clean up test suite, add .gitattributes to exclude dev files from exports and run tests on PHP 7.4 and simplify test matrix

Author: clue

.gitattributes

@@ -0,0 +1,6 @@
+/.gitattributes export-ignore
+/.gitignore export-ignore
+/.travis.yml export-ignore
+/examples/ export-ignore
+/phpunit.xml.dist export-ignore
+/tests/ export-ignore

.travis.yml

@@ -1,25 +1,23 @@
 language: php
 
-php:
-# - 5.3 # requires old distro, see below
-  - 5.4
-  - 5.5
-  - 5.6
-  - 7.0
-  - 7.1
-  - 7.2
-  - 7.3
-  - hhvm # ignore errors, see below
-
 # lock distro so future defaults will not break the build
 dist: trusty
 
 matrix:
   include:
     - php: 5.3
       dist: precise
+    - php: 5.4
+    - php: 5.5
+    - php: 5.6
+    - php: 7.0
+    - php: 7.1
+    - php: 7.2
+    - php: 7.3
+    - php: 7.4
+    - php: hhvm-3.18
   allow_failures:
-    - php: hhvm
+    - php: hhvm-3.18
 
 sudo: false
 

composer.json

@@ -23,6 +23,9 @@
     "autoload": {
         "psr-4": { "Clue\\React\\Stdio\\": "src/" }
     },
+    "autoload-dev": {
+        "psr-4": { "Clue\\Tests\\React\\Stdio\\": "tests/" }
+    },
     "require-dev": {
         "clue/arguments": "^2.0",
         "clue/commander": "^1.2",

phpunit.xml.dist

@@ -1,11 +1,6 @@
 <?xml version="1.0" encoding="UTF-8"?>
 
-<phpunit bootstrap="tests/bootstrap.php"
-         colors="true"
-         convertErrorsToExceptions="true"
-         convertNoticesToExceptions="true"
-         convertWarningsToExceptions="true"
->
+<phpunit bootstrap="vendor/autoload.php" colors="true">
     <testsuites>
         <testsuite name="Stdio React Test Suite">
             <directory>./tests/</directory>

tests/FunctionalExampleTest.php

@@ -1,5 +1,7 @@
 <?php
 
+namespace Clue\Tests\React\Stdio;
+
 class FunctionalExampleTest extends TestCase
 {
     public function testPeriodicExampleWithPipedInputEndsBecauseInputEnds()

tests/ReadlineTest.php

@@ -1,5 +1,7 @@
 <?php
 
+namespace Clue\Tests\React\Stdio;
+
 use Clue\React\Stdio\Readline;
 use React\Stream\ThroughStream;
 use Evenement\EventEmitter;

tests/StdioTest.php

@@ -1,5 +1,7 @@
 <?php
 
+namespace Clue\Tests\React\Stdio;
+
 use Clue\React\Stdio\Stdio;
 use Clue\React\Stdio\Readline;
 use React\EventLoop\Factory;

tests/bootstrap.php tests/TestCase.phptests/bootstrap.php renamed to tests/TestCase.php

@@ -1,5 +1,7 @@
 <?php
 
+namespace Clue\Tests\React\Stdio;
+
 use PHPUnit\Framework\TestCase as BaseTestCase;
 
 require_once __DIR__ . '/../vendor/autoload.php';
@@ -44,7 +46,7 @@ protected function expectCallableOnceWith($value)
      */
     protected function createCallableMock()
     {
-        return $this->getMockBuilder('CallableStub')->getMock();
+        return $this->getMockBuilder('stdClass')->setMethods(array('__invoke'))->getMock();
     }
 
     protected function expectPromiseResolve($promise)
@@ -65,10 +67,3 @@ protected function expectPromiseReject($promise)
         return $promise;
     }
 }
-
-class CallableStub
-{
-    public function __invoke()
-    {
-    }
-}