fix: add previous Exception for #19 and #20 (#22)

Signed-off-by: Florian Levis <[email protected]>

Author: Gounlaf

Helpers/HttpExceptionFactory.php

@@ -71,18 +71,19 @@ public static function build(int $responseCode, string $message = '', Throwable
      * @param int $responseCode
      * @param mixed $context mixed data you can attach to the exception
      * @param string|null $message
+     * @param Throwable|null $ex [optional] The previous throwable used for the exception chaining.
      * @return HttpExceptionInterface
      *
      * @throws ValidationException When {$responseCode} can't be mapped to an HttpException
      */
-    public static function buildWithContext(int $responseCode, $context, string $message = ''): HttpExceptionInterface
+    public static function buildWithContext(int $responseCode, $context, string $message = '', Throwable $ex = null): HttpExceptionInterface
     {
         $mapping = static::getMapping();
         if (!array_key_exists($responseCode, $mapping)) {
             throw new ValidationException('Unknown mapping for response code ' . $responseCode);
         }
 
-        return $mapping[$responseCode]::withContext($context, $message, $responseCode);
+        return $mapping[$responseCode]::withContext($context, $message, $responseCode, $ex);
     }
 
     /**

Helpers/WithContext.php

@@ -1,4 +1,5 @@
 <?php
+
 namespace Exceptions\Helpers;
 
 /**
@@ -21,15 +22,16 @@ trait WithContext
      * If you do not provide a message and/or code, it will use the getMessage and getCode interface methods to
      * retrieve the defaults for that exception.
      *
-     * @param mixed context to save to the exception
+     * @param $context
      * @param string $message to use, if null, will get the default exception message
      * @param int $code to use, if null, will get the default exception code
+     * @param \Throwable|null $previous [optional] The previous throwable used for the exception chaining.
      *
      * @return static
      */
-    public static function withContext($context, string $message = null, int $code = null)
+    public static function withContext($context, string $message = null, int $code = null, \Throwable $previous = null)
     {
-        $ex = new static($message ?? static::getDefaultMessage(), $code ?? static::getDefaultCode());
+        $ex = new static($message ?? static::getDefaultMessage(), $code ?? static::getDefaultCode(), $previous);
         $ex->context = $context;
         return $ex;
     }

Tests/HttpExceptionFactoryTest.php

@@ -38,9 +38,13 @@ public function testBuildFailed()
     public function testBuildWithContext(int $errorCode, string $expectedClass)
     {
         $ctx = new stdClass();
-        $exception = HttpExceptionFactory::buildWithContext($errorCode, $ctx);
+        $message = 'test';
+        $previousException = new \Exception();
+        $exception = HttpExceptionFactory::buildWithContext($errorCode, $ctx, $message, $previousException);
         $this->assertInstanceOf($expectedClass, $exception);
         $this->assertSame($ctx, $exception->getContext());
+        $this->assertSame('test', $exception->getMessage());
+        $this->assertSame($previousException, $exception->getPrevious());
     }
 
     public function testBuildWithContextFailed()

composer.json

@@ -29,10 +29,7 @@
   },
   "config": {
     "sort-packages": true,
-    "optimize-autoloader": true,
-    "platform": {
-      "php": "7.3"
-    }
+    "optimize-autoloader": true
   },
   "scripts": {
     "test": "vendor/bin/phpunit",