Support disabling slack route dynamically when using `SlackWebhookCha…

…nnel` (#87)

* Support disabling slack route dynamically

* Fix formatting

* Add test case

src/SlackNotificationRouterChannel.php

@@ -38,6 +38,10 @@ public function send($notifiable, Notification $notification)
     {
         $route = $notifiable->routeNotificationFor('slack', $notification);
 
+        if ($route === false) {
+            return;
+        }
+
         return $this->determineChannel($route)->send($notifiable, $notification);
     }
 

tests/SlackNotificationRouterChannelTest.php

@@ -61,6 +61,22 @@ public function it_routes_the_notification_to_the_web_api_channel_when_the_notif
 
         $channel->send(new SlackChannelTestNotifiable('#general'), new SlackChannelTestNotification());
     }
+
+    /** @test */
+    public function it_stops_sending_when_the_notifiable_route_is_false(): void
+    {
+        $app = new Container();
+        $app->bind(SlackWebhookChannel::class, fn () => new FakeSlackChannel(function () {
+            $this->fail('The Slack Webhook Channel should not have been called.');
+        }));
+        $app->bind(SlackWebApiChannel::class, fn () => new FakeSlackChannel(function () {
+            $this->fail('The Slack WebAPI Channel should not have been called.');
+        }));
+
+        $channel = new SlackNotificationRouterChannel($app);
+
+        $this->assertEquals(null, $channel->send(new SlackChannelTestNotifiable(false), new SlackChannelTestNotification()));
+    }
 }
 
 class FakeSlackChannel