refactor SmsApiMessage to remove commented code

Author: WebXScripts

src/Exceptions/MissingPhoneNumberException.php

@@ -21,4 +21,4 @@ public static function forAnonymousNotifiable(): self
             'Make sure you are using Notification::route(\'smsapi\', $phoneNumber) correctly.'
         );
     }
-}
+}

src/SmsApiChannel.php

@@ -81,11 +81,11 @@ private function getPhoneFromAnonymousNotifiable(AnonymousNotifiable $notifiable
     {
         $routes = $notifiable->routes ?? [];
 
-        if (!empty($routes['smsapi'])) {
+        if (! empty($routes['smsapi'])) {
             return $routes['smsapi'];
         }
 
-        if (!empty($routes['sms'])) {
+        if (! empty($routes['sms'])) {
             return $routes['sms'];
         }
 
@@ -134,7 +134,7 @@ private function applyMessageProperties(SendSmsBag $sms, SmsApiMessage $message)
         $properties = [
             'encoding', 'test', 'fast', 'normalize', 'noUnicode', 'single',
             'notifyUrl', 'expirationDate', 'timeRestriction', 'partnerId',
-            'checkIdx', 'idx', 'template', 'param1', 'param2', 'param3', 'param4'
+            'checkIdx', 'idx', 'template', 'param1', 'param2', 'param3', 'param4',
         ];
 
         foreach ($properties as $property) {
@@ -143,4 +143,4 @@ private function applyMessageProperties(SendSmsBag $sms, SmsApiMessage $message)
             }
         }
     }
-}
+}

src/SmsApiMessage.php

@@ -153,4 +153,4 @@ private function withProperty(string $property, mixed $value): self
 
         return new self(...$args);
     }
-}
+}

tests/Feature/RealSmsTest.php

@@ -15,7 +15,8 @@
     config(['smsapi.service' => env('SMSAPI_SERVICE')]);
     config(['smsapi.from' => env('SMSAPI_FROM')]);
 
-    $notification = new class extends Notification {
+    $notification = new class extends Notification
+    {
         public function via($notifiable): array
         {
             return ['smsapi'];
@@ -24,8 +25,7 @@ public function via($notifiable): array
         public function toSmsApi($notifiable): SmsApiMessage
         {
             return SmsApiMessage::create('test - ' . now()->format('H:i:s'))
-                ->from('getmed')
-                ->test(true);
+                ->test();
         }
     };
 
@@ -38,7 +38,7 @@ public function toSmsApi($notifiable): SmsApiMessage
 });
 
 it('can send real sms with confirmation', function () {
-    if (!env('SEND_REAL_SMS', false)) {
+    if (! env('SEND_REAL_SMS', false)) {
         test()->markTestSkipped('Set SEND_REAL_SMS=true to send real SMS');
     }
 
@@ -50,7 +50,8 @@ public function toSmsApi($notifiable): SmsApiMessage
     config(['smsapi.service' => env('SMSAPI_SERVICE')]);
     config(['smsapi.from' => env('SMSAPI_FROM')]);
 
-    $notification = new class extends Notification {
+    $notification = new class extends Notification
+    {
         public function via($notifiable): array
         {
             return ['smsapi'];
@@ -59,7 +60,6 @@ public function via($notifiable): array
         public function toSmsApi($notifiable): SmsApiMessage
         {
             return SmsApiMessage::create('test - ' . now()->format('H:i:s'))
-                ->from('getmed')
                 ->test(false);
         }
     };
@@ -70,4 +70,4 @@ public function toSmsApi($notifiable): SmsApiMessage
         ->notify($notification);
 
     expect(true)->toBeTrue();
-});
+});

tests/TestCase.php

@@ -45,4 +45,4 @@ private function loadEnvironmentVariables(): void
             }
         }
     }
-}
+}