Merge pull request #19 from binary-cats/analysis-RvBrpx

Apply fixes from StyleCI

Author: cyrillkalita

src/Event.php

@@ -16,17 +16,17 @@ final class Event implements WebhookEvent
     /**
      * Create new Event.
      *
-     * @param mixed[] $attributes
+     * @param  mixed[]  $attributes
      */
     public function __construct(array $attributes)
     {
         $this->attributes = $attributes;
     }
 
     /**
-     * Static event constructor
+     * Static event constructor.
      *
-     * @param mixed[] $data
+     * @param  mixed[]  $data
      * @return static
      */
     public static function constructFrom(array $data): self

src/Exceptions/UnexpectedValueException.php

@@ -7,7 +7,7 @@
 class UnexpectedValueException extends BaseUnexpectedValueException
 {
     /**
-     * @param \Illuminate\Http\Request $request
+     * @param  \Illuminate\Http\Request  $request
      * @return \Illuminate\Contracts\Foundation\Application|\Illuminate\Contracts\Routing\ResponseFactory|\Illuminate\Http\Response
      */
     public function render($request)

src/Exceptions/WebhookFailed.php

@@ -41,8 +41,9 @@ public static function missingType(WebhookCall $webhookCall): self
     {
         return new static("Webhook call id `{$webhookCall->getKey()}` did not contain a type. Valid Mailgun webhook calls should always contain a type.");
     }
+
     /**
-     * @param \Illuminate\Http\Request $request
+     * @param  \Illuminate\Http\Request  $request
      * @return \Illuminate\Contracts\Foundation\Application|\Illuminate\Contracts\Routing\ResponseFactory|\Illuminate\Http\Response
      */
     public function render($request)

src/Jobs/HandleDelivered.php

@@ -20,7 +20,7 @@ class HandleDelivered
     /**
      * Create new Job.
      *
-     * @param \Spatie\WebhookClient\Models\WebhookCall $webhookCall
+     * @param  \Spatie\WebhookClient\Models\WebhookCall  $webhookCall
      */
     public function __construct(WebhookCall $webhookCall)
     {

src/MailgunSignatureValidator.php

@@ -12,9 +12,8 @@ class MailgunSignatureValidator implements SignatureValidator
     /**
      * True if the signature has been valiates.
      *
-     * @param  \Illuminate\Http\Request       $request
-     * @param  \Spatie\WebhookClient\WebhookConfig $config
-     *
+     * @param  \Illuminate\Http\Request  $request
+     * @param  \Spatie\WebhookClient\WebhookConfig  $config
      * @return bool
      */
     public function isValid(Request $request, WebhookConfig $config): bool
@@ -38,7 +37,7 @@ public function isValid(Request $request, WebhookConfig $config): bool
     /**
      * Validate the incoming signature' schema.
      *
-     * @param  \Illuminate\Http\Request $request
+     * @param  \Illuminate\Http\Request  $request
      * @return string[]
      */
     protected function signature(Request $request): array

src/MailgunWebhooksController.php

@@ -12,8 +12,8 @@ class MailgunWebhooksController
     /**
      * Invoke controller method.
      *
-     * @param  \Illuminate\Http\Request $request
-     * @param  string|null $configKey
+     * @param  \Illuminate\Http\Request  $request
+     * @param  string|null  $configKey
      * @return \Illuminate\Http\Response
      */
     public function __invoke(Request $request, string $configKey = null)

src/ProcessMailgunWebhookJob.php

@@ -25,7 +25,7 @@ public function handle()
     {
         $type = Arr::get($this->webhookCall, "payload.{$this->key}");
 
-        if (!$type) {
+        if (! $type) {
             throw WebhookFailed::missingType($this->webhookCall);
         }
 
@@ -37,7 +37,7 @@ public function handle()
             return;
         }
 
-        if (!class_exists($jobClass)) {
+        if (! class_exists($jobClass)) {
             throw WebhookFailed::jobClassDoesNotExist($jobClass, $this->webhookCall);
         }
 

src/Webhook.php

@@ -9,10 +9,11 @@ class Webhook
     /**
      * Validate and raise an appropriate event.
      *
-     * @param mixed[] $payload
-     * @param string[] $signature
-     * @param string $secret
+     * @param  mixed[]  $payload
+     * @param  string[]  $signature
+     * @param  string  $secret
      * @return \BinaryCats\MailgunWebhooks\Event
+     *
      * @throws WebhookFailed
      */
     public static function constructEvent(array $payload, array $signature, string $secret): Event

src/WebhookSignature.php

@@ -27,7 +27,7 @@ final class WebhookSignature
      * @param  string[]  $signatureArray
      * @param  string  $secret
      */
-    public function __construct(array $signatureArray , string $secret)
+    public function __construct(array $signatureArray, string $secret)
     {
         $this->signatureArray = $signatureArray;
         $this->secret = $secret;
@@ -37,7 +37,7 @@ public function __construct(array $signatureArray , string $secret)
      * Static accessor into the class constructor.
      *
      * @param  string[]  $signatureArray
-     * @param  string $secret
+     * @param  string  $secret
      * @return WebhookSignature static
      */
     public static function make($signatureArray, string $secret)
@@ -73,7 +73,7 @@ protected function computeSignature()
     /**
      * Magically access items from signature array.
      *
-     * @param  string $attribute
+     * @param  string  $attribute
      * @return mixed
      */
     public function __get($attribute)

tests/DummyJob.php

@@ -16,7 +16,7 @@ class DummyJob
     /**
      * Create new Job.
      *
-     * @param \Spatie\WebhookClient\Models\WebhookCall $webhookCall
+     * @param  \Spatie\WebhookClient\Models\WebhookCall  $webhookCall
      */
     public function __construct(WebhookCall $webhookCall)
     {

tests/TestCase.php

@@ -3,7 +3,6 @@
 namespace Tests;
 
 use BinaryCats\MailgunWebhooks\MailgunWebhooksServiceProvider;
-use CreateWebhookCallsTable;
 use Exception;
 use Illuminate\Contracts\Debug\ExceptionHandler;
 use Illuminate\Foundation\Exceptions\Handler;
@@ -24,7 +23,7 @@ public function setUp(): void
     /**
      * Set up the environment.
      *
-     * @param \Illuminate\Foundation\Application $app
+     * @param  \Illuminate\Foundation\Application  $app
      */
     protected function getEnvironmentSetUp($app)
     {
@@ -43,13 +42,13 @@ protected function getEnvironmentSetUp($app)
      */
     protected function setUpDatabase()
     {
-        $migration =  include __DIR__.'/../vendor/spatie/laravel-webhook-client/database/migrations/create_webhook_calls_table.php.stub';
+        $migration = include __DIR__.'/../vendor/spatie/laravel-webhook-client/database/migrations/create_webhook_calls_table.php.stub';
 
         $migration->up();
     }
 
     /**
-     * @param \Illuminate\Foundation\Application $app
+     * @param  \Illuminate\Foundation\Application  $app
      * @return array
      */
     protected function getPackageProviders($app)