feat: Throw exception when template id don't exist on Brevo

.github/workflows/tests.yml

@@ -13,21 +13,28 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        php: ["8.1", "8.2"]
-        laravel: ["^9.0", "^10.0", "^11"]
-        dependency-version: [ prefer-lowest, prefer-stable ]
+        php: [8.1, 8.2]
+        laravel: [9.*, 10.*, 11.*]
+        dependency-version: [prefer-stable]
+        include:
+          - laravel: 11.*
+            testbench: 9.*
+          - laravel: 10.*
+            testbench: 8.*
+          - laravel: 9.*
+            testbench: 7.*
         exclude:
-          - laravel: 11
+          - laravel: 11.*
             php: 8.1
 
     name: P${{ matrix.php }} - L${{ matrix.laravel }} - ${{ matrix.dependency-version }}
 
     steps:
       - name: Checkout code
-        uses: actions/checkout@v2
+        uses: actions/checkout@v3
 
       - name: Cache dependencies
-        uses: actions/cache@v2
+        uses: actions/cache@v3
         with:
           path: ~/.composer/cache/files
           key: dependencies-laravel-${{ matrix.laravel }}-php-${{ matrix.php }}-composer-${{ hashFiles('composer.json') }}
@@ -36,12 +43,13 @@ jobs:
         uses: shivammathur/setup-php@v2
         with:
           php-version: ${{ matrix.php }}
-          extensions: dom, curl, libxml, mbstring, zip, pcntl, pdo, sqlite, pdo_sqlite, mysql, mysqli, pdo_mysql, bcmath, soap, intl, gd, exif, iconv, imagick, fileinfo
+          extensions: dom, curl, libxml, mbstring, zip, pcntl, pdo, sqlite, pdo_sqlite, bcmath, intl, exif
+          tools: composer:v2
           coverage: none
 
       - name: Install dependencies
         run: |
-          composer require "laravel/framework:${{ matrix.laravel }}" --dev --no-interaction --no-update
+          composer require "laravel/framework:${{ matrix.laravel }}" "orchestra/testbench:${{ matrix.testbench }}" --dev --no-interaction --no-update
           composer update --${{ matrix.dependency-version }} --prefer-dist --no-interaction
       - name: Execute tests
         run: vendor/bin/pest

composer.json

@@ -41,7 +41,7 @@
     "require-dev": {
         "ciareis/bypass": "^1.0",
         "laravel/pint": "^1.16",
-        "orchestra/testbench": "^7.22|^8.1",
+        "orchestra/testbench": "^7.44|^8.25|^9.0",
         "pestphp/pest": "^2.34",
         "phpunit/phpunit": "^9.5|^10.5"
     },

src/BrevoEmailChannel.php

@@ -7,6 +7,7 @@
 use Illuminate\Database\Eloquent\Model;
 use Illuminate\Notifications\AnonymousNotifiable;
 use Illuminate\Notifications\Notification;
+use YieldStudio\LaravelBrevoNotifier\Exceptions\BrevoException;
 
 class BrevoEmailChannel
 {

src/BrevoService.php

@@ -6,6 +6,7 @@
 
 use Illuminate\Http\Client\PendingRequest;
 use Illuminate\Support\Facades\Http;
+use YieldStudio\LaravelBrevoNotifier\Exceptions\BrevoException;
 
 class BrevoService
 {
@@ -45,6 +46,13 @@ public function sendEmail(BrevoEmailMessage $email, array $options = []): ?array
         } elseif (array_key_exists('emailFrom', $options)) {
             $email->from($options['emailFrom']);
         }
+
+        $templateResponse = $this->http->get('/smtp/templates/' . $email->templateId);
+
+        if (! $templateResponse->successful()) {
+            throw new BrevoException($templateResponse->toPsrResponse());
+        }
+
         $response = $this->http->post('/smtp/email', $email->toArray());
 
         if (! $response->successful()) {

src/BrevoSmsChannel.php

@@ -7,6 +7,7 @@
 use Illuminate\Database\Eloquent\Model;
 use Illuminate\Notifications\AnonymousNotifiable;
 use Illuminate\Notifications\Notification;
+use YieldStudio\LaravelBrevoNotifier\Exceptions\BrevoException;
 
 class BrevoSmsChannel
 {

src/BrevoException.php → src/Exceptions/BrevoException.php

@@ -2,7 +2,7 @@
 
 declare(strict_types=1);
 
-namespace YieldStudio\LaravelBrevoNotifier;
+namespace YieldStudio\LaravelBrevoNotifier\Exceptions;
 
 use Exception;
 use Psr\Http\Message\ResponseInterface;

tests/BrevoEmailChannelTest.php

@@ -12,7 +12,7 @@
     $mock = $this->mock(BrevoService::class)->shouldReceive('sendEmail')->once();
     $channel = new BrevoEmailChannel($mock->getMock());
 
-    $channel->send(new User(), new class extends Notification
+    $channel->send(new User, new class extends Notification
     {
         public function via()
         {
@@ -21,7 +21,7 @@ public function via()
 
         public function toBrevoEmail(User $notifiable): BrevoEmailMessage
         {
-            return new BrevoEmailMessage();
+            return new BrevoEmailMessage;
         }
     });
 });

tests/BrevoSmsChannelTest.php

@@ -12,7 +12,7 @@
 it('send notification via BrevoChannel should call BrevoService sendSms method', function () {
     $mock = $this->mock(BrevoService::class)->shouldReceive('sendSms')->once();
     $channel = new BrevoSmsChannel($mock->getMock());
-    $channel->send(new User(), new class extends Notification
+    $channel->send(new User, new class extends Notification
     {
         public function via()
         {
@@ -21,7 +21,7 @@ public function via()
 
         public function toBrevoSms(Model $notifiable): BrevoSmsMessage
         {
-            return new BrevoSmsMessage();
+            return new BrevoSmsMessage;
         }
     });
 });