Merge pull request #1 from onlime/121-enabled-config

Add enabled config

config/twilio-notification-channel.php

@@ -1,6 +1,7 @@
 <?php
 
 return [
+    'enabled' => env('TWILIO_ENABLED', true),
     'username' => env('TWILIO_USERNAME'), // optional when using auth token
     'password' => env('TWILIO_PASSWORD'), // optional when using auth token
     'auth_token' => env('TWILIO_AUTH_TOKEN'), // optional when using username and password

src/TwilioChannel.php

@@ -43,6 +43,10 @@ public function __construct(Twilio $twilio, Dispatcher $events)
      */
     public function send($notifiable, Notification $notification)
     {
+        if (! $this->isEnabled()) {
+            return;
+        }
+
         try {
             $to = $this->getTo($notifiable, $notification);
             $message = $notification->toTwilio($notifiable);
@@ -75,6 +79,20 @@ public function send($notifiable, Notification $notification)
         }
     }
 
+    /**
+     * Check if twilio is enabled.
+     *
+     * @return bool
+     */
+    protected function isEnabled()
+    {
+        if (is_null($this->twilio->config)) {
+            return true;
+        }
+
+        return $this->twilio->config->enabled();
+    }
+
     /**
      * Get the address to send a notification to.
      *

src/TwilioConfig.php

@@ -15,6 +15,11 @@ public function __construct(array $config)
         $this->config = $config;
     }
 
+    public function enabled()
+    {
+        return $this->config['enabled'] ?? true;
+    }
+
     public function usingUsernamePasswordAuth(): bool
     {
         return $this->getUsername() !== null && $this->getPassword() !== null && $this->getAccountSid() !== null;

tests/Integration/BaseIntegrationTest.php

@@ -5,7 +5,7 @@
 namespace NotificationChannels\Twilio\Tests\Integration;
 
 use NotificationChannels\Twilio\TwilioProvider;
-use Orchestra\Testbench\TestCase;
+use PHPUnit\Framework\TestCase;
 
 abstract class BaseIntegrationTest extends TestCase
 {

tests/Unit/TwilioChannelTest.php

@@ -36,6 +36,23 @@ public function setUp(): void
         $this->channel = new TwilioChannel($this->twilio, $this->dispatcher);
     }
 
+    /** @test */
+    public function it_will_not_send_a_message_if_not_enabled()
+    {
+        $notifiable = new Notifiable();
+        $notification = Mockery::mock(Notification::class);
+
+        $this->twilio->config = new TwilioConfig([
+            'enabled' => false,
+        ]);
+
+        $this->dispatcher->shouldNotReceive('dispatch');
+
+        $result = $this->channel->send($notifiable, $notification);
+
+        $this->assertNull($result);
+    }
+
     /** @test */
     public function it_will_not_send_a_message_without_known_receiver()
     {