Skip to content
This repository was archived by the owner on May 31, 2024. It is now read-only.

Commit 0d0529f

Browse files
authored
[Security] Removed unused argument in Test
After #32998 there was a minor left over, the `testHandleAuthenticationClearsToken` `$tokenClass` argument is no longer used and can be safely removed.
1 parent 1598506 commit 0d0529f

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

Guard/Tests/GuardAuthenticatorHandlerTest.php

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ public function testHandleAuthenticationFailure()
8383
/**
8484
* @dataProvider getTokenClearingTests
8585
*/
86-
public function testHandleAuthenticationClearsToken($tokenClass, $tokenProviderKey, $actualProviderKey)
86+
public function testHandleAuthenticationClearsToken($tokenProviderKey, $actualProviderKey)
8787
{
8888
$this->tokenStorage->expects($this->never())
8989
->method('setToken')
@@ -104,10 +104,10 @@ public function testHandleAuthenticationClearsToken($tokenClass, $tokenProviderK
104104
public function getTokenClearingTests()
105105
{
106106
$tests = [];
107-
// correct token class and matching firewall => clear the token
108-
$tests[] = ['Symfony\Component\Security\Guard\Token\PostAuthenticationGuardToken', 'the_firewall_key', 'the_firewall_key'];
109-
$tests[] = ['Symfony\Component\Security\Guard\Token\PostAuthenticationGuardToken', 'the_firewall_key', 'different_key'];
110-
$tests[] = ['Symfony\Component\Security\Core\Authentication\Token\UsernamePasswordToken', 'the_firewall_key', 'the_firewall_key'];
107+
// matching firewall => clear the token
108+
$tests[] = ['the_firewall_key', 'the_firewall_key'];
109+
$tests[] = ['the_firewall_key', 'different_key'];
110+
$tests[] = ['the_firewall_key', 'the_firewall_key'];
111111

112112
return $tests;
113113
}

0 commit comments

Comments
 (0)