Merge pull request #48 from stevebauman/main

Kill any forked tasks when force closed in terminal
spatie · Nov 23, 2023 · 5c38486 · 5c38486
2 parents 82a820f + 3e4c67a
commit 5c38486
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/src/Fork.php b/src/Fork.php
@@ -157,11 +157,15 @@ protected function listenForSignals(): void
 
     protected function exit(): void
     {
-        if (extension_loaded('posix')) {
-            posix_kill(getmypid(), SIGKILL);
+        if (! extension_loaded('posix')) {
+            exit;
         }
 
-        exit;
+        foreach ($this->runningTasks as $task) {
+            posix_kill($task->pid(), SIGKILL);
+        }
+
+        posix_kill(getmypid(), SIGKILL);
     }
 
     protected function currentlyInChildTask(int $pid): bool

diff --git a/tests/ForkTest.php b/tests/ForkTest.php
@@ -11,7 +11,7 @@
         )
     )->toEqual([2, 4]);
 
-it('will execute the given closure with concurrency cap ', function () {
+it('will execute the given closure with concurrency cap', function () {
     $results = Fork::new()
         ->concurrent(2)
         ->run(