Skip to content

Commit 6bb0b59

Browse files
committed
Update the tests for Slim 4
1 parent 57c8beb commit 6bb0b59

File tree

8 files changed

+55
-221
lines changed

8 files changed

+55
-221
lines changed

tests/ApplicationTest.php

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,9 @@
22

33
namespace DI\Bridge\Slim\Test;
44

5-
use DI\Bridge\Slim\App;
5+
use DI\Bridge\Slim\Bridge;
66
use DI\Bridge\Slim\Test\Mock\RequestFactory;
77
use PHPUnit\Framework\TestCase;
8-
use Slim\Http\Response;
98

109
class ApplicationTest extends TestCase
1110
{
@@ -14,14 +13,15 @@ class ApplicationTest extends TestCase
1413
*/
1514
public function runs()
1615
{
17-
$app = new App;
16+
$app = Bridge::create();
1817

1918
$called = false;
20-
$app->get('/', function () use (&$called) {
19+
$app->get('/', function ($request, $response) use (&$called) {
2120
$called = true;
21+
return $response;
2222
});
23+
$app->handle(RequestFactory::create());
2324

24-
$app->callMiddlewareStack(RequestFactory::create(), new Response);
2525
$this->assertTrue($called);
2626
}
2727
}

tests/ContainerTest.php

Lines changed: 0 additions & 59 deletions
This file was deleted.

tests/ErrorTest.php

Lines changed: 0 additions & 98 deletions
This file was deleted.

tests/Fixture/Middleware.php

Lines changed: 0 additions & 15 deletions
This file was deleted.

tests/Fixture/UserController.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
<?php
1+
<?php declare(strict_types=1);
22

33
namespace DI\Bridge\Slim\Test\Fixture;
44

tests/MiddlewareTest.php

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,13 @@
1-
<?php
1+
<?php declare(strict_types=1);
22

33
namespace DI\Bridge\Slim\Test;
44

5-
use DI\Bridge\Slim\App;
5+
use DI\Bridge\Slim\Bridge;
66
use DI\Bridge\Slim\Test\Mock\RequestFactory;
77
use PHPUnit\Framework\TestCase;
8-
use Psr\Http\Message\ResponseInterface;
98
use Psr\Http\Message\ServerRequestInterface;
10-
use Slim\Http\Response;
9+
use Psr\Http\Server\RequestHandlerInterface;
10+
use Zend\Diactoros\Response\TextResponse;
1111

1212
class MiddlewareTest extends TestCase
1313
{
@@ -16,14 +16,13 @@ class MiddlewareTest extends TestCase
1616
*/
1717
public function invokes_closure_middleware()
1818
{
19-
$app = new App;
20-
$app->add(function (ServerRequestInterface $request, ResponseInterface $response, callable $next) {
21-
$response->getBody()->write('Hello ' . $request->getQueryParams()['foo']);
22-
return $response;
19+
$app = Bridge::create();
20+
$app->add(function (ServerRequestInterface $request, RequestHandlerInterface $next) {
21+
return new TextResponse('Hello ' . $request->getQueryParams()['foo']);
2322
});
2423
$app->get('/', function () {});
2524

26-
$response = $app->callMiddlewareStack(RequestFactory::create('/', 'foo=matt'), new Response);
25+
$response = $app->handle(RequestFactory::create('/', 'foo=matt'));
2726

2827
$this->assertEquals('Hello matt', $response->getBody()->__toString());
2928
}

tests/Mock/RequestFactory.php

Lines changed: 15 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,24 @@
1-
<?php
1+
<?php declare(strict_types=1);
22

33
namespace DI\Bridge\Slim\Test\Mock;
44

5-
use Slim\Http\Environment;
6-
use Slim\Http\Request;
5+
use Psr\Http\Message\ServerRequestInterface;
6+
use Zend\Diactoros\ServerRequest;
77

88
class RequestFactory
99
{
10-
public static function create($uri = '/', $queryString = '')
10+
public static function create($uri = '/', $queryString = ''): ServerRequestInterface
1111
{
12-
return Request::createFromEnvironment(Environment::mock([
13-
'SCRIPT_NAME' => 'index.php',
14-
'REQUEST_URI' => $uri,
15-
'QUERY_STRING' => $queryString,
16-
]));
12+
parse_str($queryString, $queryParams);
13+
return new ServerRequest(
14+
[],
15+
[],
16+
$uri,
17+
'GET',
18+
'php://temp',
19+
[],
20+
[],
21+
$queryParams
22+
);
1723
}
1824
}

0 commit comments

Comments
 (0)