Merge pull request #7 from cesargb/exception

feat: add exception RotationFailed

Author: cesargb

README.md

@@ -39,13 +39,14 @@ Or you can define the options in the constructor
 
 ```php
 use Cesargb\Log\Rotation;
+use Cesargb\Log\Exceptions\RotationFailed;
 
 $rotation = new Rotation([
     'files' => 1,
     'compress' => true,
     'min-size' => 10,
-    'then' => function ($filenameTarget, $filenameRotated) {},
-    'catch' => function ($error) {},
+    'then' => function ($filename) {},
+    'catch' => function (RotationFailed $exception) {},
 ]);
 
 $rotation->rotate('file.log');

src/ErrorHandler.php

@@ -2,12 +2,15 @@
 
 namespace Cesargb\Log;
 
+use Cesargb\Log\Exceptions\RotationFailed;
 use Throwable;
 
 trait ErrorHandler
 {
     private $catchCallable = null;
 
+    private ?string $_filename = null;
+
     /**
      * Call function if roteted catch any Exception.
      */
@@ -18,14 +21,29 @@ public function catch(callable $callable): self
         return $this;
     }
 
+    protected function setFilename(string $filename): void
+    {
+        $this->_filename = $filename;
+    }
+
     protected function exception(Throwable $exception): self
     {
         if ($this->catchCallable) {
-            call_user_func($this->catchCallable, $exception);
+            call_user_func($this->catchCallable, $this->convertException($exception));
         } else {
-            throw $exception;
+            throw $this->convertException($exception);
         }
 
         return $this;
     }
+
+    private function convertException(Throwable $exception): RotationFailed
+    {
+        return new RotationFailed(
+            $exception->getMessage(),
+            $exception->getCode(),
+            $exception->getPrevious(),
+            $this->_filename
+        );
+    }
 }

src/Exceptions/RotationFailed.php

@@ -0,0 +1,23 @@
+<?php
+
+namespace Cesargb\Log\Exceptions;
+
+use Exception;
+use Throwable;
+
+class RotationFailed extends Exception
+{
+    private string $filename;
+
+    public function __construct(string $message = '', int $code = 0, ?Throwable $previus = null, ?string $filename = null)
+    {
+        parent::__construct($message, $code, $previus);
+
+        $this->filename = $filename ?? '';
+    }
+
+    public function getFilename(): ?string
+    {
+        return $this->filename;
+    }
+}

src/Rotation.php

@@ -89,6 +89,8 @@ public function then(callable $callable): self
      */
     public function rotate(string $filename): bool
     {
+        $this->setFilename($filename);
+
         if (!$this->canRotate($filename)) {
             return false;
         }

tests/ErrorHandlerTest.php

@@ -2,15 +2,14 @@
 
 namespace Cesargb\Log\Test;
 
-use Exception;
+use Cesargb\Log\Exceptions\RotationFailed;
 use Cesargb\Log\Rotation;
-use Cesargb\Log\Test\TestCase;
 
 class ErrorHandlerTest extends TestCase
 {
-    public function test_throws_exception()
+    public function testThrowsException()
     {
-        $this->expectException(Exception::class);
+        $this->expectException(RotationFailed::class);
 
         $rotation = new Rotation();
 
@@ -19,13 +18,16 @@ public function test_throws_exception()
         $this->assertFalse($result);
     }
 
-    public function test_catch_exception()
+    public function testCatchException()
     {
         $rotation = new Rotation();
 
         $result = $rotation
-            ->catch(function ($error) {
-
+            ->catch(function (RotationFailed $exception) {
+                $this->assertEquals(
+                    self::DIR_WORK.'file.log',
+                    $exception->getFilename()
+                );
             })
             ->rotate(self::DIR_WORK.'file.log');