Add SessionListener::onFinishRequest method to be inline with Symfony 3.4.12

Author: emodric

EventListener/SessionListener.php

@@ -13,6 +13,7 @@
 
 use Symfony\Component\EventDispatcher\EventSubscriberInterface;
 use Symfony\Component\HttpKernel\Event\FilterResponseEvent;
+use Symfony\Component\HttpKernel\Event\FinishRequestEvent;
 use Symfony\Component\HttpKernel\Event\GetResponseEvent;
 use Symfony\Component\HttpKernel\EventListener\SessionListener as BaseSessionListener;
 
@@ -78,6 +79,13 @@ public function onKernelResponse(FilterResponseEvent $event)
         // noop, see class description
     }
 
+    public function onFinishRequest(FinishRequestEvent $event)
+    {
+        if (method_exists($this->innerListener, 'onFinishRequest')) {
+            $this->innerListener->onFinishRequest($event);
+        }
+    }
+
     public static function getSubscribedEvents()
     {
         return BaseSessionListener::getSubscribedEvents();

Tests/Unit/EventListener/SessionListenerTest.php

@@ -47,6 +47,28 @@ public function testOnKernelRequestRemainsUntouched()
         $listener->onKernelRequest($event);
     }
 
+    public function testOnFinishRequestRemainsUntouched()
+    {
+        $event = $this
+            ->getMockBuilder('Symfony\Component\HttpKernel\Event\FinishRequestEvent')
+            ->disableOriginalConstructor()
+            ->getMock();
+
+        $inner = $this
+            ->getMockBuilder('Symfony\Component\HttpKernel\EventListener\SessionListener')
+            ->disableOriginalConstructor()
+            ->getMock();
+
+        $inner
+            ->expects($this->once())
+            ->method('onFinishRequest')
+            ->with($event)
+        ;
+
+        $listener = $this->getListener($inner);
+        $listener->onFinishRequest($event);
+    }
+
     /**
      * @dataProvider onKernelResponseProvider
      */