Merge pull request #16 from JonasPardon/#11-add-support-for-dispatch-sync

Author: Jonas Pardon

src/EventVisualizer.php

@@ -155,21 +155,15 @@ private function getDispatchedEvents(CodeParser $codeParser): Collection
         $events = [];
 
         $foundFunctionCalls = $codeParser->getFunctionCalls('event');
-        if (count($foundFunctionCalls) > 0) {
-            $events = array_merge($events, $foundFunctionCalls);
-        }
+        $events = array_merge($events, $foundFunctionCalls);
 
         foreach ($classes as $class) {
             foreach ($methods as $method) {
                 $foundStaticCalls = $codeParser->getStaticCalls($class, $method);
-                if (count($foundStaticCalls) !== 0) {
-                    $events = array_merge($events, $foundStaticCalls);
-                }
+                $events = array_merge($events, $foundStaticCalls);
 
                 $foundMethodCalls = $codeParser->getMethodCalls($class, $method);
-                if (count($foundMethodCalls) !== 0) {
-                    $events = array_merge($events, $foundMethodCalls);
-                }
+                $events = array_merge($events, $foundMethodCalls);
             }
         }
 
@@ -194,22 +188,19 @@ private function getDispatchedJobs(CodeParser $codeParser): Collection
 
         $jobs = [];
 
-        $foundFunctionCalls = $codeParser->getFunctionCalls('dispatch');
-        if (count($foundFunctionCalls) > 0) {
-            $jobs = array_merge($jobs, $foundFunctionCalls);
-        }
+        $foundFunctionCalls = array_merge(
+            $codeParser->getFunctionCalls('dispatch'),
+            $codeParser->getFunctionCalls('dispatch_sync'),
+        );
+        $jobs = array_merge($jobs, $foundFunctionCalls);
 
         foreach ($classes as $class) {
             foreach ($methods as $method) {
                 $foundStaticCalls = $codeParser->getStaticCalls($class, $method);
-                if (count($foundStaticCalls) !== 0) {
-                    $jobs = array_merge($jobs, $foundStaticCalls);
-                }
+                $jobs = array_merge($jobs, $foundStaticCalls);
 
                 $foundMethodCalls = $codeParser->getMethodCalls($class, $method);
-                if (count($foundMethodCalls) !== 0) {
-                    $jobs = array_merge($jobs, $foundMethodCalls);
-                }
+                $jobs = array_merge($jobs, $foundMethodCalls);
             }
         }
 

tests/Unit/CodeParser/FunctionCallsParsingTest.php

@@ -77,4 +77,39 @@ public function handle(): void
         $this->assertEquals($expectedHelperCall->method, $helperCalls[0]->method);
         $this->assertEquals($expectedHelperCall->dispatchedClass, $helperCalls[0]->dispatchedClass);
     }
+
+    /** @test */
+    public function it_finds_a_dispatch_sync_helper_call(): void
+    {
+        $code = <<<'CODE'
+            <?php
+            
+            namespace App\Domain\SomeDomain;
+            
+            use App\Events\SomeJob;
+            
+            class SomeClass
+            {
+                public function handle(): void
+                {
+                    dispatch_sync(new SomeJob());
+                }
+            }
+            CODE;
+
+        $codeParser = new CodeParser($code);
+
+        $helperCalls = $codeParser->getFunctionCalls('dispatch_sync');
+
+        $expectedHelperCall = new ResolvedCall(
+            dispatcherClass: 'none',
+            dispatchedClass: 'App\Events\SomeJob',
+            method: 'dispatch_sync',
+        );
+
+        $this->assertCount(1, $helperCalls);
+        $this->assertEquals($expectedHelperCall->dispatcherClass, $helperCalls[0]->dispatcherClass);
+        $this->assertEquals($expectedHelperCall->method, $helperCalls[0]->method);
+        $this->assertEquals($expectedHelperCall->dispatchedClass, $helperCalls[0]->dispatchedClass);
+    }
 }