clean up remaining event mocks

Author: xabbuh

Tests/Firewall/RememberMeListenerTest.php

@@ -45,7 +45,7 @@ public function testOnCoreSecurityDoesNotTryToPopulateNonEmptyTokenStorage()
             ->method('setToken')
         ;
 
-        $this->assertNull($listener($this->getGetResponseEvent()));
+        $this->assertNull($listener(new RequestEvent($this->createMock(HttpKernelInterface::class), new Request(), HttpKernelInterface::MASTER_REQUEST)));
     }
 
     public function testOnCoreSecurityDoesNothingWhenNoCookieIsSet()
@@ -64,9 +64,7 @@ public function testOnCoreSecurityDoesNothingWhenNoCookieIsSet()
             ->willReturn(null)
         ;
 
-        $event = $this->getGetResponseEvent();
-
-        $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 @@ public function testOnCoreSecurityIgnoresAuthenticationExceptionThrownByAuthenti
             ->willThrowException($exception)
         ;
 
-        $event = $this->getGetResponseEvent($request);
-
-        $listener($event);
+        $listener(new RequestEvent($this->createMock(HttpKernelInterface::class), $request, HttpKernelInterface::MASTER_REQUEST));
     }
 
     public function testOnCoreSecurityIgnoresAuthenticationOptionallyRethrowsExceptionThrownAuthenticationManagerImplementation()
@@ -134,9 +130,7 @@ public function testOnCoreSecurityIgnoresAuthenticationOptionallyRethrowsExcepti
             ->willThrowException($exception)
         ;
 
-        $event = $this->getGetResponseEvent();
-
-        $listener($event);
+        $listener(new RequestEvent($this->createMock(HttpKernelInterface::class), new Request(), HttpKernelInterface::MASTER_REQUEST));
     }
 
     public function testOnCoreSecurityAuthenticationExceptionDuringAutoLoginTriggersLoginFail()
@@ -166,9 +160,7 @@ public function testOnCoreSecurityAuthenticationExceptionDuringAutoLoginTriggers
             ->method('authenticate')
         ;
 
-        $event = $this->getGetResponseEvent();
-
-        $listener($event);
+        $listener(new RequestEvent($this->createMock(HttpKernelInterface::class), new Request(), HttpKernelInterface::MASTER_REQUEST));
     }
 
     public function testOnCoreSecurity()
@@ -200,9 +192,7 @@ public function testOnCoreSecurity()
             ->willReturn($token)
         ;
 
-        $event = $this->getGetResponseEvent();
-
-        $listener($event);
+        $listener(new RequestEvent($this->createMock(HttpKernelInterface::class), new Request(), HttpKernelInterface::MASTER_REQUEST));
     }
 
     public function testSessionStrategy()
@@ -244,15 +234,13 @@ public function testSessionStrategy()
         $request = new Request();
         $request->setSession($session);
 
-        $event = $this->getGetResponseEvent($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 @@ public function testSessionIsMigratedByDefault()
         $request = new Request();
         $request->setSession($session);
 
-        $event = $this->getGetResponseEvent($request);
-
-        $listener($event);
+        $listener(new RequestEvent($this->createMock(HttpKernelInterface::class), $request, HttpKernelInterface::MASTER_REQUEST));
     }
 
     public function testOnCoreSecurityInteractiveLoginEventIsDispatchedIfDispatcherIsPresent()
@@ -332,8 +318,6 @@ public function testOnCoreSecurityInteractiveLoginEventIsDispatchedIfDispatcherI
             ->willReturn($token)
         ;
 
-        $event = $this->getGetResponseEvent();
-
         $dispatcher
             ->expects($this->once())
             ->method('dispatch')
@@ -343,23 +327,7 @@ public function testOnCoreSecurityInteractiveLoginEventIsDispatchedIfDispatcherI
             )
         ;
 
-        $listener($event);
-    }
-
-    protected function getGetResponseEvent(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)

Tests/FirewallTest.php

@@ -14,6 +14,7 @@
 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 @@ public function testOnKernelRequestStopsWhenThereIsAResponse()
             ->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);