[Mime][Security] Fix missing sprintf and add tests

Author: alamirault

Firewall/AccessListener.php

@@ -37,7 +37,7 @@ class AccessListener extends AbstractListener
     public function __construct(TokenStorageInterface $tokenStorage, AccessDecisionManagerInterface $accessDecisionManager, AccessMapInterface $map, bool $exceptionOnNoToken = false)
     {
         if (false !== $exceptionOnNoToken) {
-            throw new \LogicException('Argument $exceptionOnNoToken of "%s()" must be set to "false".', __METHOD__);
+            throw new \LogicException(sprintf('Argument $exceptionOnNoToken of "%s()" must be set to "false".', __METHOD__));
         }
 
         $this->tokenStorage = $tokenStorage;

Tests/Firewall/AccessListenerTest.php

@@ -266,4 +266,18 @@ public function testLazyPublicPagesShouldNotAccessTokenStorage()
         $listener = new AccessListener($tokenStorage, $this->createMock(AccessDecisionManagerInterface::class), $accessMap, false);
         $listener(new LazyResponseEvent(new RequestEvent($this->createMock(HttpKernelInterface::class), $request, HttpKernelInterface::MAIN_REQUEST)));
     }
+
+    public function testConstructWithTrueExceptionOnNoToken()
+    {
+        $tokenStorage = $this->createMock(TokenStorageInterface::class);
+        $tokenStorage->expects($this->never())->method(self::anything());
+
+        $accessMap = $this->createMock(AccessMapInterface::class);
+
+        $this->expectExceptionObject(
+            new \LogicException('Argument $exceptionOnNoToken of "Symfony\Component\Security\Http\Firewall\AccessListener::__construct()" must be set to "false".')
+        );
+
+        new AccessListener($tokenStorage, $this->createMock(AccessDecisionManagerInterface::class), $accessMap, true);
+    }
 }