[12.x] Fix Concurrency::run to preserve callback result order (#55161)

* Fix Concurrency::run to preserve result order matching the input callbacks

* CS fix

* CS fix

Author: chaker2710

src/Illuminate/Concurrency/ForkDriver.php

@@ -25,6 +25,8 @@ public function run(Closure|array $tasks): array
         /** @phpstan-ignore class.notFound */
         $results = Fork::new()->run(...$values);
 
+        ksort($results);
+
         return array_combine($keys, $results);
     }
 

tests/Integration/Concurrency/ConcurrencyTest.php

@@ -8,6 +8,7 @@
 use Illuminate\Process\Factory as ProcessFactory;
 use Illuminate\Support\Facades\Concurrency;
 use Orchestra\Testbench\TestCase;
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\Attributes\RequiresOperatingSystem;
 
 #[RequiresOperatingSystem('Linux|DAR')]
@@ -119,6 +120,42 @@ public function testRunHandlerProcessErrorWithCustomExceptionWithParam()
             ),
         ]);
     }
+
+    public static function getConcurrencyDrivers(): array
+    {
+        return [
+            ['sync'],
+            ['process'],
+            // spatie/fork package is not included by default
+            // ['fork'],
+        ];
+    }
+
+    #[DataProvider('getConcurrencyDrivers')]
+    public function testRunPreservesCallbackOrder(string $driver)
+    {
+        [$first, $second, $third] = Concurrency::driver($driver)->run([
+            function () {
+                usleep(1000000);
+
+                return 'first';
+            },
+            function () {
+                usleep(500000);
+
+                return 'second';
+            },
+            function () {
+                usleep(200000);
+
+                return 'third';
+            },
+        ]);
+
+        $this->assertEquals('first', $first);
+        $this->assertEquals('second', $second);
+        $this->assertEquals('third', $third);
+    }
 }
 
 class ExceptionWithoutParam extends Exception