Merge pull request #16 from spatie/analysis-qgvJ7k

Apply fixes from StyleCI
spatie · Jul 9, 2019 · 8a0595c · 8a0595c
2 parents cbd4d03 + 4955a07
commit 8a0595c
Show file tree

Hide file tree

Showing 7 changed files with 10 additions and 15 deletions.
diff --git a/src/Exceptions/WebhookFailed.php b/src/Exceptions/WebhookFailed.php
@@ -8,7 +8,7 @@ class WebhookFailed extends Exception
 {
     public static function invalidSignature(): WebhookFailed
     {
-        return new static("The signature is invalid.");
+        return new static('The signature is invalid.');
     }
 
     public static function signingSecretNotSet(): WebhookFailed

diff --git a/src/Models/WebhookCall.php b/src/Models/WebhookCall.php
@@ -3,8 +3,8 @@
 namespace Spatie\WebhookClient\Models;
 
 use Exception;
-use Illuminate\Database\Eloquent\Model;
 use Illuminate\Http\Request;
+use Illuminate\Database\Eloquent\Model;
 use Spatie\WebhookClient\WebhookConfig;
 
 class WebhookCall extends Model

diff --git a/src/SignatureValidator/DefaultSignatureValidator.php b/src/SignatureValidator/DefaultSignatureValidator.php
@@ -3,7 +3,6 @@
 namespace Spatie\WebhookClient\SignatureValidator;
 
 use Illuminate\Http\Request;
-use Spatie\WebhookClient\Events\InvalidSignatureEvent;
 use Spatie\WebhookClient\WebhookConfig;
 use Spatie\WebhookClient\Exceptions\WebhookFailed;
 
@@ -14,7 +13,7 @@ public function isValid(Request $request, WebhookConfig $config): bool
         $signature = $request->header($config->signatureHeaderName);
 
         if (! $signature) {
-           return false;
+            return false;
         }
 
         $signingSecret = $config->signingSecret;

diff --git a/tests/TestClasses/EverythingIsValidSignatureValidator.php b/tests/TestClasses/EverythingIsValidSignatureValidator.php
@@ -1,17 +1,15 @@
 <?php
 
-
 namespace Spatie\WebhookClient\Tests\TestClasses;
 
-
 use Illuminate\Http\Request;
-use Spatie\WebhookClient\SignatureValidator\SignatureValidator;
 use Spatie\WebhookClient\WebhookConfig;
+use Spatie\WebhookClient\SignatureValidator\SignatureValidator;
 
 class EverythingIsValidSignatureValidator implements SignatureValidator
 {
     public function isValid(Request $request, WebhookConfig $config): bool
     {
         return true;
     }
-}
+}
diff --git a/tests/TestClasses/NothingIsValidSignatureValidator.php b/tests/TestClasses/NothingIsValidSignatureValidator.php
@@ -1,17 +1,15 @@
 <?php
 
-
 namespace Spatie\WebhookClient\Tests\TestClasses;
 
-
 use Illuminate\Http\Request;
-use Spatie\WebhookClient\SignatureValidator\SignatureValidator;
 use Spatie\WebhookClient\WebhookConfig;
+use Spatie\WebhookClient\SignatureValidator\SignatureValidator;
 
 class NothingIsValidSignatureValidator implements SignatureValidator
 {
     public function isValid(Request $request, WebhookConfig $config): bool
     {
         return false;
     }
-}
+}
diff --git a/tests/TestClasses/WebhookModelWithoutPayloadSaved.php b/tests/TestClasses/WebhookModelWithoutPayloadSaved.php
@@ -3,8 +3,8 @@
 namespace Spatie\WebhookClient\Tests\TestClasses;
 
 use Illuminate\Http\Request;
-use Spatie\WebhookClient\Models\WebhookCall;
 use Spatie\WebhookClient\WebhookConfig;
+use Spatie\WebhookClient\Models\WebhookCall;
 
 class WebhookModelWithoutPayloadSaved extends WebhookCall
 {

diff --git a/tests/WebhookControllerTest.php b/tests/WebhookControllerTest.php
@@ -8,11 +8,11 @@
 use Illuminate\Support\Facades\Route;
 use Spatie\WebhookClient\Models\WebhookCall;
 use Spatie\WebhookClient\Events\InvalidSignatureEvent;
-use Spatie\WebhookClient\Tests\TestClasses\EverythingIsValidSignatureValidator;
-use Spatie\WebhookClient\Tests\TestClasses\NothingIsValidSignatureValidator;
 use Spatie\WebhookClient\Tests\TestClasses\ProcessWebhookJobTestClass;
 use Spatie\WebhookClient\Tests\TestClasses\ProcessNothingWebhookProfile;
 use Spatie\WebhookClient\Tests\TestClasses\WebhookModelWithoutPayloadSaved;
+use Spatie\WebhookClient\Tests\TestClasses\NothingIsValidSignatureValidator;
+use Spatie\WebhookClient\Tests\TestClasses\EverythingIsValidSignatureValidator;
 
 class WebhookControllerTest extends TestCase
 {