Merge pull request #15 from BinarCode/stan

Adding a config whether to enable or not slack.

Author: binaryk

config/developer.php

@@ -12,7 +12,12 @@
     'model' => \Binarcode\LaravelDeveloper\Models\DeveloperLog::class,
 
     /**
-     * The slack incoming webhook to send notifications.
+     * Indicate whether to allow using slack helper.
+     */
+    'slack_dev_enabled' => env('SLACK_DEV_LOG_ENABLED', true),
+
+    /**
+     * The slack webhook to send notifications.
      */
     'slack_dev_hook' => env('SLACK_DEV_HOOK'),
 

src/Notifications/Slack.php

@@ -57,8 +57,12 @@ public function channel(string $channel): self
         return $this;
     }
 
-    private function send($item)
+    private function send($item): self
     {
+        if (! config('developer.slack_dev_enabled')) {
+            return $this;
+        }
+
         /**
          * @var string $class
          */

tests/Helpers/SlackHelperTest.php

@@ -13,7 +13,7 @@
 
 class SlackHelperTest extends TestCase
 {
-    public function test_slack_helper_returns_laravel_developer_instance()
+    public function test_slack_helper_returns_laravel_developer_instance(): void
     {
         Notification::fake();
 
@@ -22,7 +22,7 @@ public function test_slack_helper_returns_laravel_developer_instance()
         $this->assertInstanceOf(Slack::class, slack(new Exception()));
     }
 
-    public function test_slack_helper_can_send_message_to_slack()
+    public function test_slack_helper_can_send_message_to_slack(): void
     {
         Notification::fake();
 
@@ -31,7 +31,7 @@ public function test_slack_helper_can_send_message_to_slack()
         Notification::assertSentTo(new AnonymousNotifiable(), DevNotification::class);
     }
 
-    public function test_slack_helper_can_send_throwable_to_slack()
+    public function test_slack_helper_can_send_throwable_to_slack(): void
     {
         Notification::fake();
 
@@ -54,11 +54,11 @@ public function test_slack_helper_can_send_throwable_to_slack()
             DevNotification::class,
             function (DevNotification $class) use ($uuid) {
                 return $class->notificationDto->attachment_link === "app.test/{$uuid}";
-            }
+            },
         );
     }
 
-    public function test_slack_helper_can_send_notifications_to_slack()
+    public function test_slack_helper_can_send_notifications_to_slack(): void
     {
         Notification::fake();
 
@@ -68,7 +68,7 @@ public function test_slack_helper_can_send_notifications_to_slack()
         ]);
 
         $this->assertInstanceOf(Slack::class, slack(
-            new DummyNotification()
+            new DummyNotification(),
         )->persist());
 
         Notification::assertSentTo(new AnonymousNotifiable(), DummyNotification::class);