CS fixes

Author: keradus

Core/Authentication/Provider/PreAuthenticatedAuthenticationProvider.php

@@ -60,7 +60,7 @@ public function authenticate(TokenInterface $token)
              throw new BadCredentialsException('No pre-authenticated principal found in request.');
          }
 
-        $user = $this->userProvider->loadUserByUsername($user);
+         $user = $this->userProvider->loadUserByUsername($user);
 
          $this->userChecker->checkPostAuth($user);
 

Tests/Acl/Voter/AclVoterTest.php

@@ -27,7 +27,7 @@ class AclVoterTest extends \PHPUnit_Framework_TestCase
      */
     public function testSupportsAttribute($attribute, $supported)
     {
-        list($voter, , $permissionMap, ,) = $this->getVoter(true, false);
+        list($voter, , $permissionMap) = $this->getVoter(true, false);
 
         $permissionMap
             ->expects($this->once())
@@ -44,7 +44,7 @@ public function testSupportsAttribute($attribute, $supported)
      */
     public function testSupportsAttributeNonString($attribute)
     {
-        list($voter, , , , ,) = $this->getVoter(true, false);
+        list($voter) = $this->getVoter(true, false);
 
         $this->assertFalse($voter->supportsAttribute($attribute));
     }
@@ -72,7 +72,7 @@ public function getSupportsAttributeNonStringTests()
      */
     public function testSupportsClass($class)
     {
-        list($voter, , , ,) = $this->getVoter();
+        list($voter) = $this->getVoter();
 
         $this->assertTrue($voter->supportsClass($class));
     }
@@ -88,7 +88,7 @@ public function getSupportsClassTests()
 
     public function testVote()
     {
-        list($voter, , $permissionMap, ,) = $this->getVoter();
+        list($voter, , $permissionMap) = $this->getVoter();
         $permissionMap
             ->expects($this->atLeastOnce())
             ->method('getMasks')
@@ -103,7 +103,7 @@ public function testVote()
      */
     public function testVoteWhenNoObjectIsPassed($allowIfObjectIdentityUnavailable)
     {
-        list($voter, , $permissionMap, ,) = $this->getVoter($allowIfObjectIdentityUnavailable);
+        list($voter, , $permissionMap) = $this->getVoter($allowIfObjectIdentityUnavailable);
         $permissionMap
             ->expects($this->once())
             ->method('getMasks')
@@ -124,7 +124,7 @@ public function testVoteWhenNoObjectIsPassed($allowIfObjectIdentityUnavailable)
      */
     public function testVoteWhenOidStrategyReturnsNull($allowIfUnavailable)
     {
-        list($voter, , $permissionMap, $oidStrategy,) = $this->getVoter($allowIfUnavailable);
+        list($voter, , $permissionMap, $oidStrategy) = $this->getVoter($allowIfUnavailable);
         $permissionMap
             ->expects($this->once())
             ->method('getMasks')

Tests/Http/Firewall/RememberMeListenerTest.php

@@ -19,7 +19,7 @@ class RememberMeListenerTest extends \PHPUnit_Framework_TestCase
 {
     public function testOnCoreSecurityDoesNotTryToPopulateNonEmptySecurityContext()
     {
-        list($listener, $context, $service, ,) = $this->getListener();
+        list($listener, $context, $service) = $this->getListener();
 
         $context
             ->expects($this->once())
@@ -37,7 +37,7 @@ public function testOnCoreSecurityDoesNotTryToPopulateNonEmptySecurityContext()
 
     public function testOnCoreSecurityDoesNothingWhenNoCookieIsSet()
     {
-        list($listener, $context, $service, ,) = $this->getListener();
+        list($listener, $context, $service) = $this->getListener();
 
         $context
             ->expects($this->once())
@@ -63,7 +63,7 @@ public function testOnCoreSecurityDoesNothingWhenNoCookieIsSet()
 
     public function testOnCoreSecurityIgnoresAuthenticationExceptionThrownByAuthenticationManagerImplementation()
     {
-        list($listener, $context, $service, $manager,) = $this->getListener();
+        list($listener, $context, $service, $manager) = $this->getListener();
 
         $context
             ->expects($this->once())
@@ -101,7 +101,7 @@ public function testOnCoreSecurityIgnoresAuthenticationExceptionThrownByAuthenti
 
     public function testOnCoreSecurity()
     {
-        list($listener, $context, $service, $manager,) = $this->getListener();
+        list($listener, $context, $service, $manager) = $this->getListener();
 
         $context
             ->expects($this->once())