Merge branch 'v3' of github.com:laravel-notification-channels/cmsms into v3

Author: mbardelmeijer

composer.json

@@ -19,9 +19,10 @@
         "illuminate/support": "^9.0"
     },
     "require-dev": {
+        "friendsofphp/php-cs-fixer": "^3.6",
         "mockery/mockery": "^1.5",
-        "phpunit/phpunit": "^9.5",
-        "orchestra/testbench": "^7.0"
+        "orchestra/testbench": "^7.0",
+        "phpunit/phpunit": "^9.5"
     },
     "config": {
         "sort-packages": true
@@ -46,6 +47,8 @@
     "minimum-stability": "dev",
     "prefer-stable": true,
     "scripts": {
-        "test": "vendor/bin/phpunit"
+        "format": "vendor/bin/php-cs-fixer fix --allow-risky=yes",
+        "test": "vendor/bin/phpunit",
+        "test-coverage": "vendor/bin/phpunit --coverage-html coverage"
     }
 }

src/CmsmsChannel.php

@@ -10,13 +10,12 @@ class CmsmsChannel
 {
     public function __construct(
         protected CmsmsClient $client,
-    )
-    {
+    ) {
     }
 
     public function send($notifiable, Notification $notification): void
     {
-        if (!$recipient = $notifiable->routeNotificationFor('Cmsms')) {
+        if (! $recipient = $notifiable->routeNotificationFor('Cmsms')) {
             return;
         }
 

src/CmsmsClient.php

@@ -16,7 +16,7 @@ class CmsmsClient
     public function __construct(
         protected GuzzleClient $client,
         protected string $productToken,
-    )    {
+    ) {
     }
 
     public function send(CmsmsMessage $message, string $recipient): void
@@ -26,7 +26,7 @@ public function send(CmsmsMessage $message, string $recipient): void
         }
 
         $response = $this->client->request('POST', static::GATEWAY_URL, [
-            'body'    => $this->buildMessageXml($message, $recipient),
+            'body' => $this->buildMessageXml($message, $recipient),
             'headers' => [
                 'Content-Type' => 'application/xml',
             ],
@@ -35,7 +35,7 @@ public function send(CmsmsMessage $message, string $recipient): void
         // API returns an empty string on success
         // On failure, only the error string is passed
         $body = $response->getBody()->getContents();
-        if (!empty($body)) {
+        if (! empty($body)) {
             throw CouldNotSendNotification::serviceRespondedWithAnError($body);
         }
     }

src/CmsmsMessage.php

@@ -20,8 +20,7 @@ class CmsmsMessage
 
     private function __construct(
         protected string $body = ''
-    )
-    {
+    ) {
         $this->body($body);
     }
 
@@ -45,7 +44,7 @@ public function originator(string|int $originator): self
 
     public function reference(string $reference): self
     {
-        if (empty($reference) || strlen($reference) > 32 || !ctype_alnum($reference)) {
+        if (empty($reference) || strlen($reference) > 32 || ! ctype_alnum($reference)) {
             throw InvalidMessage::invalidReference($reference);
         }
 
@@ -81,9 +80,9 @@ public function multipart(int $minimum, int $maximum): self
     public function toXmlArray(): array
     {
         return array_filter([
-            'BODY'                        => $this->body,
-            'FROM'                        => $this->originator,
-            'REFERENCE'                   => $this->reference,
+            'BODY' => $this->body,
+            'FROM' => $this->originator,
+            'REFERENCE' => $this->reference,
             'MINIMUMNUMBEROFMESSAGEPARTS' => $this->minimumNumberOfMessageParts,
             'MAXIMUMNUMBEROFMESSAGEPARTS' => $this->maximumNumberOfMessageParts,
         ]);