refactor: wait all processes instead of just ones marked as running

Author: innocenzi

packages/process/src/InvokedProcessPool.php

@@ -15,8 +15,8 @@ final class InvokedProcessPool implements Countable
      */
     public ImmutableArray $running {
         get => $this->processes
-            ->filter(fn (InvokedProcess $process) => $process->running)
-            ->toImmutableArray();
+            ->toImmutableArray()
+            ->filter(fn (InvokedProcess $process) => $process->running);
     }
 
     /**
@@ -55,7 +55,7 @@ public function stop(float|int|Duration $timeout = 10, ?int $signal = null): Imm
     public function wait(): ProcessPoolResults
     {
         return new ProcessPoolResults(
-            $this->running->map(fn (InvokedProcess $process) => $process->wait()),
+            $this->all->map(fn (InvokedProcess $process) => $process->wait()),
         );
     }
 

packages/process/tests/PoolTest.php

@@ -32,7 +32,6 @@ public function test_pool(): void
 
         $results = $invoked->wait();
 
-        var_dump($results, $results->count());
         $this->assertSame(2, $results->count());
         $this->assertStringEqualsStringIgnoringLineEndings("hello\n", $results[0]->output);
         $this->assertStringEqualsStringIgnoringLineEndings("world\n", $results[1]->output);