use Arr and Str helper classes (#97)

Pacha Cervantes · web-flow · commit 6ab7279b6a03 · 2021-08-10T11:45:46.000+08:00
diff --git a/src/Commands/DomainWhitelistingCommand.php b/src/Commands/DomainWhitelistingCommand.php
@@ -9,6 +9,7 @@
 namespace Casperlaitw\LaravelFbMessenger\Commands;
 
 use Casperlaitw\LaravelFbMessenger\Messages\DomainWhitelisting;
+use Illuminate\Support\Arr;
 
 /**
  * Class DomainWhitelistingCommand
@@ -51,7 +52,7 @@ private function read()
     {
         $command = new DomainWhitelisting();
         $command->setAction(DomainWhitelisting::TYPE_READ)->useGet();
-        $response = collect(array_get($this->handler->send($command)->getResponse(), 'data.0.whitelisted_domains', []))
+        $response = collect(Arr::get($this->handler->send($command)->getResponse(), 'data.0.whitelisted_domains', []))
             ->map(function ($item) {
                 return [$item];
             });
diff --git a/src/Commands/HomeUrlCommand.php b/src/Commands/HomeUrlCommand.php
@@ -4,6 +4,7 @@
 namespace Casperlaitw\LaravelFbMessenger\Commands;
 
 use Casperlaitw\LaravelFbMessenger\Messages\HomeUrl;
+use Illuminate\Support\Arr;
 
 /**
  * Class HomeUrlCommand
@@ -49,7 +50,7 @@ private function read()
     {
         $command = new HomeUrl();
         $command->setAction(HomeUrl::TYPE_READ)->useGet();
-        $response = collect(array_get($this->handler->send($command)->getResponse(), 'data.0.home_url', []))
+        $response = collect(Arr::get($this->handler->send($command)->getResponse(), 'data.0.home_url', []))
             ->map(function ($item) {
                 return [$item];
             });
diff --git a/src/Commands/MessengerCodeCommand.php b/src/Commands/MessengerCodeCommand.php
@@ -3,6 +3,7 @@
 namespace Casperlaitw\LaravelFbMessenger\Commands;
 
 use Casperlaitw\LaravelFbMessenger\Messages\MessengerCode;
+use Illuminate\Support\Arr;
 
 class MessengerCodeCommand extends BaseCommand
 {
@@ -35,6 +36,6 @@ public function handle()
             $message->setRef($ref);
         }
 
-        $this->comment(array_get($this->handler->send($message), 'uri'));
+        $this->comment(Arr::get($this->handler->send($message), 'uri'));
     }
 }
diff --git a/src/Contracts/Debug/Handler.php b/src/Contracts/Debug/Handler.php
@@ -10,6 +10,7 @@
 
 use Exception;
 use Illuminate\Contracts\Debug\ExceptionHandler;
+use Illuminate\Support\Arr;
 
 class Handler implements ExceptionHandler
 {
@@ -61,9 +62,9 @@ public function render($request, Exception $e)
             'message' => $e->getMessage(),
             'trace' => collect($e->getTrace())->map(function ($item) {
                 return [
-                    'file' => array_get($item, 'file'),
-                    'line' => array_get($item, 'line'),
-                    'method' => array_get($item, 'function'),
+                    'file' => Arr::get($item, 'file'),
+                    'line' => Arr::get($item, 'line'),
+                    'method' => Arr::get($item, 'function'),
                 ];
             })->toArray(),
         ];
diff --git a/src/Contracts/HandleMessageResponse.php b/src/Contracts/HandleMessageResponse.php
@@ -7,6 +7,8 @@
 
 namespace Casperlaitw\LaravelFbMessenger\Contracts;
 
+use Illuminate\Support\Arr;
+
 /**
  * Class HandleMessageResponse
  * @package Casperlaitw\LaravelFbMessenger\Contracts
@@ -37,7 +39,7 @@ public function getResponse()
         if (!empty($this->response['error'])) {
             return $this->handleError($this->response['error']);
         }
-        return array_get($this->response, 'result', $this->response);
+        return Arr::get($this->response, 'result', $this->response);
     }
 
     /**
diff --git a/tests/Collections/ButtonCollectionTest.php b/tests/Collections/ButtonCollectionTest.php
@@ -3,6 +3,7 @@
 use Casperlaitw\LaravelFbMessenger\Exceptions\OnlyUseByItselfException;
 use Casperlaitw\LaravelFbMessenger\Exceptions\ValidatorStructureException;
 use Casperlaitw\LaravelFbMessenger\Messages\Button;
+use Illuminate\Support\Str;
 use Faker\Factory;
 use Mockery as m;
 
@@ -54,7 +55,7 @@ public function test_add_call_button()
     {
         $fake = Factory::create();
         $collection = new ButtonCollection();
-        $title = str_random();
+        $title = Str::random();
         $phone = $fake->phoneNumber;
         $collection->addCallButton($title, $phone);
 
@@ -73,7 +74,7 @@ public function test_add_share_button()
 
     public function test_add_account_link_button()
     {
-        $url = str_random();
+        $url = Str::random();
         $collection = new ButtonCollection();
         $collection->addAccountLinkButton($url);
         $expected = new Button(Button::TYPE_ACCOUNT_LINK, null, $url);
diff --git a/tests/Contracts/BotTest.php b/tests/Contracts/BotTest.php
@@ -6,6 +6,7 @@
 use Casperlaitw\LaravelFbMessenger\Messages\MessengerCode;
 use Casperlaitw\LaravelFbMessenger\Messages\User;
 use Illuminate\Broadcasting\BroadcastException;
+use Illuminate\Support\Str;
 use Illuminate\Events\Dispatcher;
 use Mockery as m;
 
@@ -24,7 +25,7 @@ protected function setUp()
 
     public function test_send_success()
     {
-        $message = new Greeting(['locale' => 'default', 'text' => str_random()]);
+        $message = new Greeting(['locale' => 'default', 'text' => Str::random()]);
         $this->bot->setSecret('test_app_secret');
         $this->bot->send($message);
     }
diff --git a/tests/Contracts/Debug/HandlerTest.php b/tests/Contracts/Debug/HandlerTest.php
@@ -4,6 +4,7 @@
 use Casperlaitw\LaravelFbMessenger\Contracts\Debug\Debug;
 use Illuminate\Contracts\Debug\ExceptionHandler;
 use Illuminate\Events\Dispatcher;
+use Illuminate\Support\Arr;
 use Mockery as m;
 use Symfony\Component\HttpFoundation\Response;
 
@@ -56,6 +57,6 @@ public function test_render()
 
         $actual = $this->getPrivateProperty(Handler::class, 'debug')->getValue($this->handler);
         $this->assertEquals(500, $actual->getStatus());
-        $this->assertEquals('ERROR', array_get($actual->getResponse(), 'message'));
+        $this->assertEquals('ERROR', Arr::get($actual->getResponse(), 'message'));
     }
 }
diff --git a/tests/Contracts/Messages/AttachmentTest.php b/tests/Contracts/Messages/AttachmentTest.php
@@ -1,6 +1,7 @@
 <?php
 use Casperlaitw\LaravelFbMessenger\Contracts\Messages\Attachment;
 use Casperlaitw\LaravelFbMessenger\Exceptions\InvalidTypeException;
+use Illuminate\Support\Str;
 use Faker\Factory;
 use Mockery as m;
 
@@ -22,7 +23,7 @@ protected function setUp()
     {
         parent::setUp();
         $this->faker = Factory::create();
-        $sender = str_random();
+        $sender = Str::random();
         $type = Attachment::TYPE_IMAGE;
         $image = $this->faker->url;
         $this->attachment = new AttachmentStub($sender, $type, ['url' => $image]);
@@ -57,7 +58,7 @@ public function test_disable_reuse()
 
     public function test_set_attachment_id()
     {
-        $id = str_random();
+        $id = Str::random();
         $this->attachment->setAttachmentId($id);
 
         $this->assertEquals(['attachment_id' => $id], $this->attachment->getPayload());
diff --git a/tests/Contracts/Messages/StructuredTest.php b/tests/Contracts/Messages/StructuredTest.php
@@ -2,6 +2,7 @@
 
 use Casperlaitw\LaravelFbMessenger\Messages\Button;
 use Casperlaitw\LaravelFbMessenger\Messages\ButtonTemplate;
+use Illuminate\Support\Str;
 use Mockery as m;
 
 /**
@@ -13,13 +14,13 @@ class StructuredTest extends TestCase
 {
     public function test_call_method()
     {
-        $button = new ButtonTemplate(str_random(), str_random());
+        $button = new ButtonTemplate(Str::random(), Str::random());
         $this->assertTrue($button->validator($this->getMessageButtonMock()));
     }
 
     public function test_non_collection_method()
     {
-        $button = new ButtonTemplate(str_random(), str_random());
+        $button = new ButtonTemplate(Str::random(), Str::random());
         $button->getError();
     }
 
diff --git a/tests/Controllers/WebhookControllerTest.php b/tests/Controllers/WebhookControllerTest.php
@@ -7,6 +7,7 @@
 use Illuminate\Events\Dispatcher;
 use Illuminate\Http\Request;
 use Illuminate\Http\Response;
+use Illuminate\Support\Str;
 use Mockery as m;
 use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
 
@@ -20,7 +21,7 @@ class WebhookControllerTest extends TestCase
     public function test_index_token_verify_pass_response_challenge_token()
     {
         $verifyToken = 'MY_VERIFY_TOKEN';
-        $challenge = str_random();
+        $challenge = Str::random();
         $request = m::mock(Request::class)
             ->shouldReceive('get')
             ->with('hub_mode')
@@ -55,7 +56,7 @@ public function test_index_and_abort()
             ->getMock();
         $config = m::mock(Repository::class)
             ->shouldReceive('get')
-            ->andReturn(str_random())
+            ->andReturn(Str::random())
             ->getMock();
         $debug = m::mock(Debug::class);
 
diff --git a/tests/Events/BroadcastTest.php b/tests/Events/BroadcastTest.php
@@ -1,5 +1,6 @@
 <?php
 use Casperlaitw\LaravelFbMessenger\Events\Broadcast;
+use Illuminate\Support\Str;
 
 /**
  * Created by PhpStorm.
@@ -20,7 +21,7 @@ public function setUp()
     {
         parent::setUp();
 
-        $this->id = $this->response = $this->code = $this->webhook = $this->request = str_random();
+        $this->id = $this->response = $this->code = $this->webhook = $this->request = Str::random();
         $this->broadcast = new Broadcast($this->id, $this->webhook, $this->request, $this->response, $this->code);
     }
 
diff --git a/tests/Messages/AudioTest.php b/tests/Messages/AudioTest.php
@@ -1,6 +1,7 @@
 <?php
 
 use Casperlaitw\LaravelFbMessenger\Messages\Audio;
+use Illuminate\Support\Str;
 use Faker\Factory;
 
 /**
@@ -13,7 +14,7 @@ class AudioTest extends TestCase
     public function test_to_data()
     {
         $faker = Factory::create();
-        $sender = str_random();
+        $sender = Str::random();
         $url = $faker->url;
 
         $actual = new Audio($sender, $url);
diff --git a/tests/Messages/ButtonTemplateTest.php b/tests/Messages/ButtonTemplateTest.php
@@ -1,6 +1,7 @@
 <?php
 
 use Casperlaitw\LaravelFbMessenger\Messages\ButtonTemplate;
+use Illuminate\Support\Str;
 use Mockery as m;
 use pimax\Messages\StructuredMessage;
 
@@ -19,7 +20,7 @@ class ButtonTemplateTest extends TestCase
 
     public function setUp()
     {
-        $this->sender = str_random();
+        $this->sender = Str::random();
         $this->text = 'abc';
         $this->case = new ButtonTemplate($this->sender, $this->text);
     }
diff --git a/tests/Messages/ElementTest.php b/tests/Messages/ElementTest.php
@@ -2,6 +2,7 @@
 use Casperlaitw\LaravelFbMessenger\Collections\ButtonCollection;
 use Casperlaitw\LaravelFbMessenger\Messages\Element;
 use Casperlaitw\LaravelFbMessenger\Messages\UrlButton;
+use Illuminate\Support\Str;
 
 /**
  * User: casperlai
@@ -13,13 +14,13 @@ class ElementTest extends TestCase
 {
     public function test_button()
     {
-        $element = new Element(str_random(), str_random());
+        $element = new Element(Str::random(), Str::random());
         $this->assertInstanceOf(ButtonCollection::class, $element->buttons());
     }
 
     public function test_default_action()
     {
-        $element = new Element(str_random(), str_random());
+        $element = new Element(Str::random(), Str::random());
         $element->setDefaultAction(new UrlButton('title', 'url'));
 
         $this->assertArraySubset([
diff --git a/tests/Messages/FileTest.php b/tests/Messages/FileTest.php
@@ -1,5 +1,6 @@
 <?php
 use Casperlaitw\LaravelFbMessenger\Messages\File;
+use Illuminate\Support\Str;
 use Faker\Factory;
 
 /**
@@ -12,7 +13,7 @@ class FileTest extends TestCase
     public function test_to_data()
     {
         $faker = Factory::create();
-        $sender = str_random();
+        $sender = Str::random();
         $url = $faker->url;
 
         $actual = new File($sender, $url);
diff --git a/tests/Messages/GenericTemplateTest.php b/tests/Messages/GenericTemplateTest.php
@@ -2,6 +2,7 @@
 
 use Casperlaitw\LaravelFbMessenger\Messages\Element;
 use Casperlaitw\LaravelFbMessenger\Messages\GenericTemplate;
+use Illuminate\Support\Str;
 use pimax\Messages\StructuredMessage;
 
 /**
@@ -17,7 +18,7 @@ class GenericTemplateTest extends TestCase
 
     public function setUp()
     {
-        $this->sender = str_random();
+        $this->sender = Str::random();
         $this->case = [
             new Element('title1', 'description1'),
             new Element('title2', 'description2')
diff --git a/tests/Messages/GreetingTest.php b/tests/Messages/GreetingTest.php
@@ -1,5 +1,6 @@
 <?php
 use Casperlaitw\LaravelFbMessenger\Messages\Greeting;
+use Illuminate\Support\Str;
 
 /**
  * User: casperlai
@@ -10,7 +11,7 @@ class GreetingTest extends TestCase
 {
     public function test_to_data()
     {
-        $greetingText = str_random();
+        $greetingText = Str::random();
         $expected = [
             'greeting' => [
                 [
diff --git a/tests/Messages/ImageTest.php b/tests/Messages/ImageTest.php
@@ -1,5 +1,6 @@
 <?php
 use Casperlaitw\LaravelFbMessenger\Messages\Image;
+use Illuminate\Support\Str;
 
 /**
  * User: casperlai
@@ -10,8 +11,8 @@ class ImageTest extends TestCase
 {
     public function test_to_data()
     {
-        $sender = str_random();
-        $image = str_random();
+        $sender = Str::random();
+        $image = Str::random();
         $expected = [
             'recipient' => [
                 'id' => $sender,
diff --git a/tests/Messages/ListTemplateTest.php b/tests/Messages/ListTemplateTest.php
@@ -3,6 +3,7 @@
 use Casperlaitw\LaravelFbMessenger\Messages\Button;
 use Casperlaitw\LaravelFbMessenger\Messages\Element;
 use Casperlaitw\LaravelFbMessenger\Messages\ListTemplate;
+use Illuminate\Support\Str;
 
 /**
  * Created by PhpStorm.
@@ -18,7 +19,7 @@ class ListTemplateTest extends TestCase
 
     public function setUp()
     {
-        $this->sender = str_random();
+        $this->sender = Str::random();
         $this->case = [
             new Element('title1', 'description1', 'image1'),
             new Element('title2', 'description2', 'image2'),
diff --git a/tests/Messages/QuickReplyTest.php b/tests/Messages/QuickReplyTest.php
@@ -1,5 +1,6 @@
 <?php
 use Casperlaitw\LaravelFbMessenger\Messages\QuickReply;
+use Illuminate\Support\Str;
 
 /**
  * Created by PhpStorm.
@@ -36,7 +37,7 @@ public function test_set_image()
     {
         $title = 'Red';
         $payload = 'PAYLOAD_RED';
-        $image = str_random();
+        $image = Str::random();
 
         $expected = [
             'content_type' => 'text',
diff --git a/tests/Messages/ReceiveMessageTest.php b/tests/Messages/ReceiveMessageTest.php
diff --git a/tests/Messages/StartButtonTest.php b/tests/Messages/StartButtonTest.php
diff --git a/tests/Messages/TextTest.php b/tests/Messages/TextTest.php
diff --git a/tests/Messages/UserTest.php b/tests/Messages/UserTest.php
diff --git a/tests/Messages/VideoTest.php b/tests/Messages/VideoTest.php
diff --git a/tests/Transformers/ButtonTransformerTest.php b/tests/Transformers/ButtonTransformerTest.php
diff --git a/tests/Transformers/GenericTransformerTest.php b/tests/Transformers/GenericTransformerTest.php
diff --git a/tests/Transformers/ListTransformerTest.php b/tests/Transformers/ListTransformerTest.php

Original file line number	Diff line number	Diff line change
`@@ -3,6 +3,7 @@`
`3`	`3`	`namespace Casperlaitw\LaravelFbMessenger\Commands;`
`4`	`4`
`5`	`5`	`use Casperlaitw\LaravelFbMessenger\Messages\MessengerCode;`
	`6`	`+use Illuminate\Support\Arr;`
`6`	`7`
`7`	`8`	`class MessengerCodeCommand extends BaseCommand`
`8`	`9`	`{`
`@@ -35,6 +36,6 @@ public function handle()`
`35`	`36`	`$message->setRef($ref);`
`36`	`37`	`}`
`37`	`38`
`38`		`- $this->comment(array_get($this->handler->send($message), 'uri'));`
	`39`	`+ $this->comment(Arr::get($this->handler->send($message), 'uri'));`
`39`	`40`	`}`
`40`	`41`	`}`
Original file line number	Diff line number	Diff line change
`@@ -7,6 +7,8 @@`
`7`	`7`
`8`	`8`	`namespace Casperlaitw\LaravelFbMessenger\Contracts;`
`9`	`9`
	`10`	`+use Illuminate\Support\Arr;`
	`11`	`+`
`10`	`12`	`/**`
`11`	`13`	`* Class HandleMessageResponse`
`12`	`14`	`* @package Casperlaitw\LaravelFbMessenger\Contracts`
`@@ -37,7 +39,7 @@ public function getResponse()`
`37`	`39`	`if (!empty($this->response['error'])) {`
`38`	`40`	`return $this->handleError($this->response['error']);`
`39`	`41`	`}`
`40`		`- return array_get($this->response, 'result', $this->response);`
	`42`	`+ return Arr::get($this->response, 'result', $this->response);`
`41`	`43`	`}`
`42`	`44`
`43`	`45`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -6,6 +6,7 @@`
`6`	`6`	`use Casperlaitw\LaravelFbMessenger\Messages\MessengerCode;`
`7`	`7`	`use Casperlaitw\LaravelFbMessenger\Messages\User;`
`8`	`8`	`use Illuminate\Broadcasting\BroadcastException;`
	`9`	`+use Illuminate\Support\Str;`
`9`	`10`	`use Illuminate\Events\Dispatcher;`
`10`	`11`	`use Mockery as m;`
`11`	`12`
`@@ -24,7 +25,7 @@ protected function setUp()`
`24`	`25`
`25`	`26`	`public function test_send_success()`
`26`	`27`	`{`
`27`		`- $message = new Greeting(['locale' => 'default', 'text' => str_random()]);`
	`28`	`+ $message = new Greeting(['locale' => 'default', 'text' => Str::random()]);`
`28`	`29`	`$this->bot->setSecret('test_app_secret');`
`29`	`30`	`$this->bot->send($message);`
`30`	`31`	`}`
Original file line number	Diff line number	Diff line change
`@@ -4,6 +4,7 @@`
`4`	`4`	`use Casperlaitw\LaravelFbMessenger\Contracts\Debug\Debug;`
`5`	`5`	`use Illuminate\Contracts\Debug\ExceptionHandler;`
`6`	`6`	`use Illuminate\Events\Dispatcher;`
	`7`	`+use Illuminate\Support\Arr;`
`7`	`8`	`use Mockery as m;`
`8`	`9`	`use Symfony\Component\HttpFoundation\Response;`
`9`	`10`
`@@ -56,6 +57,6 @@ public function test_render()`
`56`	`57`
`57`	`58`	`$actual = $this->getPrivateProperty(Handler::class, 'debug')->getValue($this->handler);`
`58`	`59`	`$this->assertEquals(500, $actual->getStatus());`
`59`		`- $this->assertEquals('ERROR', array_get($actual->getResponse(), 'message'));`
	`60`	`+ $this->assertEquals('ERROR', Arr::get($actual->getResponse(), 'message'));`
`60`	`61`	`}`
`61`	`62`	`}`
Original file line number	Diff line number	Diff line change
`@@ -2,6 +2,7 @@`
`2`	`2`
`3`	`3`	`use Casperlaitw\LaravelFbMessenger\Messages\Button;`
`4`	`4`	`use Casperlaitw\LaravelFbMessenger\Messages\ButtonTemplate;`
	`5`	`+use Illuminate\Support\Str;`
`5`	`6`	`use Mockery as m;`
`6`	`7`
`7`	`8`	`/**`
`@@ -13,13 +14,13 @@ class StructuredTest extends TestCase`
`13`	`14`	`{`
`14`	`15`	`public function test_call_method()`
`15`	`16`	`{`
`16`		`- $button = new ButtonTemplate(str_random(), str_random());`
	`17`	`+ $button = new ButtonTemplate(Str::random(), Str::random());`
`17`	`18`	`$this->assertTrue($button->validator($this->getMessageButtonMock()));`
`18`	`19`	`}`
`19`	`20`
`20`	`21`	`public function test_non_collection_method()`
`21`	`22`	`{`
`22`		`- $button = new ButtonTemplate(str_random(), str_random());`
	`23`	`+ $button = new ButtonTemplate(Str::random(), Str::random());`
`23`	`24`	`$button->getError();`
`24`	`25`	`}`
`25`	`26`