refactor: started providing previous exceptions where possible

Author: alexsoft

src/Exceptions/CouldNotSendNotification.php

@@ -6,25 +6,26 @@
 
 use Psr\Http\Message\ResponseInterface;
 use RuntimeException;
+use Throwable;
 
 class CouldNotSendNotification extends RuntimeException
 {
-    public static function pushbulletRespondedWithAnError(ResponseInterface $response): self
+    public static function pushbulletRespondedWithAnError(ResponseInterface $response, Throwable $previous = null): self
     {
         $code = $response->getStatusCode();
 
         $message = $response->getBody();
 
-        return new self("Pushbullet responded with error: `{$code} - {$message}`.");
+        return new self("Pushbullet responded with error: `{$code} - {$message}`.", 0, $previous);
     }
 
     public static function providedEmailIsInvalid(string $email): self
     {
         return new self("Provided email `{$email}` of `notifiable` is not valid.");
     }
 
-    public static function couldNotCommunicateWithPushbullet(): self
+    public static function couldNotCommunicateWithPushbullet(Throwable $previous = null): self
     {
-        return new self('Could not connect to Pushbullet API.');
+        return new self('Could not connect to Pushbullet API.', 0, $previous);
     }
 }

src/Pushbullet.php

@@ -68,9 +68,9 @@ public function send($params): ResponseInterface
                 'headers' => $this->getHeaders(),
             ]);
         } catch (ClientException $exception) {
-            throw CouldNotSendNotification::pushbulletRespondedWithAnError($exception->getResponse());
+            throw CouldNotSendNotification::pushbulletRespondedWithAnError($exception->getResponse(), $exception);
         } catch (Exception $exception) {
-            throw CouldNotSendNotification::couldNotCommunicateWithPushbullet();
+            throw CouldNotSendNotification::couldNotCommunicateWithPushbullet($exception);
         }
     }
 }

tests/Exceptions/CouldNotSendNotificationTest.php

@@ -4,6 +4,7 @@
 
 namespace NotificationChannels\Pushbullet\Test\Exceptions;
 
+use Exception;
 use NotificationChannels\Pushbullet\Exceptions\CouldNotSendNotification;
 use PHPUnit\Framework\TestCase;
 use Psr\Http\Message\ResponseInterface;
@@ -24,12 +25,14 @@ public function invalid_email_exception_can_be_created(): void
     /** @test */
     public function pushbullet_connection_exception_can_be_created(): void
     {
-        $exception = CouldNotSendNotification::couldNotCommunicateWithPushbullet();
+        $previousException = new Exception();
+        $exception = CouldNotSendNotification::couldNotCommunicateWithPushbullet($previousException);
 
         $this->assertEquals(
             'Could not connect to Pushbullet API.',
             $exception->getMessage()
         );
+        $this->assertSame($previousException, $exception->getPrevious());
     }
 
     /** @test */
@@ -45,11 +48,14 @@ public function pushbullet_error_exception_can_be_created(): void
             ->method('getBody')
             ->willReturn('Oops');
 
-        $exception = CouldNotSendNotification::pushbulletRespondedWithAnError($response);
+        $previousException = new Exception();
+
+        $exception = CouldNotSendNotification::pushbulletRespondedWithAnError($response, $previousException);
 
         $this->assertEquals(
             'Pushbullet responded with error: `400 - Oops`.',
             $exception->getMessage()
         );
+        $this->assertSame($previousException, $exception->getPrevious());
     }
 }