From 3608a6d068d768be8bd75e68b2e050ad8eb9a866 Mon Sep 17 00:00:00 2001 From: Alejandro Celaya Date: Sat, 22 Oct 2022 07:28:15 +0200 Subject: [PATCH] Migrated CloseDbConnectionEventListenerDelegatorTest to use PHPUnit mocks --- ...seDbConnectionEventListenerDelegatorTest.php | 17 +++++++---------- 1 file changed, 7 insertions(+), 10 deletions(-) diff --git a/module/Core/test/EventDispatcher/CloseDbConnectionEventListenerDelegatorTest.php b/module/Core/test/EventDispatcher/CloseDbConnectionEventListenerDelegatorTest.php index b826802b..7123aa29 100644 --- a/module/Core/test/EventDispatcher/CloseDbConnectionEventListenerDelegatorTest.php +++ b/module/Core/test/EventDispatcher/CloseDbConnectionEventListenerDelegatorTest.php @@ -4,23 +4,20 @@ declare(strict_types=1); namespace ShlinkioTest\Shlink\Core\EventDispatcher; +use PHPUnit\Framework\MockObject\MockObject; use PHPUnit\Framework\TestCase; -use Prophecy\PhpUnit\ProphecyTrait; -use Prophecy\Prophecy\ObjectProphecy; use Psr\Container\ContainerInterface; use Shlinkio\Shlink\Common\Doctrine\ReopeningEntityManagerInterface; use Shlinkio\Shlink\Core\EventDispatcher\CloseDbConnectionEventListenerDelegator; class CloseDbConnectionEventListenerDelegatorTest extends TestCase { - use ProphecyTrait; - private CloseDbConnectionEventListenerDelegator $delegator; - private ObjectProphecy $container; + private MockObject $container; protected function setUp(): void { - $this->container = $this->prophesize(ContainerInterface::class); + $this->container = $this->createMock(ContainerInterface::class); $this->delegator = new CloseDbConnectionEventListenerDelegator(); } @@ -35,12 +32,12 @@ class CloseDbConnectionEventListenerDelegatorTest extends TestCase }; }; - $em = $this->prophesize(ReopeningEntityManagerInterface::class); - $getEm = $this->container->get('em')->willReturn($em->reveal()); + $this->container->expects($this->once())->method('get')->with($this->equalTo('em'))->willReturn( + $this->createMock(ReopeningEntityManagerInterface::class), + ); - ($this->delegator)($this->container->reveal(), '', $callback); + ($this->delegator)($this->container, '', $callback); self::assertTrue($callbackInvoked); - $getEm->shouldHaveBeenCalledOnce(); } }