added unit tests

composer.json

@@ -20,13 +20,15 @@
     ],
     "require": {
         "php": "~8.1.0 || ~8.2.0 || ~8.3.0",
+        "laminas/laminas-filter": "^2.37",
         "laminas/laminas-servicemanager": "^3.22",
         "laminas/laminas-validator": "^2.64"
     },
     "require-dev": {
         "phpunit/phpunit": "^10.2",
         "laminas/laminas-coding-standard": "^2.5",
-        "vimeo/psalm": "^5.13"
+        "vimeo/psalm": "^5.13",
+        "ext-curl": "*"
     },
     "autoload": {
         "psr-4": {

log/.gitignore

@@ -0,0 +1,4 @@
+# Ignore everything in this directory
+*
+# Except this file
+!.gitignore

src/Filter/Priority.php

@@ -7,7 +7,6 @@
 use Dot\Log\Exception\InvalidArgumentException;
 use Traversable;
 
-use function ctype_digit;
 use function gettype;
 use function is_array;
 use function is_int;
@@ -34,14 +33,14 @@ public function __construct(iterable|int $priority, ?string $operator = null)
             $operator = $priority['operator'] ?? null;
             $priority = $priority['priority'] ?? null;
         }
-        if (! is_int($priority) && ! ctype_digit($priority)) {
+        if (! is_int($priority)) {
             throw new InvalidArgumentException(sprintf(
                 'Priority must be a number, received "%s"',
                 gettype($priority)
             ));
         }
 
-        $this->priority = (int) $priority;
+        $this->priority = $priority;
         $this->operator = $operator ?? '<=';
     }
 

src/Manager/FilterPluginManager.php

@@ -12,6 +12,7 @@
 use Laminas\ServiceManager\AbstractPluginManager;
 use Laminas\ServiceManager\Exception\InvalidServiceException;
 use Laminas\ServiceManager\Factory\InvokableFactory;
+use Psr\Container\ContainerInterface;
 
 use function gettype;
 use function is_object;
@@ -50,6 +51,11 @@ class FilterPluginManager extends AbstractPluginManager
      */
     protected $sharedByDefault = false;
 
+    public function __construct(ContainerInterface $container, array $config = [])
+    {
+        parent::__construct($container, $config);
+    }
+
     /**
      * Validate the plugin is of the expected type.
      *

src/Manager/FormatterPluginManager.php

@@ -9,6 +9,7 @@
 use Laminas\ServiceManager\AbstractPluginManager;
 use Laminas\ServiceManager\Exception\InvalidServiceException;
 use Laminas\ServiceManager\Factory\InvokableFactory;
+use Psr\Container\ContainerInterface;
 
 use function gettype;
 use function is_object;
@@ -40,6 +41,11 @@ class FormatterPluginManager extends AbstractPluginManager
      */
     protected $sharedByDefault = false;
 
+    public function __construct(ContainerInterface $container, array $config = [])
+    {
+        parent::__construct($container, $config);
+    }
+
     /**
      * Validate the plugin is of the expected type.
      *

src/Manager/ProcessorPluginManager.php

@@ -12,6 +12,7 @@
 use Laminas\ServiceManager\AbstractPluginManager;
 use Laminas\ServiceManager\Exception\InvalidServiceException;
 use Laminas\ServiceManager\Factory\InvokableFactory;
+use Psr\Container\ContainerInterface;
 
 use function gettype;
 use function is_object;
@@ -49,6 +50,11 @@ class ProcessorPluginManager extends AbstractPluginManager
      */
     protected $sharedByDefault = false;
 
+    public function __construct(ContainerInterface $container, array $config = [])
+    {
+        parent::__construct($container, $config);
+    }
+
     /**
      * Validate the plugin is of the expected type.
      *

src/Manager/WriterPluginManager.php

@@ -10,6 +10,7 @@
 use Dot\Log\Writer\WriterInterface;
 use Laminas\ServiceManager\AbstractPluginManager;
 use Laminas\ServiceManager\Exception\InvalidServiceException;
+use Psr\Container\ContainerInterface;
 
 use function gettype;
 use function is_object;
@@ -48,6 +49,11 @@ class WriterPluginManager extends AbstractPluginManager
      */
     protected $sharedByDefault = false;
 
+    public function __construct(ContainerInterface $container, array $config = [])
+    {
+        parent::__construct($container, $config);
+    }
+
     /**
      * Validate the plugin is of the expected type.
      *

src/Processor/RequestId.php

@@ -8,7 +8,7 @@
 
 class RequestId implements ProcessorInterface
 {
-    protected string $identifier;
+    protected ?string $identifier = null;
 
     /**
      * Adds an identifier for the request to the log, unless one has already been set.

src/Writer/Stream.php

@@ -68,7 +68,7 @@ public function __construct(
 
         if (! is_string($streamOrUrl) && ! is_resource($streamOrUrl)) {
             throw new InvalidArgumentException(sprintf(
-                'Resource is not a stream nor a string; received "%s',
+                'Resource is not a stream nor a string; received "%s"',
                 gettype($streamOrUrl)
             ));
         }

test/Factory/WriterFactoryTest.php

@@ -0,0 +1,47 @@
+<?php
+
+declare(strict_types=1);
+
+namespace DotTest\Log\Factory;
+
+use Dot\Log\Factory\WriterFactory;
+use Dot\Log\Writer\AbstractWriter;
+use Dot\Log\Writer\Stream;
+use PHPUnit\Framework\MockObject\Exception;
+use PHPUnit\Framework\MockObject\MockObject;
+use PHPUnit\Framework\TestCase;
+use Psr\Container\ContainerInterface;
+
+class WriterFactoryTest extends TestCase
+{
+    private ContainerInterface|MockObject $container;
+
+    private WriterFactory $subject;
+
+    /**
+     * @throws Exception
+     */
+    protected function setUp(): void
+    {
+        $this->container = $this->createMock(ContainerInterface::class);
+        $this->subject   = new WriterFactory();
+    }
+
+    public function testWillInstantiate(): void
+    {
+        $factory = (new WriterFactory())(
+            $this->container,
+            Stream::class,
+            ['stream' => __DIR__ . '/../../log/dk.log']
+        );
+
+        $this->assertInstanceOf(AbstractWriter::class, $factory);
+    }
+
+    public function testSetCreationOptions(): void
+    {
+        $input = [];
+
+        $this->assertNull($this->subject->setCreationOptions($input));
+    }
+}

test/Filter/PriorityTest.php

@@ -0,0 +1,60 @@
+<?php
+
+declare(strict_types=1);
+
+namespace DotTest\Log\Filter;
+
+use Dot\Log\Exception\InvalidArgumentException;
+use Dot\Log\Filter\Priority;
+use PHPUnit\Framework\TestCase;
+
+class PriorityTest extends TestCase
+{
+    private Priority $subject;
+
+    public function setUp(): void
+    {
+        $this->subject = new Priority(47);
+    }
+
+    public function testWillInstantiateWithInt(): void
+    {
+        $this->assertInstanceOf(Priority::class, $this->subject);
+    }
+
+    public function testWillInstantiateWithArray(): void
+    {
+        $input = ['priority' => 47];
+
+        $result = new Priority($input);
+
+        $this->assertInstanceOf(Priority::class, $result);
+    }
+
+    public function testWillNotInstantiateWithEmptyArray(): void
+    {
+        $input = [];
+
+        $this->expectExceptionMessage('Priority must be a number, received "NULL"');
+        $this->expectException(InvalidArgumentException::class);
+        new Priority($input);
+    }
+
+    public function testFilterWillAcceptMessage(): void
+    {
+        $input = ['priority' => 47];
+
+        $result = $this->subject->filter($input);
+
+        $this->assertTrue($result);
+    }
+
+    public function testFilterWillNotAcceptMessage(): void
+    {
+        $input = ['priority' => 244];
+
+        $result = $this->subject->filter($input);
+
+        $this->assertFalse($result);
+    }
+}

test/Filter/RegexTest.php

@@ -0,0 +1,58 @@
+<?php
+
+declare(strict_types=1);
+
+namespace DotTest\Log\Filter;
+
+use Dot\Log\Filter\Regex;
+use ErrorException;
+use PHPUnit\Framework\TestCase;
+use TypeError;
+
+class RegexTest extends TestCase
+{
+    private Regex $subject;
+
+    /**
+     * @throws ErrorException
+     */
+    public function setUp(): void
+    {
+        $this->subject = new Regex(['regex' => '/(a)(b)*(c)/']);
+    }
+
+    public function testWillInstantiate(): void
+    {
+        $this->assertInstanceOf(Regex::class, $this->subject);
+    }
+
+    /**
+     * @throws ErrorException
+     */
+    public function testWillNotInstantiateWithEmptyArray(): void
+    {
+        $this->expectExceptionMessage(
+            'preg_match(): Argument #1 ($pattern) must be of type string, null given'
+        );
+        $this->expectException(TypeError::class);
+        new Regex([]);
+    }
+
+    public function testFilterWillAcceptMessage(): void
+    {
+        $input = ['message' => 'ac'];
+
+        $result = $this->subject->filter($input);
+
+        $this->assertTrue($result);
+    }
+
+    public function testFilterWillNotAcceptMessage(): void
+    {
+        $input = ['message' => 'What a wonderful day to write tests'];
+
+        $result = $this->subject->filter($input);
+
+        $this->assertFalse($result);
+    }
+}

test/Filter/SuppressFilterTest.php

@@ -0,0 +1,45 @@
+<?php
+
+declare(strict_types=1);
+
+namespace DotTest\Log\Filter;
+
+use Dot\Log\Exception\InvalidArgumentException;
+use Dot\Log\Filter\SuppressFilter;
+use PHPUnit\Framework\TestCase;
+
+class SuppressFilterTest extends TestCase
+{
+    private SuppressFilter $subject;
+
+    public function setUp(): void
+    {
+        $this->subject = new SuppressFilter();
+    }
+
+    public function testWillInstantiate(): void
+    {
+        $this->assertInstanceOf(SuppressFilter::class, $this->subject);
+    }
+
+    public function testWillNotInstantiate(): void
+    {
+        $this->expectExceptionMessage(
+            'Suppress must be a boolean; received "string"'
+        );
+        $this->expectException(InvalidArgumentException::class);
+        new SuppressFilter(['suppress' => 'oups']);
+    }
+
+    public function testFilter(): void
+    {
+        $result = $this->subject->filter([]);
+
+        $this->assertIsBool($result);
+    }
+
+    public function testSuppress(): void
+    {
+        $this->assertNull($this->subject->suppress(true));
+    }
+}