Clean up test suite

Author: SimonFrings

composer.json

@@ -13,6 +13,9 @@
     "autoload": {
         "psr-4": { "Clue\\React\\EventSource\\": "src/" }
     },
+    "autoload-dev": {
+        "psr-4": { "Clue\\Tests\\React\\EventSource\\": "tests/" }
+    },
     "require": {
         "php": ">=5.4",
         "clue/buzz-react": "^2.5",

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 name="EventSource Test Suite">
             <directory>./tests/</directory>

tests/EventSourceTest.php

@@ -1,5 +1,7 @@
 <?php
 
+namespace Clue\Tests\React\EventSource;
+
 use PHPUnit\Framework\TestCase;
 use Clue\React\EventSource\EventSource;
 use React\Promise\Promise;
@@ -44,7 +46,7 @@ public function testConstructorCanBeCalledWithoutBrowser()
 
         $es = new EventSource('http://example.invalid', $loop);
 
-        $ref = new ReflectionProperty($es, 'browser');
+        $ref = new \ReflectionProperty($es, 'browser');
         $ref->setAccessible(true);
         $browser = $ref->getValue($es);
 
@@ -98,7 +100,7 @@ public function testCloseWillNotEmitErrorEventWhenGetRequestCancellationHandlerR
         $loop = $this->getMockBuilder('React\EventLoop\LoopInterface')->getMock();
 
         $pending = new Promise(function () { }, function () {
-            throw new RuntimeException();
+            throw new \RuntimeException();
         });
         $browser = $this->getMockBuilder('Clue\React\Buzz\Browser')->disableOriginalConstructor()->getMock();
         $browser->expects($this->once())->method('withOptions')->willReturnSelf();
@@ -131,7 +133,7 @@ public function testConstructorWillStartGetRequestThatWillStartRetryTimerWhenGet
 
         $es = new EventSource('http://example.com', $loop, $browser);
 
-        $deferred->reject(new RuntimeException());
+        $deferred->reject(new \RuntimeException());
     }
 
     public function testConstructorWillStartGetRequestThatWillStartRetryTimerThatWillRetryGetRequestWhenInitialGetRequestRejects()
@@ -156,7 +158,7 @@ public function testConstructorWillStartGetRequestThatWillStartRetryTimerThatWil
 
         $es = new EventSource('http://example.com', $loop, $browser);
 
-        $deferred->reject(new RuntimeException());
+        $deferred->reject(new \RuntimeException());
 
         $this->assertNotNull($timerRetry);
         $timerRetry();
@@ -181,7 +183,7 @@ public function testConstructorWillStartGetRequestThatWillEmitErrorWhenGetReques
         $es->on('error', function ($e) use (&$caught) {
             $caught = $e;
         });
-        $deferred->reject($expected = new RuntimeException());
+        $deferred->reject($expected = new \RuntimeException());
 
         $this->assertSame($expected, $caught);
     }
@@ -200,7 +202,7 @@ public function testConstructorWillStartGetRequestThatWillNotStartRetryTimerWhen
         $es->on('error', function () use ($es) {
             $es->close();
         });
-        $deferred->reject(new RuntimeException());
+        $deferred->reject(new \RuntimeException());
     }
 
     public function testCloseAfterGetRequestFromConstructorFailsWillCancelPendingRetryTimer()
@@ -220,7 +222,7 @@ public function testCloseAfterGetRequestFromConstructorFailsWillCancelPendingRet
 
         $es = new EventSource('http://example.com', $loop, $browser);
 
-        $deferred->reject(new RuntimeException());
+        $deferred->reject(new \RuntimeException());
 
         $es->close();
     }

tests/MessageEventTest.php

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