Merge pull request #51 from puggan/issue-50

Switch from kill-all-exit to a normal exit when a child is done
spatie · Dec 7, 2023 · c79b4c1 · c79b4c1
2 parents ee80755 + 90a6193
commit c79b4c1
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/src/Fork.php b/src/Fork.php
@@ -129,7 +129,7 @@ protected function forkForTask(Task $task): Task
             try {
                 $this->executeInChildTask($task, $socketToParent);
             } finally {
-                $this->exit();
+                exit();
             }
         }
 

diff --git a/tests/ForkTest.php b/tests/ForkTest.php
@@ -138,3 +138,17 @@ function () {
             fn () => 1
         );
 });
+
+test('allow 2nd process to be done before the 1st')
+    ->expect(
+        fn () => Fork::new()->run(
+            static function () {
+                usleep(200_000);
+                return 2;
+            },
+            static function () {
+                usleep(100_000);
+                return 1;
+            },
+        )
+    )->toEqual([2,1]);