[Security] Removed an unnecessary call to sprintf() and added a test case.
This commit is contained in:
parent
6b25213e23
commit
314f29a718
@ -49,7 +49,7 @@ class AnonymousAuthenticationListener implements ListenerInterface
|
|||||||
$this->context->setToken(new AnonymousToken($this->key, 'anon.', array()));
|
$this->context->setToken(new AnonymousToken($this->key, 'anon.', array()));
|
||||||
|
|
||||||
if (null !== $this->logger) {
|
if (null !== $this->logger) {
|
||||||
$this->logger->info(sprintf('Populated SecurityContext with an anonymous Token'));
|
$this->logger->info('Populated SecurityContext with an anonymous Token');
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -59,4 +59,21 @@ class AnonymousAuthenticationListenerTest extends \PHPUnit_Framework_TestCase
|
|||||||
$listener = new AnonymousAuthenticationListener($context, 'TheKey');
|
$listener = new AnonymousAuthenticationListener($context, 'TheKey');
|
||||||
$listener->handle($this->getMock('Symfony\Component\HttpKernel\Event\GetResponseEvent', array(), array(), '', false));
|
$listener->handle($this->getMock('Symfony\Component\HttpKernel\Event\GetResponseEvent', array(), array(), '', false));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
public function testHandledEventIsLogged()
|
||||||
|
{
|
||||||
|
if (!interface_exists('Psr\Log\LoggerInterface')) {
|
||||||
|
$this->markTestSkipped('The "LoggerInterface" is not available');
|
||||||
|
}
|
||||||
|
|
||||||
|
$context = $this->getMock('Symfony\Component\Security\Core\SecurityContextInterface');
|
||||||
|
$logger = $this->getMock('Psr\Log\LoggerInterface');
|
||||||
|
$logger->expects($this->once())
|
||||||
|
->method('info')
|
||||||
|
->with('Populated SecurityContext with an anonymous Token')
|
||||||
|
;
|
||||||
|
|
||||||
|
$listener = new AnonymousAuthenticationListener($context, 'TheKey', $logger);
|
||||||
|
$listener->handle($this->getMock('Symfony\Component\HttpKernel\Event\GetResponseEvent', array(), array(), '', false));
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
Reference in New Issue
Block a user