From 5df32386644b4653018f14e1bd00e4db3d0e2d84 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Gr=C3=A9goire=20Pineau?= Date: Wed, 17 Aug 2022 23:30:28 +0200 Subject: [PATCH] Fix bug with level in GitProcessor (#1749) --- src/Monolog/Processor/GitProcessor.php | 2 +- tests/Monolog/Processor/GitProcessorTest.php | 12 ++++++++++++ 2 files changed, 13 insertions(+), 1 deletion(-) diff --git a/src/Monolog/Processor/GitProcessor.php b/src/Monolog/Processor/GitProcessor.php index 4cbd9c84..5a70ac2e 100644 --- a/src/Monolog/Processor/GitProcessor.php +++ b/src/Monolog/Processor/GitProcessor.php @@ -44,7 +44,7 @@ class GitProcessor implements ProcessorInterface public function __invoke(LogRecord $record): LogRecord { // return if the level is not high enough - if ($record->level < $this->level) { + if ($record->level->isLowerThan($this->level)) { return $record; } diff --git a/tests/Monolog/Processor/GitProcessorTest.php b/tests/Monolog/Processor/GitProcessorTest.php index 4b0e1b0c..a8fc3d00 100644 --- a/tests/Monolog/Processor/GitProcessorTest.php +++ b/tests/Monolog/Processor/GitProcessorTest.php @@ -11,6 +11,7 @@ namespace Monolog\Processor; +use Monolog\Level; use Monolog\Test\TestCase; class GitProcessorTest extends TestCase @@ -26,4 +27,15 @@ class GitProcessorTest extends TestCase $this->assertArrayHasKey('git', $record->extra); $this->assertTrue(!is_array($record->extra['git']['branch'])); } + + /** + * @covers Monolog\Processor\GitProcessor::__invoke + */ + public function testProcessorWithLevel() + { + $processor = new GitProcessor(Level::Error); + $record = $processor($this->getRecord()); + + $this->assertArrayNotHasKey('git', $record->extra); + } }