Fix closures in method prophecies

kwhat · Dec 19, 2019 · c853037 · c853037
1 parent 941828c
commit c853037
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 14 deletions.
diff --git a/tests/Routing/RouteCollectorProxyTest.php b/tests/Routing/RouteCollectorProxyTest.php
@@ -9,6 +9,7 @@
 
 namespace Slim\Tests\Routing;
 
+use Prophecy\Argument;
 use Psr\Container\ContainerInterface;
 use Psr\Http\Message\ResponseFactoryInterface;
 use Psr\Http\Message\ResponseInterface;
@@ -134,7 +135,7 @@ public function testGet()
 
         $routeCollectorProphecy = $this->prophesize(RouteCollectorInterface::class);
         $routeCollectorProphecy
-            ->map(['GET'], $pattern, $callable)
+            ->map(['GET'], $pattern, Argument::is($callable))
             ->willReturn($routeProphecy->reveal())
             ->shouldBeCalledOnce();
 
@@ -167,7 +168,7 @@ public function testPost()
 
         $routeCollectorProphecy = $this->prophesize(RouteCollectorInterface::class);
         $routeCollectorProphecy
-            ->map(['POST'], $pattern, $callable)
+            ->map(['POST'], $pattern, Argument::is($callable))
             ->willReturn($routeProphecy->reveal())
             ->shouldBeCalledOnce();
 
@@ -200,7 +201,7 @@ public function testPut()
 
         $routeCollectorProphecy = $this->prophesize(RouteCollectorInterface::class);
         $routeCollectorProphecy
-            ->map(['PUT'], $pattern, $callable)
+            ->map(['PUT'], $pattern, Argument::is($callable))
             ->willReturn($routeProphecy->reveal())
             ->shouldBeCalledOnce();
 
@@ -233,7 +234,7 @@ public function testPatch()
 
         $routeCollectorProphecy = $this->prophesize(RouteCollectorInterface::class);
         $routeCollectorProphecy
-            ->map(['PATCH'], $pattern, $callable)
+            ->map(['PATCH'], $pattern, Argument::is($callable))
             ->willReturn($routeProphecy->reveal())
             ->shouldBeCalledOnce();
 
@@ -266,7 +267,7 @@ public function testDelete()
 
         $routeCollectorProphecy = $this->prophesize(RouteCollectorInterface::class);
         $routeCollectorProphecy
-            ->map(['DELETE'], $pattern, $callable)
+            ->map(['DELETE'], $pattern, Argument::is($callable))
             ->willReturn($routeProphecy->reveal())
             ->shouldBeCalledOnce();
 
@@ -299,7 +300,7 @@ public function testOptions()
 
         $routeCollectorProphecy = $this->prophesize(RouteCollectorInterface::class);
         $routeCollectorProphecy
-            ->map(['OPTIONS'], $pattern, $callable)
+            ->map(['OPTIONS'], $pattern, Argument::is($callable))
             ->willReturn($routeProphecy->reveal())
             ->shouldBeCalledOnce();
 
@@ -332,7 +333,7 @@ public function testAny()
 
         $routeCollectorProphecy = $this->prophesize(RouteCollectorInterface::class);
         $routeCollectorProphecy
-            ->map(['GET', 'POST', 'PUT', 'PATCH', 'DELETE', 'OPTIONS'], $pattern, $callable)
+            ->map(['GET', 'POST', 'PUT', 'PATCH', 'DELETE', 'OPTIONS'], $pattern, Argument::is($callable))
             ->willReturn($routeProphecy->reveal())
             ->shouldBeCalledOnce();
 
@@ -366,7 +367,7 @@ public function testMap()
 
         $routeCollectorProphecy = $this->prophesize(RouteCollectorInterface::class);
         $routeCollectorProphecy
-            ->map($methods, $pattern, $callable)
+            ->map($methods, $pattern, Argument::is($callable))
             ->willReturn($routeProphecy->reveal())
             ->shouldBeCalledOnce();
 
@@ -440,7 +441,7 @@ public function testGroup()
 
         $routeCollectorProphecy = $this->prophesize(RouteCollectorInterface::class);
         $routeCollectorProphecy
-            ->group($pattern, $callable)
+            ->group($pattern, Argument::is($callable))
             ->willReturn($routeGroupProphecy->reveal())
             ->shouldBeCalledOnce();
 

diff --git a/tests/Routing/RouteCollectorTest.php b/tests/Routing/RouteCollectorTest.php
@@ -9,6 +9,7 @@
 
 namespace Slim\Tests\Routing;
 
+use Prophecy\Argument;
 use Psr\Container\ContainerInterface;
 use Psr\Http\Message\ResponseFactoryInterface;
 use RuntimeException;
@@ -73,7 +74,7 @@ public function testMapPrependsGroupPattern()
 
         $callableResolverProphecy = $this->prophesize(CallableResolverInterface::class);
         $callableResolverProphecy
-            ->resolve($callable)
+            ->resolve(Argument::is($callable))
             ->willReturn($callable)
             ->shouldBeCalledOnce();
 

diff --git a/tests/Routing/RouteTest.php b/tests/Routing/RouteTest.php
@@ -49,7 +49,7 @@ public function createRoute($methods = 'GET', string $pattern = '/', $callable =
 
         $callableResolverProphecy = $this->prophesize(CallableResolverInterface::class);
         $callableResolverProphecy
-            ->resolve($callable)
+            ->resolve(Argument::is($callable))
             ->willReturn($callable);
 
         $streamProphecy = $this->prophesize(StreamInterface::class);
@@ -280,7 +280,7 @@ public function testAddMiddlewareOnGroup()
 
         $callableResolverProphecy = $this->prophesize(CallableResolverInterface::class);
         $callableResolverProphecy
-            ->resolve($callable)
+            ->resolve(Argument::is($callable))
             ->willReturn($callable)
             ->shouldBeCalledOnce();
 
@@ -410,7 +410,7 @@ public function testControllerMethodAsStringResolvesWithContainer()
         $callable = 'CallableTest:toCall';
 
         $callableResolverProphecy
-            ->resolve($callable)
+            ->resolve(Argument::is($callable))
             ->willReturn(function (
                 ServerRequestInterface $request,
                 ResponseInterface $response
@@ -425,7 +425,7 @@ public function testControllerMethodAsStringResolvesWithContainer()
 
         $deferred = $callableResolverProphecy->reveal()->resolve($callable);
         $callableResolverProphecy
-            ->resolve($deferred)
+            ->resolve(Argument::is($deferred))
             ->willReturn($deferred)
             ->shouldBeCalledOnce();