Feb 15, 2025 · Feb 13, 2025 · Feb 14, 2025
diff --git a/src/Symfony/Component/Process/CHANGELOG.md b/src/Symfony/Component/Process/CHANGELOG.md
 CHANGELOG
 =========


 7.3
 ---

 * Add`RunProcessMessage::fromShellCommandline()` to instantiate a Process via the fromShellCommandline method

 7.1
 ---

diff --git a/src/Symfony/Component/Process/Messenger/RunProcessMessage.php b/src/Symfony/Component/Process/Messenger/RunProcessMessage.php
 */
 class RunProcessMessageimplements \Stringable
 {
 public ?string$commandLine =null;

 publicfunction__construct(
 publicreadonlyarray$command,
 publicreadonly ?string$cwd =null,

 publicfunction__toString():string
    {
 returnimplode('',$this->command);
 return$this->commandLine ??implode('',$this->command);
    }

 /**
     * Create a process message instance that will instantiate a Process using the fromShellCommandline method.
     *
     * @see Process::fromShellCommandline
     */
 publicstaticfunctionfromShellCommandline(string$command, ?string$cwd =null, ?array$env =null,mixed$input =null, ?float$timeout =60):self
    {
 $message =newself([],$cwd,$env,$input,$timeout);
 $message->commandLine =$command;

 return$message;
    }
 }
diff --git a/src/Symfony/Component/Process/Messenger/RunProcessMessageHandler.php b/src/Symfony/Component/Process/Messenger/RunProcessMessageHandler.php
 {
    public function __invoke(RunProcessMessage $message): RunProcessContext
    {
        $process = new Process($message->command, $message->cwd, $message->env, $message->input, $message->timeout);
        $process = match ($message->commandLine) {
            null => new Process($message->command, $message->cwd, $message->env, $message->input, $message->timeout),
            default => Process::fromShellCommandline($message->commandLine, $message->cwd, $message->env, $message->input, $message->timeout),
        };

        try {
            return new RunProcessContext($message, $process->mustRun());
diff --git a/src/Symfony/Component/Process/Tests/Messenger/RunProcessMessageHandlerTest.php b/src/Symfony/Component/Process/Tests/Messenger/RunProcessMessageHandlerTest.php

        $this->fail('Exception not thrown');
    }

    public function testRunSuccessfulProcessFromShellCommandline()
    {
        $context = (new RunProcessMessageHandler())(RunProcessMessage::fromShellCommandline('ls | grep Test', cwd: __DIR__));

        $this->assertSame('ls | grep Test', $context->message->commandLine);
        $this->assertSame(0, $context->exitCode);
        $this->assertStringContainsString(basename(__FILE__), $context->output);
    }

    public function testRunFailedProcessFromShellCommandline()
    {
        try {
            (new RunProcessMessageHandler())(RunProcessMessage::fromShellCommandline('invalid'));
            $this->fail('Exception not thrown');
        } catch (RunProcessFailedException $e) {
            $this->assertSame('invalid', $e->context->message->commandLine);
            $this->assertContains(
                $e->context->exitCode,
                [null, '\\' === \DIRECTORY_SEPARATOR ? 1 : 127],
                'Exit code should be 1 on Windows, 127 on other systems, or null',
            );
        }
    }
 }
Original file line number	Diff line number	Diff line change
		@@ -1,6 +1,12 @@
		CHANGELOG
		=========


		7.3
		---

		* Add`RunProcessMessage::fromShellCommandline()` to instantiate a Process via the fromShellCommandline method
nicolas-grekas marked this conversation as resolved. Show resolvedHide resolved

		7.1
		---

Expand Down
Original file line number	Diff line number	Diff line change
Expand Up		@@ -16,6 +16,8 @@
		*/
		class RunProcessMessageimplements \Stringable
		{
		public ?string$commandLine =null;

		publicfunction__construct(
		publicreadonlyarray$command,
		publicreadonly ?string$cwd =null,
Expand All		@@ -27,6 +29,19 @@ public function __construct(

		publicfunction__toString():string
		{
		returnimplode('',$this->command);
		return$this->commandLine ??implode('',$this->command);
		}

		/**
		* Create a process message instance that will instantiate a Process using the fromShellCommandline method.
		*
		* @see Process::fromShellCommandline
		*/
		publicstaticfunctionfromShellCommandline(string$command, ?string$cwd =null, ?array$env =null,mixed$input =null, ?float$timeout =60):self
		{
		$message =newself([],$cwd,$env,$input,$timeout);
		$message->commandLine =$command;

		return$message;
		}
		}
Original file line number	Diff line number	Diff line change
Expand Up		@@ -22,7 +22,10 @@ final class RunProcessMessageHandler
		{
		public function __invoke(RunProcessMessage $message): RunProcessContext
		{
		$process = new Process($message->command, $message->cwd, $message->env, $message->input, $message->timeout);
		$process = match ($message->commandLine) {
		null => new Process($message->command, $message->cwd, $message->env, $message->input, $message->timeout),
		default => Process::fromShellCommandline($message->commandLine, $message->cwd, $message->env, $message->input, $message->timeout),
		};

		try {
		return new RunProcessContext($message, $process->mustRun());
Expand Down
Original file line number	Diff line number	Diff line change
Expand Up		@@ -44,4 +44,28 @@ public function testRunFailedProcess()

		$this->fail('Exception not thrown');
		}

		public function testRunSuccessfulProcessFromShellCommandline()
		{
		$context = (new RunProcessMessageHandler())(RunProcessMessage::fromShellCommandline('ls \| grep Test', cwd: __DIR__));

		$this->assertSame('ls \| grep Test', $context->message->commandLine);
		$this->assertSame(0, $context->exitCode);
		$this->assertStringContainsString(basename(__FILE__), $context->output);
		}

		public function testRunFailedProcessFromShellCommandline()
		{
		try {
		(new RunProcessMessageHandler())(RunProcessMessage::fromShellCommandline('invalid'));
		$this->fail('Exception not thrown');
		} catch (RunProcessFailedException $e) {
		$this->assertSame('invalid', $e->context->message->commandLine);
		$this->assertContains(
		$e->context->exitCode,
		[null, '\\' === \DIRECTORY_SEPARATOR ? 1 : 127],
		'Exit code should be 1 on Windows, 127 on other systems, or null',
		);
		}
		}
		}