do not let context classes extend the message classes

xabbuh · xabbuh · commit cfc2c65dfa3b · 2023-11-02T14:21:29.000+01:00
diff --git a/src/Symfony/Component/Console/Messenger/RunCommandContext.php b/src/Symfony/Component/Console/Messenger/RunCommandContext.php
@@ -14,10 +14,9 @@
 /**
  * @author Kevin Bond <kevinbond@gmail.com>
  */
-final class RunCommandContext extends RunCommandMessage
+final class RunCommandContext
 {
-    public function __construct(RunCommandMessage $message, public readonly int $exitCode, public readonly string $output)
+    public function __construct(public readonly RunCommandMessage $message, public readonly int $exitCode, public readonly string $output)
     {
-        parent::__construct($message->input, $message->throwOnFailure, $message->catchExceptions);
     }
 }
diff --git a/src/Symfony/Component/Process/Messenger/RunProcessContext.php b/src/Symfony/Component/Process/Messenger/RunProcessContext.php
@@ -16,16 +16,14 @@
 /**
  * @author Kevin Bond <kevinbond@gmail.com>
  */
-final class RunProcessContext extends RunProcessMessage
+final class RunProcessContext
 {
     public readonly ?int $exitCode;
     public readonly ?string $output;
     public readonly ?string $errorOutput;
 
-    public function __construct(RunProcessMessage $message, Process $process)
+    public function __construct(public readonly RunProcessMessage $message, Process $process)
     {
-        parent::__construct($message->command, $message->cwd, $message->env, $message->input, $message->timeout);
-
         $this->exitCode = $process->getExitCode();
         $this->output = $process->isOutputDisabled() ? null : $process->getOutput();
         $this->errorOutput = $process->isOutputDisabled() ? null : $process->getErrorOutput();
diff --git a/src/Symfony/Component/Process/Tests/Messenger/RunProcessMessageHandlerTest.php b/src/Symfony/Component/Process/Tests/Messenger/RunProcessMessageHandlerTest.php
@@ -22,7 +22,7 @@ public function testRunSuccessfulProcess()
     {
         $context = (new RunProcessMessageHandler())(new RunProcessMessage(['ls'], cwd: __DIR__));
 
-        $this->assertSame(['ls'], $context->command);
+        $this->assertSame(['ls'], $context->message->command);
         $this->assertSame(0, $context->exitCode);
         $this->assertStringContainsString(basename(__FILE__), $context->output);
     }
@@ -32,7 +32,7 @@ public function testRunFailedProcess()
         try {
             (new RunProcessMessageHandler())(new RunProcessMessage(['invalid']));
         } catch (RunProcessFailedException $e) {
-            $this->assertSame(['invalid'], $e->context->command);
+            $this->assertSame(['invalid'], $e->context->message->command);
             $this->assertSame('\\' === \DIRECTORY_SEPARATOR ? 1 : 127, $e->context->exitCode);
 
             return;

Original file line number	Diff line number	Diff line change
`@@ -14,10 +14,9 @@`
`14`	`14`	`/**`
`15`	`15`	`* @author Kevin Bond <kevinbond@gmail.com>`
`16`	`16`	`*/`
`17`		`-final class RunCommandContext extends RunCommandMessage`
	`17`	`+final class RunCommandContext`
`18`	`18`	`{`
`19`		`- public function __construct(RunCommandMessage $message, public readonly int $exitCode, public readonly string $output)`
	`19`	`+ public function __construct(public readonly RunCommandMessage $message, public readonly int $exitCode, public readonly string $output)`
`20`	`20`	`{`
`21`		`- parent::__construct($message->input, $message->throwOnFailure, $message->catchExceptions);`
`22`	`21`	`}`
`23`	`22`	`}`