Merge pull request #1942 from hydephp/update-file-not-found-exception…

…-to-support-appending-strings [2.x] Update FileNotFoundException to support appending strings
hydephp · Aug 20, 2024 · 53cee6d · 53cee6d
2 parents 3e1cf4b + 5a569b3
commit 53cee6d
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/packages/framework/src/Framework/Exceptions/FileNotFoundException.php b/packages/framework/src/Framework/Exceptions/FileNotFoundException.php
@@ -17,8 +17,8 @@ class FileNotFoundException extends Exception
     /** @var int */
     protected $code = 404;
 
-    public function __construct(?string $path = null, ?string $customMessage = null)
+    public function __construct(?string $path = null, ?string $customMessage = null, ?string $appendAfterPath = '.')
     {
-        parent::__construct($customMessage ?? ($path ? sprintf('File [%s] not found.', Hyde::pathToRelative($path)) : $this->message));
+        parent::__construct($customMessage ?? ($path ? sprintf('File [%s] not found'.$appendAfterPath, Hyde::pathToRelative($path)) : $this->message));
     }
 }
diff --git a/packages/framework/tests/Unit/CustomExceptionsTest.php b/packages/framework/tests/Unit/CustomExceptionsTest.php
@@ -62,6 +62,11 @@ public function testFileNotFoundExceptionWithCustomPath()
         $this->assertSame('foo', (new FileNotFoundException(customMessage: 'foo'))->getMessage());
     }
 
+    public function testFileNotFoundExceptionWithAppendAfterPath()
+    {
+        $this->assertSame('File [foo] not found!', (new FileNotFoundException('foo', null, '!'))->getMessage());
+    }
+
     public function testRouteNotFoundExceptionWithDefaultMessage()
     {
         $this->assertSame('Route not found.', (new RouteNotFoundException())->getMessage());