refactor: remove support for array-based process invocation

Author: innocenzi

packages/process/src/GenericProcessExecutor.php

@@ -7,7 +7,7 @@
 
 final class GenericProcessExecutor implements ProcessExecutor
 {
-    public function start(array|string|PendingProcess $command): InvokedSystemProcess
+    public function start(string|PendingProcess $command): InvokedSystemProcess
     {
         $pending = $this->createPendingProcess($command);
         $command = $this->createSymfonyProcess($pending);
@@ -16,7 +16,7 @@ public function start(array|string|PendingProcess $command): InvokedSystemProces
         return new InvokedSystemProcess($command);
     }
 
-    public function run(array|string|PendingProcess $command): ProcessResult
+    public function run(string|PendingProcess $command): ProcessResult
     {
         return $this->start($command)->wait();
     }

packages/process/src/ProcessExecutor.php

@@ -7,16 +7,16 @@ interface ProcessExecutor
     /**
      * Runs the given process.
      *
-     * @param string[]|string|PendingProcess $command
+     * @param string|PendingProcess $command
      */
-    public function run(array|string|PendingProcess $command): ProcessResult;
+    public function run(string|PendingProcess $command): ProcessResult;
 
     /**
      * Invokes the given process asynchronously.
      *
-     * @param string[]|string|PendingProcess $command
+     * @param string|PendingProcess $command
      */
-    public function start(array|string|PendingProcess $command): InvokedProcess;
+    public function start(string|PendingProcess $command): InvokedProcess;
 
     /**
      * Returns a pool of processes, which can be executed.

packages/process/src/Testing/RestrictedProcessExecutor.php

@@ -11,12 +11,12 @@
 
 final class RestrictedProcessExecutor implements ProcessExecutor
 {
-    public function run(array|string|PendingProcess $command): ProcessResult
+    public function run(string|PendingProcess $command): ProcessResult
     {
         throw ProcessExecutionWasForbidden::forPendingProcess($command);
     }
 
-    public function start(array|string|PendingProcess $command): InvokedSystemProcess
+    public function start(string|PendingProcess $command): InvokedSystemProcess
     {
         throw ProcessExecutionWasForbidden::forPendingProcess($command);
     }

packages/process/src/Testing/TestingProcessExecutor.php

@@ -28,7 +28,7 @@ public function __construct(
         public bool $allowRunningActualProcesses = false,
     ) {}
 
-    public function run(array|string|PendingProcess $command): ProcessResult
+    public function run(string|PendingProcess $command): ProcessResult
     {
         if ($result = $this->findMockedProcess($command)) {
             return $this->recordExecution($command, $result);
@@ -41,7 +41,7 @@ public function run(array|string|PendingProcess $command): ProcessResult
         return $this->recordExecution($command, $this->start($command)->wait());
     }
 
-    public function start(array|string|PendingProcess $command): InvokedProcess
+    public function start(string|PendingProcess $command): InvokedProcess
     {
         if ($processResult = $this->findInvokedProcessDescription($command)) {
             $this->recordExecution($command, $process = new InvokedTestingProcess($processResult));
@@ -69,7 +69,7 @@ public function concurrently(iterable $pool): ProcessPoolResults
         return $this->pool($pool)->start()->wait();
     }
 
-    private function findMockedProcess(array|string|PendingProcess $command): ?ProcessResult
+    private function findMockedProcess(string|PendingProcess $command): ?ProcessResult
     {
         $process = $this->createPendingProcess($command);
 
@@ -92,7 +92,7 @@ private function findMockedProcess(array|string|PendingProcess $command): ?Proce
         return null;
     }
 
-    private function findInvokedProcessDescription(array|string|PendingProcess $command): ?InvokedProcessDescription
+    private function findInvokedProcessDescription(string|PendingProcess $command): ?InvokedProcessDescription
     {
         $process = $this->createPendingProcess($command);
 
@@ -111,7 +111,7 @@ private function findInvokedProcessDescription(array|string|PendingProcess $comm
         return null;
     }
 
-    private function recordExecution(array|string|PendingProcess $command, InvokedProcess|ProcessResult $result): ProcessResult
+    private function recordExecution(string|PendingProcess $command, InvokedProcess|ProcessResult $result): ProcessResult
     {
         $process = $this->createPendingProcess($command);
         $result = match (true) {

tests/Integration/Process/ProcessExecutorTest.php

@@ -2,11 +2,9 @@
 
 namespace Tests\Tempest\Integration\Process;
 
-use PHPUnit\Framework\ExpectationFailedException;
 use Tempest\Process\InvokedProcess;
 use Tempest\Process\ProcessExecutor;
 use Tempest\Process\ProcessResult;
-use Tempest\Process\Testing\InvokedProcessDescription;
 use Tests\Tempest\Integration\FrameworkIntegrationTestCase;
 
 final class ProcessExecutorTest extends FrameworkIntegrationTestCase