diff --git a/src/Symfony/Component/Security/Guard/Tests/Firewall/GuardAuthenticationListenerTest.php b/src/Symfony/Component/Security/Guard/Tests/Firewall/GuardAuthenticationListenerTest.php index 512bf313f7..759504711b 100644 --- a/src/Symfony/Component/Security/Guard/Tests/Firewall/GuardAuthenticationListenerTest.php +++ b/src/Symfony/Component/Security/Guard/Tests/Firewall/GuardAuthenticationListenerTest.php @@ -16,6 +16,7 @@ use Psr\Log\LoggerInterface; use Symfony\Component\HttpFoundation\Request; use Symfony\Component\HttpFoundation\Response; use Symfony\Component\HttpKernel\Event\RequestEvent; +use Symfony\Component\HttpKernel\HttpKernelInterface; use Symfony\Component\Security\Core\Authentication\AuthenticationProviderManager; use Symfony\Component\Security\Core\Authentication\Token\TokenInterface; use Symfony\Component\Security\Core\Exception\AuthenticationException; @@ -322,14 +323,7 @@ class GuardAuthenticationListenerTest extends TestCase $this->guardAuthenticatorHandler = $this->createMock(GuardAuthenticatorHandler::class); $this->request = new Request([], [], [], [], [], []); - $this->event = $this->getMockBuilder(RequestEvent::class) - ->disableOriginalConstructor() - ->setMethods(['getRequest']) - ->getMock(); - $this->event - ->expects($this->any()) - ->method('getRequest') - ->willReturn($this->request); + $this->event = new RequestEvent($this->createMock(HttpKernelInterface::class), $this->request, HttpKernelInterface::MASTER_REQUEST); $this->logger = $this->createMock(LoggerInterface::class); $this->rememberMeServices = $this->createMock(RememberMeServicesInterface::class); diff --git a/src/Symfony/Component/Security/Http/Tests/Firewall/RememberMeListenerTest.php b/src/Symfony/Component/Security/Http/Tests/Firewall/RememberMeListenerTest.php index 42c493cfaa..18a178e69b 100644 --- a/src/Symfony/Component/Security/Http/Tests/Firewall/RememberMeListenerTest.php +++ b/src/Symfony/Component/Security/Http/Tests/Firewall/RememberMeListenerTest.php @@ -45,7 +45,7 @@ class RememberMeListenerTest extends TestCase ->method('setToken') ; - $this->assertNull($listener($this->getRequestEvent())); + $this->assertNull($listener(new RequestEvent($this->createMock(HttpKernelInterface::class), new Request(), HttpKernelInterface::MASTER_REQUEST))); } public function testOnCoreSecurityDoesNothingWhenNoCookieIsSet() @@ -64,9 +64,7 @@ class RememberMeListenerTest extends TestCase ->willReturn(null) ; - $event = $this->getRequestEvent(); - - $this->assertNull($listener($event)); + $this->assertNull($listener(new RequestEvent($this->createMock(HttpKernelInterface::class), new Request(), HttpKernelInterface::MASTER_REQUEST))); } public function testOnCoreSecurityIgnoresAuthenticationExceptionThrownByAuthenticationManagerImplementation() @@ -99,9 +97,7 @@ class RememberMeListenerTest extends TestCase ->willThrowException($exception) ; - $event = $this->getRequestEvent($request); - - $listener($event); + $listener(new RequestEvent($this->createMock(HttpKernelInterface::class), $request, HttpKernelInterface::MASTER_REQUEST)); } public function testOnCoreSecurityIgnoresAuthenticationOptionallyRethrowsExceptionThrownAuthenticationManagerImplementation() @@ -134,9 +130,7 @@ class RememberMeListenerTest extends TestCase ->willThrowException($exception) ; - $event = $this->getRequestEvent(); - - $listener($event); + $listener(new RequestEvent($this->createMock(HttpKernelInterface::class), new Request(), HttpKernelInterface::MASTER_REQUEST)); } public function testOnCoreSecurityAuthenticationExceptionDuringAutoLoginTriggersLoginFail() @@ -166,9 +160,7 @@ class RememberMeListenerTest extends TestCase ->method('authenticate') ; - $event = $this->getRequestEvent(); - - $listener($event); + $listener(new RequestEvent($this->createMock(HttpKernelInterface::class), new Request(), HttpKernelInterface::MASTER_REQUEST)); } public function testOnCoreSecurity() @@ -200,9 +192,7 @@ class RememberMeListenerTest extends TestCase ->willReturn($token) ; - $event = $this->getRequestEvent(); - - $listener($event); + $listener(new RequestEvent($this->createMock(HttpKernelInterface::class), new Request(), HttpKernelInterface::MASTER_REQUEST)); } public function testSessionStrategy() @@ -244,15 +234,13 @@ class RememberMeListenerTest extends TestCase $request = new Request(); $request->setSession($session); - $event = $this->getRequestEvent($request); - $sessionStrategy ->expects($this->once()) ->method('onAuthentication') ->willReturn(null) ; - $listener($event); + $listener(new RequestEvent($this->createMock(HttpKernelInterface::class), $request, HttpKernelInterface::MASTER_REQUEST)); } public function testSessionIsMigratedByDefault() @@ -298,9 +286,7 @@ class RememberMeListenerTest extends TestCase $request = new Request(); $request->setSession($session); - $event = $this->getRequestEvent($request); - - $listener($event); + $listener(new RequestEvent($this->createMock(HttpKernelInterface::class), $request, HttpKernelInterface::MASTER_REQUEST)); } public function testOnCoreSecurityInteractiveLoginEventIsDispatchedIfDispatcherIsPresent() @@ -332,8 +318,6 @@ class RememberMeListenerTest extends TestCase ->willReturn($token) ; - $event = $this->getRequestEvent(); - $dispatcher ->expects($this->once()) ->method('dispatch') @@ -343,23 +327,7 @@ class RememberMeListenerTest extends TestCase ) ; - $listener($event); - } - - protected function getRequestEvent(Request $request = null): RequestEvent - { - $request = $request ?? new Request(); - - $event = $this->getMockBuilder(RequestEvent::class) - ->setConstructorArgs([$this->createMock(HttpKernelInterface::class), $request, HttpKernelInterface::MASTER_REQUEST]) - ->getMock(); - $event - ->expects($this->any()) - ->method('getRequest') - ->willReturn($request) - ; - - return $event; + $listener(new RequestEvent($this->createMock(HttpKernelInterface::class), new Request(), HttpKernelInterface::MASTER_REQUEST)); } protected function getListener($withDispatcher = false, $catchExceptions = true, $withSessionStrategy = false) diff --git a/src/Symfony/Component/Security/Http/Tests/FirewallTest.php b/src/Symfony/Component/Security/Http/Tests/FirewallTest.php index f607aa3857..b345f80306 100644 --- a/src/Symfony/Component/Security/Http/Tests/FirewallTest.php +++ b/src/Symfony/Component/Security/Http/Tests/FirewallTest.php @@ -14,6 +14,7 @@ namespace Symfony\Component\Security\Http\Tests; use PHPUnit\Framework\TestCase; use Symfony\Component\EventDispatcher\EventDispatcherInterface; use Symfony\Component\HttpFoundation\Request; +use Symfony\Component\HttpFoundation\Response; use Symfony\Component\HttpKernel\Event\RequestEvent; use Symfony\Component\HttpKernel\HttpKernelInterface; use Symfony\Component\Security\Http\Firewall; @@ -68,20 +69,8 @@ class FirewallTest extends TestCase ->willReturn([[$first, $second], null, null]) ; - $event = $this->getMockBuilder(RequestEvent::class) - ->setMethods(['hasResponse']) - ->setConstructorArgs([ - $this->createMock(HttpKernelInterface::class), - $this->createMock(Request::class), - HttpKernelInterface::MASTER_REQUEST, - ]) - ->getMock() - ; - $event - ->expects($this->once()) - ->method('hasResponse') - ->willReturn(true) - ; + $event = new RequestEvent($this->createMock(HttpKernelInterface::class), new Request(), HttpKernelInterface::MASTER_REQUEST); + $event->setResponse(new Response()); $firewall = new Firewall($map, $this->createMock(EventDispatcherInterface::class)); $firewall->onKernelRequest($event); diff --git a/src/Symfony/Component/Stopwatch/Tests/StopwatchTest.php b/src/Symfony/Component/Stopwatch/Tests/StopwatchTest.php index 75d3bb5d5b..d0bc1b0a7c 100644 --- a/src/Symfony/Component/Stopwatch/Tests/StopwatchTest.php +++ b/src/Symfony/Component/Stopwatch/Tests/StopwatchTest.php @@ -71,12 +71,7 @@ class StopwatchTest extends TestCase $events = new \ReflectionProperty(Section::class, 'events'); $events->setAccessible(true); - $stopwatchMockEvent = $this->getMockBuilder(StopwatchEvent::class) - ->setConstructorArgs([microtime(true) * 1000]) - ->getMock() - ; - - $events->setValue(end($section), ['foo' => $stopwatchMockEvent]); + $events->setValue(end($section), ['foo' => new StopwatchEvent(microtime(true) * 1000)]); $this->assertFalse($stopwatch->isStarted('foo')); }