From 78498d3b814ae2fc8d9b9419d520e628441fd865 Mon Sep 17 00:00:00 2001 From: "Alexander M. Turek" Date: Fri, 6 Jul 2018 21:52:06 +0200 Subject: [PATCH] Prefer PSR-3 to interact with Monolog in tests. --- .../Tests/Handler/ConsoleHandlerTest.php | 8 +++--- .../Bridge/Monolog/Tests/LoggerTest.php | 26 +++++++++---------- 2 files changed, 17 insertions(+), 17 deletions(-) diff --git a/src/Symfony/Bridge/Monolog/Tests/Handler/ConsoleHandlerTest.php b/src/Symfony/Bridge/Monolog/Tests/Handler/ConsoleHandlerTest.php index 63b5a8f07b..790bed3f59 100644 --- a/src/Symfony/Bridge/Monolog/Tests/Handler/ConsoleHandlerTest.php +++ b/src/Symfony/Bridge/Monolog/Tests/Handler/ConsoleHandlerTest.php @@ -180,19 +180,19 @@ class ConsoleHandlerTest extends TestCase $dispatcher = new EventDispatcher(); $dispatcher->addListener(ConsoleEvents::COMMAND, function () use ($logger) { - $logger->addInfo('Before command message.'); + $logger->info('Before command message.'); }); $dispatcher->addListener(ConsoleEvents::TERMINATE, function () use ($logger) { - $logger->addInfo('Before terminate message.'); + $logger->info('Before terminate message.'); }); $dispatcher->addSubscriber($handler); $dispatcher->addListener(ConsoleEvents::COMMAND, function () use ($logger) { - $logger->addInfo('After command message.'); + $logger->info('After command message.'); }); $dispatcher->addListener(ConsoleEvents::TERMINATE, function () use ($logger) { - $logger->addInfo('After terminate message.'); + $logger->info('After terminate message.'); }); $event = new ConsoleCommandEvent(new Command('foo'), $this->getMockBuilder('Symfony\Component\Console\Input\InputInterface')->getMock(), $output); diff --git a/src/Symfony/Bridge/Monolog/Tests/LoggerTest.php b/src/Symfony/Bridge/Monolog/Tests/LoggerTest.php index be60fc2789..e35bcb6f97 100644 --- a/src/Symfony/Bridge/Monolog/Tests/LoggerTest.php +++ b/src/Symfony/Bridge/Monolog/Tests/LoggerTest.php @@ -24,7 +24,7 @@ class LoggerTest extends TestCase $handler = new TestHandler(); $logger = new Logger(__METHOD__, array($handler)); - $this->assertTrue($logger->error('error message')); + $logger->error('error message'); $this->assertSame(array(), $logger->getLogs()); } @@ -33,7 +33,7 @@ class LoggerTest extends TestCase $handler = new TestHandler(); $logger = new Logger(__METHOD__, array($handler)); - $this->assertTrue($logger->error('error message')); + $logger->error('error message'); $this->assertSame(0, $logger->countErrors()); } @@ -43,7 +43,7 @@ class LoggerTest extends TestCase $processor = new DebugProcessor(); $logger = new Logger(__METHOD__, array($handler), array($processor)); - $this->assertTrue($logger->error('error message')); + $logger->error('error message'); $this->assertCount(1, $logger->getLogs()); } @@ -53,15 +53,15 @@ class LoggerTest extends TestCase $processor = new DebugProcessor(); $logger = new Logger(__METHOD__, array($handler), array($processor)); - $this->assertTrue($logger->debug('test message')); - $this->assertTrue($logger->info('test message')); - $this->assertTrue($logger->notice('test message')); - $this->assertTrue($logger->warning('test message')); + $logger->debug('test message'); + $logger->info('test message'); + $logger->notice('test message'); + $logger->warning('test message'); - $this->assertTrue($logger->error('test message')); - $this->assertTrue($logger->critical('test message')); - $this->assertTrue($logger->alert('test message')); - $this->assertTrue($logger->emergency('test message')); + $logger->error('test message'); + $logger->critical('test message'); + $logger->alert('test message'); + $logger->emergency('test message'); $this->assertSame(4, $logger->countErrors()); } @@ -72,7 +72,7 @@ class LoggerTest extends TestCase $logger = new Logger('test', array($handler)); $logger->pushProcessor(new DebugProcessor()); - $logger->addInfo('test'); + $logger->info('test'); $this->assertCount(1, $logger->getLogs()); list($record) = $logger->getLogs(); @@ -101,7 +101,7 @@ class LoggerTest extends TestCase $logger = new Logger('test', array($handler)); $logger->pushProcessor(new DebugProcessor()); - $logger->addInfo('test'); + $logger->info('test'); $logger->clear(); $this->assertEmpty($logger->getLogs());