Fix styling

Author: irazasyed

src/Commands/Command.php

@@ -202,14 +202,14 @@ private function makeRegexPattern(): array
 
     private function relevantMessageSubString(): string
     {
-        //Get all the bot_command offsets in the Update object
+        // Get all the bot_command offsets in the Update object
         $commandOffsets = $this->allCommandOffsets();
 
         if ($commandOffsets->isEmpty()) {
             return $this->getUpdate()->getMessage()->text ?? '';
         }
 
-        //Extract the current offset for this command and, if it exists, the offset of the NEXT bot_command entity
+        // Extract the current offset for this command and, if it exists, the offset of the NEXT bot_command entity
         $splice = $commandOffsets->splice(
             $commandOffsets->search($this->entity['offset']),
             2

src/Laravel/config/telegram.php

@@ -40,7 +40,7 @@
              */
             'allowed_updates' => null,
             'commands' => [
-                //Acme\Project\Commands\MyTelegramBot\BotCommand::class
+                // Acme\Project\Commands\MyTelegramBot\BotCommand::class
             ],
         ],
 

src/Objects/CallbackQuery.php

@@ -32,7 +32,7 @@ public function relations(): array
 
     public function objectType(): ?string
     {
-        //TODO - Check if message and inline_message_id are exclusive to each other
+        // TODO - Check if message and inline_message_id are exclusive to each other
         return $this->findType(['data', 'game_short_name']);
     }
 }

src/Traits/CommandsHandler.php

@@ -65,7 +65,7 @@ protected function useGetUpdates(): array
             $this->processCommand($update);
         }
 
-        //An update is considered confirmed as soon as getUpdates is called with an offset higher than it's update_id.
+        // An update is considered confirmed as soon as getUpdates is called with an offset higher than it's update_id.
         if ($highestId !== -1) {
             $this->markUpdateAsRead($highestId);
         }

src/Traits/Http.php

@@ -242,7 +242,7 @@ public function getClient(): TelegramClient
      */
     protected function uploadFile(string $endpoint, array $params, string $inputFileField): TelegramResponse
     {
-        //Check if the field in the $params array (that is being used to send the relative file), is a file id.
+        // Check if the field in the $params array (that is being used to send the relative file), is a file id.
         if (! isset($params[$inputFileField])) {
             throw CouldNotUploadInputFile::missingParam($inputFileField);
         }
@@ -251,7 +251,7 @@ protected function uploadFile(string $endpoint, array $params, string $inputFile
             return $this->post($endpoint, $params);
         }
 
-        //Sending an actual file requires it to be sent using multipart/form-data
+        // Sending an actual file requires it to be sent using multipart/form-data
         return $this->post($endpoint, $this->prepareMultipartParams($params, $inputFileField), true);
     }
 
@@ -288,7 +288,7 @@ protected function prepareMultipartParams(array $params, string $inputFileField)
     {
         $this->validateInputFileField($params, $inputFileField);
 
-        //Iterate through all param options and convert to multipart/form-data.
+        // Iterate through all param options and convert to multipart/form-data.
         return collect($params)
             ->reject(static fn ($value): bool => $value === null)
             ->map(fn ($contents, $name) => $this->generateMultipartData($contents, $name))

tests/Integration/Api.php

@@ -88,7 +88,7 @@ function createIncomeWebhookRequestInstance(array $updateData): Request
     /** @var Request $request */
     $request = $this->getHistory()->pluck('request')->first();
 
-    expect($request->getBody())->toEqual('') //'The get request had a body when it should be blank.'
+    expect($request->getBody())->toEqual('') // 'The get request had a body when it should be blank.'
         ->and($request->getUri()->getScheme())->toEqual('https')
         ->and($request->getUri()->getHost())->toEqual('api.telegram.org')
         ->and($request->getUri()->getPath())->toEqual('/botSpecial_Bot_Token/getChatMember')
@@ -211,7 +211,7 @@ function createIncomeWebhookRequestInstance(array $updateData): Request
     ->throws(TelegramSDKException::class);
 
 it('throws an exception if the param key used to upload file does not match the method being used', function () {
-    //We want to send a document but the params have a voice key instead.
+    // We want to send a document but the params have a voice key instead.
     $this->api->sendDocument([
         'chat_id' => 123456789,
         'voice' => InputFile::create(fopen('php://input', 'rb'), 'Myvoice.ogg'),
@@ -248,7 +248,7 @@ function createIncomeWebhookRequestInstance(array $updateData): Request
     $data = [];
     $api = api($this->getGuzzleHttpClient([$this->makeFakeInboundUpdate($data)]));
 
-    //We can use any file input here, for testing a stream is quick and easy.
+    // We can use any file input here, for testing a stream is quick and easy.
     $api->sendDocument([
         'chat_id' => 123456789,
         'document' => InputFile::create(streamFor('Some text'), 'testing.txt'),
@@ -265,8 +265,8 @@ function createIncomeWebhookRequestInstance(array $updateData): Request
 });
 
 it('can set a webhook with its own certificate successfully', function () {
-    //Probably not the best way to attempt to create a file on a server.
-    //Help appreciated.
+    // Probably not the best way to attempt to create a file on a server.
+    // Help appreciated.
     $fakeFile = fopen('php://temp', 'wb+');
     fwrite($fakeFile, pubKey());
     fseek($fakeFile, 0);
@@ -284,7 +284,7 @@ function createIncomeWebhookRequestInstance(array $updateData): Request
         'certificate' => InputFile::create(streamFor(pubKey()), 'public.key'),
     ]);
 
-    //If the user uses just a string to the path/filename of the webhook cert.
+    // If the user uses just a string to the path/filename of the webhook cert.
     $api->setWebhook([
         'url' => 'https://example.com',
         'certificate' => $fakeFile,

tests/Integration/BotsManager.php

@@ -50,7 +50,7 @@
         ->toHaveKey('bot1');
 });
 
-//test('duplicated commands dont cause a problem', function () {
+// test('duplicated commands dont cause a problem', function () {
 //    $manager = new BotsManager(
 //        [
 //            'commands' => [
@@ -125,4 +125,4 @@
 //        ->and($commands03)->toHaveCount(5)
 //        ->and($commands04)->toHaveCount(7)
 //        ->and($commands05)->toHaveCount(8);
-//})->skip('This test is not working properly');
+// })->skip('This test is not working properly');

tests/Unit/Commands/Command.php

@@ -13,7 +13,7 @@
 });
 
 test('a command with no pattern set will return an empty argument array', function () {
-    //Condensed update data
+    // Condensed update data
     $update = new Update([
         'message' => [
             'text' => 'This /demo',
@@ -36,7 +36,7 @@
 });
 
 test('a command with no pattern set but has text after the command will return an empty argument array', function () {
-    //Condensed update data
+    // Condensed update data
     $update = new Update([
         'message' => [
             'text' => '/demo@testing_Bot John Doe',
@@ -59,7 +59,7 @@
 });
 
 test('a command with only required pattern variables is parsed correctly', function () {
-    //Condensed update data
+    // Condensed update data
     $update = new Update([
         'message' => [
             'text' => '/demo@testing_Bot John Doe 77 200',
@@ -81,7 +81,7 @@
 });
 
 test('a command with required and optional pattern variables is parsed correctly', function () {
-    //Condensed update data
+    // Condensed update data
     $update = new Update([
         'message' => [
             'text' => '/demo@testing_Bot John Doe 77 200',
@@ -103,7 +103,7 @@
 });
 
 test('a command with more required pattern variables than exists in update message is not matched', function () {
-    //Condensed update data
+    // Condensed update data
     $update = new Update([
         'message' => [
             'text' => '/demo@testing_Bot John Doe 77',
@@ -130,7 +130,7 @@
 });
 
 test('a command with custom regex set as pattern will return an array with the match value', function () {
-    //Condensed update data
+    // Condensed update data
     $update = new Update([
         'message' => [
             'text' => '/demo@testing_Bot eidw einn egaa egcc',
@@ -152,7 +152,7 @@
 });
 
 test('a command with more advance custom regex set as pattern will return an array with the match value', function () {
-    //Condensed update data
+    // Condensed update data
     $update = new Update([
         'message' => [
             'text' => '/demo@testing_Bot ei105 22/03/2017',
@@ -174,7 +174,7 @@
 });
 
 it('checks the arguments can be detected in a message with multiple commands that are the same', function () {
-    //Condensed update data
+    // Condensed update data
     $update = new Update([
         'message' => [
             'text' => 'This /demo john doe command with /demo jane doe and a unrelated /test command',
@@ -200,17 +200,17 @@
 
     $this->command->setPattern('{fname} {lname}');
 
-    //First time the command is triggered for entity "0"
+    // First time the command is triggered for entity "0"
     $entity0 = $update->getMessage()->entities->get(0)->toArray();
     $this->command->make($this->api, $update, $entity0);
     expect($this->command->getArguments())->toEqual(['fname' => 'john', 'lname' => 'doe']);
 
-    //Second time the command is triggered for entity "1"
+    // Second time the command is triggered for entity "1"
     $entity1 = $update->getMessage()->entities->get(1)->toArray();
     $this->command->make($this->api, $update, $entity1);
     expect($this->command->getArguments())->toEqual(['fname' => 'jane', 'lname' => 'doe']);
 
-    //This command should not be triggered for entity "2". But if it is, the arguments should be blank.
+    // This command should not be triggered for entity "2". But if it is, the arguments should be blank.
     $entity2 = $update->getMessage()->entities->get(2)->toArray();
     $this->command->make($this->api, $update, $entity2);
     expect($this->command->getArguments())->toEqual([

tests/Unit/Commands/CommandBus.php

@@ -77,7 +77,7 @@
     expect($result)->toHaveCount(4);
     $this->assertStringContainsString('MockCommand3', $commandNames);
 
-    //Remove Specific command.
+    // Remove Specific command.
     $this->bus->removeCommand('MockCommand3');
 
     $newResult = $this->bus->getCommands();
@@ -98,7 +98,7 @@
     $this->assertStringContainsString('MockCommand1', $commandNames);
     $this->assertStringContainsString('MockCommand4', $commandNames);
 
-    //Remove multiple commands at once
+    // Remove multiple commands at once
     $this->bus->removeCommands(['MockCommand1', 'MockCommand4']);
 
     $newResult = $this->bus->getCommands();