Merge branch '4.4' into 5.2

nicolas-grekas · nicolas-grekas · commit 3685a3aa8892 · 2021-06-03T14:00:06.000+02:00
* 4.4:
  [Console] Escape synopsis output
  Document null support in NumberToLocalizedStringTransformer
diff --git a/Application.php b/Application.php
@@ -795,7 +795,7 @@ public function renderThrowable(\Throwable $e, OutputInterface $output): void
         $this->doRenderThrowable($e, $output);
 
         if (null !== $this->runningCommand) {
-            $output->writeln(sprintf('<info>%s</info>', sprintf($this->runningCommand->getSynopsis(), $this->getName())), OutputInterface::VERBOSITY_QUIET);
+            $output->writeln(sprintf('<info>%s</info>', OutputFormatter::escape(sprintf($this->runningCommand->getSynopsis(), $this->getName()))), OutputInterface::VERBOSITY_QUIET);
             $output->writeln('', OutputInterface::VERBOSITY_QUIET);
         }
     }
diff --git a/Tests/ApplicationTest.php b/Tests/ApplicationTest.php
@@ -927,6 +927,23 @@ public function testRenderExceptionStackTraceContainsRootException()
         $this->assertStringContainsString('Dummy type "class@anonymous" is invalid.', $tester->getDisplay(true));
     }
 
+    public function testRenderExceptionEscapesLinesOfSynopsis()
+    {
+        $application = new Application();
+        $application->setAutoExit(false);
+        $application
+            ->register('foo')
+            ->setCode(function () {
+                throw new \Exception('some exception');
+            })
+            ->addArgument('info')
+        ;
+        $tester = new ApplicationTester($application);
+
+        $tester->run(['command' => 'foo'], ['decorated' => false]);
+        $this->assertStringMatchesFormatFile(self::$fixturesPath.'/application_rendersynopsis_escapesline.txt', $tester->getDisplay(true), '->renderException() escapes lines containing formatting of synopsis');
+    }
+
     public function testRun()
     {
         $application = new Application();
diff --git a/Tests/Command/CommandTest.php b/Tests/Command/CommandTest.php
@@ -185,7 +185,8 @@ public function testGetSynopsis()
         $command = new \TestCommand();
         $command->addOption('foo');
         $command->addArgument('bar');
-        $this->assertEquals('namespace:name [--foo] [--] [<bar>]', $command->getSynopsis(), '->getSynopsis() returns the synopsis');
+        $command->addArgument('info');
+        $this->assertEquals('namespace:name [--foo] [--] [<bar> [<info>]]', $command->getSynopsis(), '->getSynopsis() returns the synopsis');
     }
 
     public function testAddGetUsages()
diff --git a/Tests/Fixtures/application_rendersynopsis_escapesline.txt b/Tests/Fixtures/application_rendersynopsis_escapesline.txt
@@ -0,0 +1,7 @@
+
+In ApplicationTest.php line %d:
+                  
+  some exception  
+                  
+
+foo [<info>]

Original file line number	Diff line number	Diff line change
`@@ -795,7 +795,7 @@ public function renderThrowable(\Throwable $e, OutputInterface $output): void`
`795`	`795`	`$this->doRenderThrowable($e, $output);`
`796`	`796`
`797`	`797`	`if (null !== $this->runningCommand) {`
`798`		`- $output->writeln(sprintf('<info>%s</info>', sprintf($this->runningCommand->getSynopsis(), $this->getName())), OutputInterface::VERBOSITY_QUIET);`
	`798`	`+ $output->writeln(sprintf('<info>%s</info>', OutputFormatter::escape(sprintf($this->runningCommand->getSynopsis(), $this->getName()))), OutputInterface::VERBOSITY_QUIET);`
`799`	`799`	`$output->writeln('', OutputInterface::VERBOSITY_QUIET);`
`800`	`800`	`}`
`801`	`801`	`}`
Original file line number	Diff line number	Diff line change
`@@ -185,7 +185,8 @@ public function testGetSynopsis()`
`185`	`185`	`$command = new \TestCommand();`
`186`	`186`	`$command->addOption('foo');`
`187`	`187`	`$command->addArgument('bar');`
`188`		`- $this->assertEquals('namespace:name [--foo] [--] [<bar>]', $command->getSynopsis(), '->getSynopsis() returns the synopsis');`
	`188`	`+ $command->addArgument('info');`
	`189`	`+ $this->assertEquals('namespace:name [--foo] [--] [<bar> [<info>]]', $command->getSynopsis(), '->getSynopsis() returns the synopsis');`
`189`	`190`	`}`
`190`	`191`
`191`	`192`	`public function testAddGetUsages()`