Changed order of constructor arguments for command since run configuration is probably the one that will be set the least

Signed-off-by: Bastian Schwarz <bastian@codename-php.de>

Author: bastianschwarz

src/Command.php

@@ -27,17 +27,17 @@ final class Command implements iCommand {
 
   /**
    * @param string $binary The binary to be executed, e.g. ls or composer
-   * @param iRunConfiguration $runConfiguration The run configuration to use when running the command
    * @param array<string> $arguments Arguments to use when running the command, e.g. '-l', 'install', '--composer', '--name=value'
    * @param array<string,string> $envVars Env vars to set when running the command. They will be prepended to the binary with the key as name
    * @param bool $sudo Flag to run the command with sudo
+   * @param iRunConfiguration $runConfiguration The run configuration to use when running the command
    */
   public function __construct(
     public string            $binary,
-    public iRunConfiguration $runConfiguration = new SimpleContainer(),
     public array             $arguments = [],
     public array             $envVars = [],
     public bool              $sudo = false,
+    public iRunConfiguration $runConfiguration = new SimpleContainer(),
   ) {}
 
   public function getRunConfiguration() : iRunConfiguration {

test/CommandTest.php

@@ -37,13 +37,13 @@ public function test__construct() : void {
     $arguments = ['test', '--test'];
     $envVars = ['some' => 'env'];
 
-    $this->sut = new Command($binary, $runConfiguration, $arguments, $envVars, true);
+    $this->sut = new Command($binary, $arguments, $envVars, true, $runConfiguration);
 
     self::assertSame($binary, $this->sut->binary);
-    self::assertSame($runConfiguration, $this->sut->runConfiguration);
     self::assertSame($arguments, $this->sut->arguments);
     self::assertSame($envVars, $this->sut->envVars);
     self::assertTrue($this->sut->sudo);
+    self::assertSame($runConfiguration, $this->sut->runConfiguration);
   }
 
   public function test__toString() : void {