Add pint and run it

Author: christophrumpel

.scrutinizer.yml

@@ -20,9 +20,10 @@
         "ext-fileinfo": "*"
     },
     "require-dev": {
+        "laravel/pint": "^1.10",
         "mockery/mockery": "^1.3.1",
-        "phpunit/phpunit": "^9.3",
-        "orchestra/testbench": "^8.0"
+        "orchestra/testbench": "^8.0",
+        "phpunit/phpunit": "^9.3"
     },
     "autoload": {
         "psr-4": {

.styleci.yml

@@ -11,7 +11,7 @@ public static function serviceRespondsNotSuccessful(mixed $response): CouldNotSe
         if (isset($response->error)) {
             return new static("Couldn't post notification. Response: ".$response->error);
         }
-     
+
         $responseBody = print_r($response->detail, true);
 
         return new static("Couldn't post notification. Response: ".$responseBody);

composer.json

@@ -15,7 +15,7 @@ public function __construct(protected TwitterOAuth $twitter)
     /**
      * Send the given notification.
      *
-     * @param mixed $notifiable Should be an object that uses the Illuminate\Notifications\Notifiable trait.
+     * @param  mixed  $notifiable Should be an object that uses the Illuminate\Notifications\Notifiable trait.
      *
      * @throws CouldNotSendNotification
      */
@@ -40,7 +40,7 @@ public function send($notifiable, Notification $notification): array|object
             $requestBody,
             $twitterMessage->isJsonRequest,
         );
-      
+
         if ($this->twitter->getLastHttpCode() !== 201) {
             throw CouldNotSendNotification::serviceRespondsNotSuccessful($this->twitter->getLastBody());
         }
@@ -51,7 +51,7 @@ public function send($notifiable, Notification $notification): array|object
     /**
      * Use per user settings instead of default ones.
      *
-     * @param object $notifiable Provide an object that uses the Illuminate\Notifications\Notifiable trait.
+     * @param  object  $notifiable Provide an object that uses the Illuminate\Notifications\Notifiable trait.
      */
     private function changeTwitterSettingsIfNeeded(object $notifiable)
     {

src/Exceptions/CouldNotSendNotification.php

@@ -21,7 +21,7 @@ public function boot()
                     config('services.twitter.access_token'),
                     config('services.twitter.access_secret')
                 );
-            
+
                 return $connection;
             });
     }

src/TwitterChannel.php

@@ -9,9 +9,13 @@
 class TwitterStatusUpdate extends TwitterMessage
 {
     public ?Collection $imageIds = null;
+
     public ?Collection $videoIds = null;
+
     private ?array $images = null;
+
     private ?array $videos = null;
+
     private ?int $inReplyToTweetId = null;
 
     /**
@@ -90,9 +94,6 @@ public function inReplyTo(int $tweetId): self
         return $this;
     }
 
-    /**
-     * @return int|null
-     */
     public function getInReplyToTweetId(): ?int
     {
         return $this->inReplyToTweetId;
@@ -112,7 +113,7 @@ public function getRequestBody(): array
 
         if ($mediaIds->count() > 0) {
             $body['media'] = [
-                'media_ids' => $mediaIds->toArray()
+                'media_ids' => $mediaIds->toArray(),
             ];
         }
 

src/TwitterServiceProvider.php

@@ -42,7 +42,7 @@ public function it_can_send_a_status_update_notification()
         $this->twitter->shouldReceive('getLastHttpCode')
             ->once()
             ->andReturn(201);
-        
+
         $this->channel->send(new TestNotifiable(), new TestNotification());
     }
 
@@ -60,7 +60,7 @@ public function it_can_send_a_status_update_notification_with_images()
             ->once()
             ->with(
                 'tweets',
-                ['text' => 'Laravel Notification Channels are awesome!', 'media' => [ 'media_ids' => [2]]],
+                ['text' => 'Laravel Notification Channels are awesome!', 'media' => ['media_ids' => [2]]],
                 true
             )
             ->andReturn([]);
@@ -96,7 +96,7 @@ public function it_can_send_a_status_update_notification_with_videos()
             ->once()
             ->with(
                 'tweets',
-                ['text' => 'Laravel Notification Channels are awesome!', 'media' => [ 'media_ids' => [2]]],
+                ['text' => 'Laravel Notification Channels are awesome!', 'media' => ['media_ids' => [2]]],
                 true
             )
             ->andReturn([]);
@@ -195,7 +195,7 @@ public function it_throws_an_exception_when_it_could_not_send_the_notification_w
             ->once()
             ->with($media->media_id_string)
             ->andReturn($status);
- 
+
         $this->expectException(CouldNotSendNotification::class);
 
         $this->channel->send(new TestNotifiable(), new TestNotificationWithVideo());
@@ -265,18 +265,11 @@ class TestNotificationWithReplyToStatusId extends Notification
 {
     private int $replyToStatusId;
 
-    /**
-     * @param  int  $replyToStatusId
-     */
     public function __construct(int $replyToStatusId)
     {
         $this->replyToStatusId = $replyToStatusId;
     }
 
-    /**
-     * @param  mixed  $notifiable
-     * @return TwitterMessage
-     */
     public function toTwitter(mixed $notifiable): TwitterMessage
     {
         return (new TwitterStatusUpdate('Laravel Notification Channels are awesome!'))

src/TwitterStatusUpdate.php

@@ -9,7 +9,9 @@
 class TwitterDirectMessageTest extends TestCase
 {
     protected TwitterDirectMessage $messageWithUserId;
+
     protected TwitterDirectMessage $messageWithScreenName;
+
     protected TwitterOAuth $twitter;
 
     public function setUp(): void

tests/TwitterChannelTest.php

@@ -87,10 +87,10 @@ public function it_constructs_a_request_body(): void
         $message->videoIds = collect([534, 535, 536]);
 
         $this->assertEquals([
-            'text'    => 'myMessage',
+            'text' => 'myMessage',
             'media' => [
-                'media_ids' => [434,435,436,534,535,536]
-            ]
+                'media_ids' => [434, 435, 436, 534, 535, 536],
+            ],
         ], $message->getRequestBody());
     }