Fix quotes in exception messages

Author: fabpot

DataCollector/SecurityDataCollector.php

@@ -121,7 +121,7 @@ public function collect(Request $request, Response $response, \Exception $except
 
             $extractRoles = function ($role) {
                 if (!$role instanceof RoleInterface && !$role instanceof Role) {
-                    throw new \InvalidArgumentException(sprintf('Roles must be instances of %s or %s (%s given).', RoleInterface::class, Role::class, \is_object($role) ? \get_class($role) : \gettype($role)));
+                    throw new \InvalidArgumentException(sprintf('Roles must be instances of "%s" or "%s" ("%s" given).', RoleInterface::class, Role::class, \is_object($role) ? \get_class($role) : \gettype($role)));
                 }
 
                 return $role->getRole();

DependencyInjection/Compiler/AddSecurityVotersPass.php

@@ -51,7 +51,7 @@ public function process(ContainerBuilder $container)
 
             if (!method_exists($class, 'vote')) {
                 // in case the vote method is completely missing, to prevent exceptions when voting
-                throw new LogicException(sprintf('%s should implement the %s interface when used as voter.', $class, VoterInterface::class));
+                throw new LogicException(sprintf('"%s" should implement the "%s" interface when used as voter.', $class, VoterInterface::class));
             }
         }
 

Security/FirewallContext.php

@@ -41,7 +41,7 @@ public function __construct($listeners, ExceptionListener $exceptionListener = n
             $this->logoutListener = $logoutListener;
             $this->config = $config;
         } else {
-            throw new \InvalidArgumentException(sprintf('Argument 3 passed to %s() must be instance of %s or null, %s given.', __METHOD__, LogoutListener::class, \is_object($logoutListener) ? \get_class($logoutListener) : \gettype($logoutListener)));
+            throw new \InvalidArgumentException(sprintf('Argument 3 passed to "%s()" must be instance of "%s" or null, "%s" given.', __METHOD__, LogoutListener::class, \is_object($logoutListener) ? \get_class($logoutListener) : \gettype($logoutListener)));
         }
     }
 

Tests/DependencyInjection/Compiler/AddSecurityVotersPassTest.php

@@ -97,7 +97,7 @@ public function testVoterMissingInterface()
     public function testVoterMissingInterfaceAndMethod()
     {
         $exception = LogicException::class;
-        $message = 'stdClass should implement the Symfony\Component\Security\Core\Authorization\Voter\VoterInterface interface when used as voter.';
+        $message = '"stdClass" should implement the "Symfony\Component\Security\Core\Authorization\Voter\VoterInterface" interface when used as voter.';
 
         $this->expectException($exception);
         $this->expectExceptionMessage($message);