Merge pull request #82 from clue-labs/unhandled-rejections

Update test suite to avoid unhandled promise rejections

Author: SimonFrings

composer.json

@@ -16,7 +16,7 @@
         "react/event-loop": "^1.2",
         "react/http": "^1.8",
         "react/promise": "^3 || ^2.0 || ^1.1",
-        "react/promise-stream": "^1.4",
+        "react/promise-stream": "^1.6",
         "react/socket": "^1.12",
         "react/stream": "^1.2",
         "rize/uri-template": "^0.3"

tests/IntegrationTest.php

@@ -4,6 +4,7 @@
 
 use Clue\React\Docker\Client;
 use Psr\Http\Message\ServerRequestInterface;
+use React\Http\Message\Response;
 use React\Promise\Deferred;
 
 class IntegrationTest extends TestCase
@@ -26,6 +27,7 @@ public function testPingCtorWithExplicitUnixUrlSendsRequestToGivenUnixSocket()
         $deferred = new Deferred();
         $http = new \React\Http\HttpServer(function (ServerRequestInterface $request) use ($deferred) {
             $deferred->resolve($request->getRequestTarget());
+            return Response::plaintext('OK');
         });
 
         $http->listen($socket);
@@ -46,6 +48,7 @@ public function testPingCtorWithExplicitHttpUrlSendsRequestToGivenHttpUrlWithBas
         $deferred = new Deferred();
         $http = new \React\Http\HttpServer(function (ServerRequestInterface $request) use ($deferred) {
             $deferred->resolve($request->getRequestTarget());
+            return Response::plaintext('OK');
         });
 
         $socket = new \React\Socket\SocketServer('127.0.0.1:0');