MAGE-1341 Fix unit tests for Monolog v3 on Magento 2.4.8

Author: cammonro

Test/Unit/Logger/AlgoliaLoggerTest.php

@@ -0,0 +1,45 @@
+<?php
+
+namespace Algolia\AlgoliaSearch\Test\Unit\Logger\Handler;
+
+use Magento\Framework\Logger\Handler\Base;
+use PHPUnit\Framework\TestCase;
+
+abstract class AbstractHandlerTest extends TestCase
+{
+    protected ?Base $handler = null;
+
+    /**
+     * Monolog v3 uses LogRecord
+     * v2 uses arrays only
+     */
+    protected function makeLogRecord(int $level, string $message)
+    {
+        if (class_exists(\Monolog\LogRecord::class)) {
+            // Monolog v3
+            $levelEnum = $this->convertLevelToEnum($level);
+            return new \Monolog\LogRecord(
+                new \DateTimeImmutable(),
+                'test',
+                $levelEnum,
+                $message
+            );
+        }
+
+        // Monolog v2
+        return [
+            'message' => $message,
+            'level' => $level
+        ];
+    }
+
+    protected function convertLevelToEnum(int $level): \Monolog\Level
+    {
+        if (class_exists(\Monolog\Level::class)) {
+            return \Monolog\Level::from($level);
+        }
+
+        throw new \RuntimeException('Monolog v3 Level enum not available');
+    }
+
+}

Test/Unit/Logger/Handler/AbstractHandlerTest.php

@@ -0,0 +1,38 @@
+<?php
+
+namespace Algolia\AlgoliaSearch\Test\Unit\Logger\Handler;
+
+use Algolia\AlgoliaSearch\Logger\Handler\AlgoliaLoggerHandler;
+use Magento\Framework\Filesystem\DriverInterface;
+
+class AlgoliaLoggerHandlerTest extends AbstractHandlerTest
+{
+    protected function setUp(): void
+    {
+        $this->handler = new AlgoliaLoggerHandler(
+            $this->createMock(DriverInterface::class)
+        );
+    }
+
+    public function testAlgoliaHandlerLogsEverything(): void
+    {
+        $debugRecord = $this->makeLogRecord(
+            \Monolog\Logger::DEBUG,
+            'Should log'
+        );
+
+        $infoRecord = $this->makeLogRecord(
+            \Monolog\Logger::INFO,
+            'Should log'
+        );
+
+        $errorRecord = $this->makeLogRecord(
+            \Monolog\Logger::ERROR,
+            'Should log'
+        );
+
+        $this->assertTrue($this->handler->isHandling($debugRecord), 'DEBUG should be handled');
+        $this->assertTrue($this->handler->isHandling($infoRecord), 'INFO should be handled');
+        $this->assertTrue($this->handler->isHandling($errorRecord), 'ERROR should be handled');
+    }
+}

Test/Unit/Logger/Handler/AlgoliaLoggerHandlerTest.php

@@ -0,0 +1,37 @@
+<?php
+
+namespace Algolia\AlgoliaSearch\Test\Unit\Logger\Handler;
+
+use Algolia\AlgoliaSearch\Logger\Handler\SystemLoggerHandler;
+use Magento\Framework\Filesystem\DriverInterface;
+use Magento\Framework\Logger\Handler\Exception as ExceptionHandler;
+
+class SystemLoggerHandlerTest extends AbstractHandlerTest
+{
+    /**
+     * @throws \Exception
+     */
+    protected function setUp(): void
+    {
+        $this->handler = new SystemLoggerHandler(
+            $this->createMock(DriverInterface::class),
+            $this->createMock(ExceptionHandler::class),
+        );
+    }
+
+    public function testSystemHandlerFiltersBelowError(): void
+    {
+        $infoRecord = $this->makeLogRecord(
+            \Monolog\Logger::INFO,
+            'Should not log'
+        );
+
+        $errorRecord = $this->makeLogRecord(
+             \Monolog\Logger::ERROR,
+            'Should log'
+        );
+
+        $this->assertFalse($this->handler->isHandling($infoRecord), 'INFO should be ignored');
+        $this->assertTrue($this->handler->isHandling($errorRecord), 'ERROR should be handled');
+    }
+}