[BUGFIX] Check error level is higher

When the error level is set to warning on the spy errors should
also be found as fail.

Author: jaapio

packages/guides-cli/composer.json

@@ -26,7 +26,7 @@
     ],
     "require": {
         "php": "^8.1",
-        "monolog/monolog": "^2.9 || ^3.0",
+        "monolog/monolog": "^3.0",
         "phpdocumentor/guides": "^1.0 || ^2.0",
         "phpdocumentor/guides-restructured-text": "^1.0 || ^2.0",
         "symfony/config": "^5.4 || ^6.3 || ^7.0",

packages/guides-cli/src/Logger/SpyProcessor.php

@@ -13,6 +13,7 @@
 
 namespace phpDocumentor\Guides\Cli\Logger;
 
+use Monolog\Level;
 use Monolog\LogRecord;
 use Monolog\Processor\ProcessorInterface;
 use Psr\Log\LogLevel;
@@ -27,9 +28,15 @@
 final class SpyProcessor implements ProcessorInterface
 {
     private bool $hasBeenCalled = false;
+    private Level $level;
 
-    public function __construct(private string|null $level = LogLevel::WARNING)
+    public function __construct(string|null $level = LogLevel::WARNING)
     {
+        if ($level === null) {
+            $level = LogLevel::WARNING;
+        }
+
+        $this->level = Level::fromName(strtolower($level));
     }
 
     public function hasBeenCalled(): bool
@@ -39,7 +46,8 @@ public function hasBeenCalled(): bool
 
     public function __invoke(array|LogRecord $record): array|LogRecord
     {
-        if (strtolower($record['level_name']) === $this->level) {
+        $recordLevel = Level::fromName($record['level_name']);
+        if ($this->level->includes($recordLevel)) {
             $this->hasBeenCalled = true;
         }
 

packages/guides-cli/tests/unit/Logger/SpyProcessorTest.php

@@ -14,6 +14,7 @@
 namespace phpDocumentor\Guides\Cli\Logger;
 
 use PHPUnit\Framework\TestCase;
+use Psr\Log\LogLevel;
 
 final class SpyProcessorTest extends TestCase
 {
@@ -30,4 +31,18 @@ public function testItKnowsWhenALogIsEmitted(): void
         $process(['channel' => 'test', 'level_name' => 'warning']);
         self::assertTrue($process->hasBeenCalled());
     }
+
+    public function testItKnowsWhenAErrorIsEmitted(): void
+    {
+        $process = new SpyProcessor();
+        $process(['channel' => 'test', 'level_name' => 'error']);
+        self::assertTrue($process->hasBeenCalled());
+    }
+
+    public function testIsNotCalledWhenLevelIsTolow(): void
+    {
+        $process = new SpyProcessor(LogLevel::ERROR);
+        $process(['channel' => 'test', 'level_name' => 'warning']);
+        self::assertFalse($process->hasBeenCalled());
+    }
 }