Skip to content

Commit 86dac9e

Browse files
committed
Ensure the running method outputs are tested
1 parent 510e2eb commit 86dac9e

File tree

1 file changed

+10
-3
lines changed

1 file changed

+10
-3
lines changed

tests/MiddlewareRunnerTest.php

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,8 @@
99
use GuzzleHttp\Psr7\Request;
1010
use GuzzleHttp\Psr7\Response;
1111
use Phake;
12+
use React\EventLoop\Factory;
13+
use Throwable;
1214
use function Clue\React\Block\await;
1315
use function React\Promise\reject;
1416
use function React\Promise\resolve;
@@ -17,6 +19,7 @@ class MiddlewareRunnerTest extends TestCase
1719
{
1820
public function testAll()
1921
{
22+
$loop = Factory::create();
2023
$request = new Request('GET', 'https://example.com/');
2124
$response = new Response(200);
2225
$exception = new Exception();
@@ -48,9 +51,13 @@ public function testAll()
4851
];
4952

5053
$executioner = new MiddlewareRunner(...$args);
51-
$executioner->pre($request);
52-
$executioner->post($response);
53-
$executioner->error($exception);
54+
self::assertSame($request, await($executioner->pre($request), $loop));
55+
self::assertSame($response, await($executioner->post($response), $loop));
56+
try {
57+
await($executioner->error($exception), $loop);
58+
} catch (Throwable $throwable) {
59+
self::assertSame($exception, $throwable);
60+
}
5461

5562
Phake::inOrder(
5663
Phake::verify($middlewareOne)->pre($request, $options),

0 commit comments

Comments
 (0)