Merge pull request #3221 from mehdihasanpour/cleanup

Cleanup

Author: l0gicgate

tests/AppTest.php

@@ -1763,7 +1763,7 @@ public function testHandleReturnsEmptyResponseBodyWithHeadRequestMethod(): void
         $responseProphecy->getBody()->willReturn($streamProphecy->reveal());
         $responseProphecy
             ->withBody(Argument::type(StreamInterface::class))
-            ->will(function ($args) use ($streamProphecy) {
+            ->will(function ($args) {
                 $this->getBody()->willReturn($args[0]);
                 return $this;
             });
@@ -1869,7 +1869,7 @@ public function testCanBeReExecutedRecursivelyDuringDispatch(): void
         $middlewareProphecy2->process(
             Argument::type(ServerRequestInterface::class),
             Argument::type(RequestHandlerInterface::class)
-        )->will(function ($args) use ($app) {
+        )->will(function ($args) {
             /** @var ServerRequestInterface $request */
             $request = $args[0];
 

tests/Mocks/MockStream.php

@@ -263,6 +263,6 @@ public function getMetadata($key = null)
 
         $meta = stream_get_meta_data($this->stream);
 
-        return isset($meta[$key]) ? $meta[$key] : null;
+        return $meta[$key] ?? null;
     }
 }

tests/Routing/RouteTest.php

@@ -50,7 +50,7 @@ class RouteTest extends TestCase
      */
     public function createRoute($methods = 'GET', string $pattern = '/', $callable = null): Route
     {
-        $callable = $callable ?? function (ServerRequestInterface $request, ResponseInterface $response) {
+        $callable ??= function (ServerRequestInterface $request, ResponseInterface $response) {
                 return $response;
         };