Fix: autowiring

Author: tatevikg1

src/EventSubscriber/UnauthorizedSubscriber.php

@@ -7,7 +7,6 @@
 use Symfony\Component\EventDispatcher\EventSubscriberInterface;
 use Symfony\Component\HttpFoundation\JsonResponse;
 use Symfony\Component\HttpFoundation\RedirectResponse;
-use Symfony\Component\HttpFoundation\Session\Flash\FlashBagInterface;
 use Symfony\Component\HttpKernel\Event\ExceptionEvent;
 use Symfony\Component\HttpKernel\KernelEvents;
 use Symfony\Component\Routing\Generator\UrlGeneratorInterface;
@@ -17,7 +16,6 @@ class UnauthorizedSubscriber implements EventSubscriberInterface
 {
     public function __construct(
         private readonly UrlGeneratorInterface $urlGenerator,
-        private readonly FlashBagInterface $flashBag,
     ) {
     }
 
@@ -53,7 +51,11 @@ public function onKernelException(ExceptionEvent $event): void
             }
 
             if ($request->hasSession()) {
-                $this->flashBag->add('error', 'Your session has expired. Please log in again.');
+                $session = $request->getSession();
+
+                if (method_exists($session, 'getFlashBag')) {
+                    $session->getFlashBag()->add('error', 'Your session has expired. Please log in again.');
+                }
             }
 
             $event->setResponse(new RedirectResponse($loginUrl));

tests/Unit/EventSubscriber/UnauthorizedSubscriberTest.php

@@ -13,6 +13,7 @@
 use Symfony\Component\HttpFoundation\RedirectResponse;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Session\Flash\FlashBagInterface;
+use Symfony\Component\HttpFoundation\Session\Session;
 use Symfony\Component\HttpFoundation\Session\SessionInterface;
 use Symfony\Component\HttpKernel\Event\ExceptionEvent;
 use Symfony\Component\HttpKernel\HttpKernelInterface;
@@ -23,13 +24,11 @@ class UnauthorizedSubscriberTest extends TestCase
 {
     private UnauthorizedSubscriber $subscriber;
     private UrlGeneratorInterface&MockObject $urlGenerator;
-    private FlashBagInterface&MockObject $flashBag;
 
     protected function setUp(): void
     {
         $this->urlGenerator = $this->createMock(UrlGeneratorInterface::class);
-        $this->flashBag = $this->createMock(FlashBagInterface::class);
-        $this->subscriber = new UnauthorizedSubscriber($this->urlGenerator, $this->flashBag);
+        $this->subscriber = new UnauthorizedSubscriber($this->urlGenerator);
     }
 
     public function testGetSubscribedEvents(): void
@@ -44,13 +43,15 @@ public function testOnKernelExceptionWithUnauthorizedException(): void
     {
         $authException = new AuthenticationException('Unauthorized');
 
-        $session = $this->createMock(SessionInterface::class);
-        $session->expects($this->once())->method('invalidate');
-
-        $this->flashBag->expects($this->once())
+        $flashBag = $this->createMock(FlashBagInterface::class);
+        $flashBag->expects($this->once())
             ->method('add')
             ->with('error', 'Your session has expired. Please log in again.');
 
+        $session = $this->createMock(Session::class);
+        $session->expects($this->once())->method('invalidate');
+        $session->method('getFlashBag')->willReturn($flashBag);
+
         $request = $this->createMock(Request::class);
         $request->method('hasSession')->willReturn(true);
         $request->method('getSession')->willReturn($session);