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

.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 new 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

@@ -13,6 +13,9 @@
     "autoload": {
         "psr-4": { "Clue\\React\\Soap\\": "src/" }
     },
+    "autoload-dev": {
+        "psr-4": { "Clue\\Tests\\React\\Soap\\": "tests/" }
+    },
     "require": {
         "php": ">=5.3",
         "clue/buzz-react": "^2.5",
@@ -22,6 +25,6 @@
     },
     "require-dev": {
         "clue/block-react": "^1.0",
-        "phpunit/phpunit": "^6.4 || ^5.7 || ^4.8.35"
+        "phpunit/phpunit": "^7.0 || ^6.4 || ^5.7 || ^4.8.35"
     }
 }

phpunit.xml.dist

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

tests/ClientTest.php

@@ -1,12 +1,18 @@
 <?php
 
+namespace Clue\Tests\React\Soap;
+
 use Clue\React\Soap\Client;
 use PHPUnit\Framework\TestCase;
 use React\Promise\Promise;
 use Psr\Http\Message\RequestInterface;
 
 class ClientTest extends TestCase
 {
+
+    /**
+     * @expectedException SoapFault
+     */
     public function testConstructorThrowsWhenUrlIsInvalid()
     {
         if (extension_loaded('xdebug')) {

tests/FunctionalTest.php

@@ -1,5 +1,7 @@
 <?php
 
+namespace Clue\Tests\React\Soap;
+
 use Clue\React\Block;
 use Clue\React\Buzz\Browser;
 use Clue\React\Soap\Client;
@@ -16,7 +18,7 @@ class BankResponse
 class FunctionalTest extends TestCase
 {
     /**
-     * @var React\EventLoop\LoopInterface
+     * @var \React\EventLoop\LoopInterface
      */
     private $loop;
 
@@ -34,7 +36,7 @@ public static function setUpBeforeClass()
 
     public function setUp()
     {
-        $this->loop = React\EventLoop\Factory::create();
+        $this->loop = \React\EventLoop\Factory::create();
         $this->client = new Client(new Browser($this->loop), self::$wsdl);
     }
 
@@ -58,7 +60,7 @@ public function testBlzServiceWithClassmapReturnsExpectedType()
     {
         $this->client = new Client(new Browser($this->loop), self::$wsdl, array(
             'classmap' => array(
-                'getBankResponseType' => 'BankResponse'
+                'getBankResponseType' => 'Clue\Tests\React\Soap\BankResponse'
             )
         ));
 
@@ -71,7 +73,7 @@ public function testBlzServiceWithClassmapReturnsExpectedType()
 
         $result = Block\await($promise, $this->loop);
 
-        $this->assertInstanceOf('BankResponse', $result);
+        $this->assertInstanceOf('Clue\Tests\React\Soap\BankResponse', $result);
         $this->assertTrue(isset($result->details));
         $this->assertTrue(isset($result->details->bic));
     }
@@ -107,7 +109,7 @@ public function testBlzServiceNonWsdlModeReturnedWithoutOuterResultStructure()
 
         // try encoding the "blz" parameter with the correct namespace (see uri)
         // $promise = $api->getBank(new SoapParam('12070000', 'ns1:blz'));
-        $promise = $api->getBank(new SoapVar('12070000', XSD_STRING, null, null, 'blz', 'http://thomas-bayer.com/blz/'));
+        $promise = $api->getBank(new \SoapVar('12070000', XSD_STRING, null, null, 'blz', 'http://thomas-bayer.com/blz/'));
 
         $result = Block\await($promise, $this->loop);
 

tests/ProxyTest.php

@@ -1,5 +1,7 @@
 <?php
 
+namespace Clue\Tests\React\Soap;
+
 use PHPUnit\Framework\TestCase;
 use Clue\React\Soap\Proxy;