Merge pull request #82 from dmason30/dispatch_sync

Add dispatchSync method
spatie · Apr 28, 2021 · ea5d717 · ea5d717
2 parents 5efc998 + d6c105b
commit ea5d717
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 1 deletion.
diff --git a/src/WebhookCall.php b/src/WebhookCall.php
@@ -191,6 +191,18 @@ public function dispatchNow()
         return dispatch_now($this->callWebhookJob);
     }
 
+    public function dispatchSync(): void
+    {
+        if (function_exists('dispatch_sync')) {
+            $this->prepareForDispatch();
+
+            dispatch_sync($this->callWebhookJob);
+            return;
+        }
+
+        $this->dispatchNow();
+    }
+
     protected function prepareForDispatch(): void
     {
         if (! $this->callWebhookJob->webhookUrl) {

diff --git a/tests/CallWebhookJobTest.php b/tests/CallWebhookJobTest.php
@@ -43,7 +43,7 @@ public function it_can_make_a_webhook_call()
     }
 
     /** @test */
-    public function it_can_make_a_synchronous_webhook_call()
+    public function it_can_make_a_legacy_synchronous_webhook_call()
     {
         $this->baseWebhook()->dispatchNow();
 
@@ -52,6 +52,16 @@ public function it_can_make_a_synchronous_webhook_call()
             ->assertRequestsMade([$this->baseRequest()]);
     }
 
+    /** @test */
+    public function it_can_make_a_synchronous_webhook_call()
+    {
+        $this->baseWebhook()->dispatchSync();
+
+        $this
+            ->testClient
+            ->assertRequestsMade([$this->baseRequest()]);
+    }
+
     /** @test */
     public function it_can_use_a_different_http_verb()
     {