thecodingmachine / laravel-server-monitor-plugin-notification-by-host

src/Notifications/Notifiable.php

Spacing +6 added lines, -6 removed lines

@@ -5,9 +5,9 @@ 
 
 class Notifiable extends \Spatie\ServerMonitor\Notifications\Notifiable
 {
-    public function routeNotificationForMail(): ?array
+    public function routeNotificationForMail(): ? array
     {
-        if($notification = $this->getSpecificConfiguration('mail')) {
+        if ($notification = $this->getSpecificConfiguration('mail')) {
             return $notification;
         }
         return parent::routeNotificationForMail();
@@ -15,7 +15,7 @@ 
 
     public function routeNotificationForSlack(): ?string
     {
-        if($notification = $this->getSpecificConfiguration('slack')) {
+        if ($notification = $this->getSpecificConfiguration('slack')) {
             return $notification;
         }
         return parent::routeNotificationForSlack();
@@ -30,11 +30,11 @@ 
 
     private function getSpecificConfiguration($type) {
         $notifications = $this->getHost()->getCustomProperty('notifications');
-        if($notifications) {
-            if(!isset($notifications['configuration'][$type])) {
+        if ($notifications) {
+            if (!isset($notifications['configuration'][$type])) {
                 return null;
             }
-            if(count($notifications['configuration'][$type]) == 1) {
+            if (count($notifications['configuration'][$type]) == 1) {
                 $configuration = config('server-monitor-plugin-notification-by-host.notifications.channels.'.$type);
                 return $notifications['configuration'][$type][array_keys($configuration)[0]];
             }

src/Notifications/BaseNotificationTrait.php

Spacing +2 added lines, -2 removed lines

@@ -16,12 +16,12 @@
         /* @var $host \Spatie\ServerMonitor\Models\Host */
         $host = $event->check->host()->first();
         $notifications = $host->getCustomProperty('notifications');
-        if(!$notifications) {
+        if (!$notifications) {
             return config('server-monitor.notifications.notifications.'.static::class);
         }
         var_dump($notifications);
         var_dump(static::class);
-        if(isset($notifications[static::class]) && isset($notifications[static::class]['channels'])) {
+        if (isset($notifications[static::class]) && isset($notifications[static::class]['channels'])) {
             return $notifications[static::class]['channels'];
         }
         return [];

src/Commands/AddNotificationByHost.php

Spacing +12 added lines, -12 removed lines

@@ -37,17 +37,17 @@ 
         $configure = [];
         foreach ($notifications as $channel) {
             $attributes = config('server-monitor-plugin-notification-by-host.notifications.channels.'.$channel);
-            if($attributes) {
+            if ($attributes) {
                 foreach ($attributes as $attribute => $type) {
-                    $configure[$channel][$attribute] = $this->ask("For channel `$channel` attribute `$attribute`?".($type == 'array'?" (values separate by ,)":""), false);
-                    if($type == 'array') {
+                    $configure[$channel][$attribute] = $this->ask("For channel `$channel` attribute `$attribute`?".($type == 'array' ? " (values separate by ,)" : ""), false);
+                    if ($type == 'array') {
                         $configure[$channel][$attribute] = explode(',', $configure[$channel][$attribute]);
                     }
-                    if(!$configure[$channel][$attribute]) {
+                    if (!$configure[$channel][$attribute]) {
                         unset($configure[$channel][$attribute]);
                     }
                 }
-                if(!$configure[$channel]) {
+                if (!$configure[$channel]) {
                     unset($configure[$channel]);
                 }
             }
@@ -68,10 +68,10 @@ 
                 : $this->choice('Which host?', $hostList);
 
             $host = $this->determineHostModelClass()::where('name', $hostName)->first();
-            if(!$host) {
+            if (!$host) {
                 $this->warn("This host `{$hostName}` doesn't exist");
             }
-        } while(!$host);
+        } while (!$host);
 
         $this->info("Host: {$host->name}");
         return $host;
@@ -85,11 +85,11 @@ 
             $notifications[] = $channel;
         }
         $notificationsParameters = [];
-        foreach(config('server-monitor.notifications.notifications') as $class => $notification) {
+        foreach (config('server-monitor.notifications.notifications') as $class => $notification) {
             $this->line("Notification for ".$class);
 
             $channels = $this->choice('Which channel? (values separate by ,)', $notifications, 0, null, true);
-            if($channels[0] != 'No channel') {
+            if ($channels[0] != 'No channel') {
                 $notificationsParameters[$class]['channels'] = $channels;
             }
         }
@@ -97,7 +97,7 @@ 
     }
 
     private function configuration($host, $notificationsParameters) {
-        if(!$notificationsParameters) {
+        if (!$notificationsParameters) {
             $this->warn("The host {$host->name} has no notification configure !");
         }
         else {
@@ -108,12 +108,12 @@ 
 
             $notificationsUsed = [];
             foreach ($notificationsParameters as $check) {
-                if(isset($check['channels']) && $check['channels']) {
+                if (isset($check['channels']) && $check['channels']) {
                     $notificationsUsed = array_unique(array_merge($notificationsUsed, $check['channels']), SORT_REGULAR);
                 }
             }
 
-            if($notificationsUsed) {
+            if ($notificationsUsed) {
                 $configure = $this->askConfiguration($notificationsUsed);
                 $notificationsParameters['configuration'] = $configure;
             }

Braces +2 added lines, -4 removed lines

@@ -50,8 +50,7 @@ 
                 if(!$configure[$channel]) {
                     unset($configure[$channel]);
                 }
-            }
-            else {
+            } else {
                 $this->error("No configuration for ".$channel." in server-monitor-plugin-notification-by-host.php file configuration");
             }
         }
@@ -99,8 +98,7 @@ 
     private function configuration($host, $notificationsParameters) {
         if(!$notificationsParameters) {
             $this->warn("The host {$host->name} has no notification configure !");
-        }
-        else {
+        } else {
 
             $this->line("----- Notification configuration -----");
 

tests/Notifications/NotifiableTest.php

Indentation +2 added lines, -2 removed lines

@@ -42,8 +42,8 @@
 
     /** @test */
     function route_notification_for_mail() {
-       $mail = $this->notifiable->routeNotificationForMail();
-       $this->assertSame('original@test.com', $mail[0]);
+        $mail = $this->notifiable->routeNotificationForMail();
+        $this->assertSame('original@test.com', $mail[0]);
     }
 
     /** @test */

Spacing +9 added lines, -9 removed lines

@@ -29,7 +29,7 @@ 
             'port' => 22,
         ]);
 
-        $host->checks()->saveMany(collect($checks)->map(function (string $checkName) {
+        $host->checks()->saveMany(collect($checks)->map(function(string $checkName) {
             return new Check([
                 'type' => $checkName,
                 'status' => CheckStatus::NOT_YET_CHECKED,
@@ -50,12 +50,12 @@ 
     function route_notification_for_mail_custom() {
         /* @var $host Host */
         $host = $this->notifiable->getEvent()->check->host()->first();
-        $notifications =  [
+        $notifications = [
             'TheCodingMachine\ServerMonitorPluginNotificationByHost\TheCodingMachine\ServerMonitorPluginNotificationByHost\Notifications\Notifications\CheckSucceeded' =>
                 [
                     'channels' => ['mail']
                 ],
-            'configuration' => ['mail' => [ 'to' => ['test@test.com'] ]]
+            'configuration' => ['mail' => ['to' => ['test@test.com']]]
         ];
 
         $host->setCustomProperty('notifications', $notifications);
@@ -70,12 +70,12 @@ 
     function route_notification_no_custom_data() {
         /* @var $host Host */
         $host = $this->notifiable->getEvent()->check->host()->first();
-        $notifications =  [
+        $notifications = [
             'TheCodingMachine\ServerMonitorPluginNotificationByHost\TheCodingMachine\ServerMonitorPluginNotificationByHost\Notifications\Notifications\CheckFailed' =>
                 [
                     'channels' => ['slack']
                 ],
-            'configuration' => ['slack' => [ 'webhook' => ['urlSlack'] ]]
+            'configuration' => ['slack' => ['webhook' => ['urlSlack']]]
         ];
 
         $host->setCustomProperty('notifications', $notifications);
@@ -90,12 +90,12 @@ 
     function route_notification_for_mail_custom_array() {
         /* @var $host Host */
         $host = $this->notifiable->getEvent()->check->host()->first();
-        $notifications =  [
+        $notifications = [
             'TheCodingMachine\ServerMonitorPluginNotificationByHost\Test\TheCodingMachine\ServerMonitorPluginNotificationByHost\Notifications\Notifications\CheckSucceeded' =>
                 [
                     'channels' => ['mail']
                 ],
-            'configuration' => ['mail' => [ 'to' => ['test@test.com'], 'other' => 'otherData']]
+            'configuration' => ['mail' => ['to' => ['test@test.com'], 'other' => 'otherData']]
         ];
 
         $host->setCustomProperty('notifications', $notifications);
@@ -118,12 +118,12 @@ 
     function route_notification_for_slack_custom() {
         /* @var $host Host */
         $host = $this->notifiable->getEvent()->check->host()->first();
-        $notifications =  [
+        $notifications = [
             'TheCodingMachine\ServerMonitorPluginNotificationByHost\Test\TheCodingMachine\ServerMonitorPluginNotificationByHost\Notifications\Notifications\CheckSucceeded' =>
                 [
                     'channels' => ['slack']
                 ],
-            'configuration' => ['slack' => [ 'webhook_url' => 'urlSlack' ]]
+            'configuration' => ['slack' => ['webhook_url' => 'urlSlack']]
         ];
 
         $host->setCustomProperty('notifications', $notifications);

tests/Notifications/BaseNotificationTraitTest.php

Spacing +7 added lines, -7 removed lines

@@ -33,7 +33,7 @@ 
             'port' => 22,
         ]);
 
-        $host->checks()->saveMany(collect($checks)->map(function (string $checkName) {
+        $host->checks()->saveMany(collect($checks)->map(function(string $checkName) {
             return new Check([
                 'type' => $checkName,
                 'status' => CheckStatus::NOT_YET_CHECKED,
@@ -58,7 +58,7 @@ 
             'port' => 22,
         ]);
 
-        $host->checks()->saveMany(collect($checks)->map(function (string $checkName) {
+        $host->checks()->saveMany(collect($checks)->map(function(string $checkName) {
             return new Check([
                 'type' => $checkName,
                 'status' => CheckStatus::NOT_YET_CHECKED,
@@ -69,12 +69,12 @@ 
         $this->notifiable->setEvent($event);
 
         $host = $this->notifiable->getEvent()->check->host()->first();
-        $notifications =  [
+        $notifications = [
             'TheCodingMachine\ServerMonitorPluginNotificationByHost\Notifications\Notifications\CheckSucceeded' =>
                 [
                     'channels' => ['mail']
                 ],
-            'configuration' => ['mail' => [ 'to' => ['test@test.com'] ]]
+            'configuration' => ['mail' => ['to' => ['test@test.com']]]
         ];
 
         $host->setCustomProperty('notifications', $notifications);
@@ -96,7 +96,7 @@ 
             'port' => 22,
         ]);
 
-        $host->checks()->saveMany(collect($checks)->map(function (string $checkName) {
+        $host->checks()->saveMany(collect($checks)->map(function(string $checkName) {
             return new Check([
                 'type' => $checkName,
                 'status' => CheckStatus::NOT_YET_CHECKED,
@@ -107,12 +107,12 @@ 
         $this->notifiable->setEvent($event);
 
         $host = $this->notifiable->getEvent()->check->host()->first();
-        $notifications =  [
+        $notifications = [
             'TheCodingMachine\ServerMonitorPluginNotificationByHost\Notifications\Notifications\CheckFailed' =>
                 [
                     'channels' => ['mail']
                 ],
-            'configuration' => ['mail' => [ 'to' => ['test@test.com'] ]]
+            'configuration' => ['mail' => ['to' => ['test@test.com']]]
         ];
 
         $host->setCustomProperty('notifications', $notifications);

tests/TestCase.php

Spacing +4 added lines, -4 removed lines

@@ -53,7 +53,7 @@ 
         ]);
 
         /* @var $app['config'] Repository */
-        $app['config']->set(['server-monitor' => include(__DIR__ . '/../vendor/spatie/laravel-server-monitor/config/server-monitor.php')]);
+        $app['config']->set(['server-monitor' => include(__DIR__.'/../vendor/spatie/laravel-server-monitor/config/server-monitor.php')]);
 
         $app['config']->set('server-monitor.notifications.notifications', [
             \TheCodingMachine\ServerMonitorPluginNotificationByHost\Notifications\Notifications\CheckSucceeded::class => [],
@@ -66,7 +66,7 @@ 
         $app['config']->set('server-monitor.notifications.slack', ['webhook_url' => 'test']);
         $app['config']->set('server-monitor.notifications.notifiable', TheCodingMachine\ServerMonitorPluginNotificationByHost\Notifications\Notifiable::class);
 
-        $app['config']->set(['server-monitor-plugin-notification-by-host' => include(__DIR__ . '/../config/server-monitor-plugin-notification-by-host.php')]);
+        $app['config']->set(['server-monitor-plugin-notification-by-host' => include(__DIR__.'/../config/server-monitor-plugin-notification-by-host.php')]);
 
         $this->setUpDatabase();
     }
@@ -86,7 +86,7 @@ 
      */
     protected function seeInConsoleOutput($searchStrings)
     {
-        if (! is_array($searchStrings)) {
+        if (!is_array($searchStrings)) {
             $searchStrings = [$searchStrings];
         }
         $output = $this->getArtisanOutput();
@@ -100,7 +100,7 @@ 
      */
     protected function dontSeeInConsoleOutput($searchStrings)
     {
-        if (! is_array($searchStrings)) {
+        if (!is_array($searchStrings)) {
             $searchStrings = [$searchStrings];
         }
         $output = $this->getArtisanOutput();

tests/Commands/AddNotificationByHostTest.php

Spacing +6 added lines, -6 removed lines

@@ -28,27 +28,27 @@
             {
                 echo $question."\n";
                 $result = $this->answers[$this->answer];
-                if($multiple) {
-                    if(!is_array($this->answers[$this->answer])) {
+                if ($multiple) {
+                    if (!is_array($this->answers[$this->answer])) {
                         $result = [$this->answers[$this->answer]];
                     }
                 }
-                $this->answer ++;
+                $this->answer++;
                 return $result;
             }
             public function confirm($question, $default = false)
             {
                 echo $question."\n";
-                return $this->answers[$this->answer ++];
+                return $this->answers[$this->answer++];
             }
 
             public function ask($question, $default = null) {
                 echo $question."\n";
-                return $this->answers[$this->answer ++];
+                return $this->answers[$this->answer++];
             }
         };
 
-        $this->app->bind('command.server-monitor:add-notification-host', function () {
+        $this->app->bind('command.server-monitor:add-notification-host', function() {
             return $this->command;
         });
     }

tests/Commands/ListNotificationsTest.php

Spacing +6 added lines, -6 removed lines

@@ -28,7 +28,7 @@ 
             }
 
             public function choice($question, array $choices, $default = null, $attempts = null, $multiple = null) {
-                echo $question . "\n";
+                echo $question."\n";
                 $result = $this->answers[$this->answer];
                 if ($multiple) {
                     if (!is_array($this->answers[$this->answer])) {
@@ -40,17 +40,17 @@ 
             }
 
             public function confirm($question, $default = false) {
-                echo $question . "\n";
+                echo $question."\n";
                 return $this->answers[$this->answer++];
             }
 
             public function ask($question, $default = null) {
-                echo $question . "\n";
+                echo $question."\n";
                 return $this->answers[$this->answer++];
             }
         };
 
-        $this->app->bind('command.server-monitor:list-notifications', function () {
+        $this->app->bind('command.server-monitor:list-notifications', function() {
             return $this->command;
         });
     }
@@ -70,7 +70,7 @@ 
     /** @test */
     public function it_check() {
 
-        $notifications =  [
+        $notifications = [
             'TheCodingMachine\ServerMonitorPluginNotificationByHost\Notifications\Notifications\CheckSucceeded' =>
                 [
                     'channels' => ['mail']
@@ -79,7 +79,7 @@ 
                 [
                     'channels' => ['slack']
                 ],
-            'configuration' => ['mail' => [ 'to' => 'test@test.com' ]]
+            'configuration' => ['mail' => ['to' => 'test@test.com']]
             ];
 
         $host = Host::create([