[Console] Give errors back to error handlers if not handled by console.error listeners

chalasr · chalasr · commit 5a5bf54d3782 · 2017-04-04T10:56:16.000+02:00
diff --git a/src/Symfony/Component/Console/Application.php b/src/Symfony/Component/Console/Application.php
@@ -138,6 +138,9 @@ public function run(InputInterface $input = null, OutputInterface $output = null
                 $e = null;
                 $exitCode = 0;
             } else {
+                if (!$e instanceof \Exception) {
+                    throw $e;
+                }
                 $exitCode = $e->getCode();
             }
 
diff --git a/src/Symfony/Component/Console/Tests/ApplicationTest.php b/src/Symfony/Component/Console/Tests/ApplicationTest.php
@@ -1117,6 +1117,30 @@ public function testRunWithError()
         $tester->run(array('command' => 'dym'));
     }
 
+    /**
+     * @requires PHP 7
+     */
+    public function testErrorIsRethrownIfNotHandledByConsoleErrorEvent()
+    {
+        $application = new Application();
+        $application->setAutoExit(false);
+        $application->setDispatcher(new EventDispatcher());
+
+        $application->register('dym')->setCode(function (InputInterface $input, OutputInterface $output) {
+            new \UnknownClass();
+        });
+
+        $tester = new ApplicationTester($application);
+
+        try {
+            $tester->run(array('command' => 'dym'));
+            $this->fail('->run() should rethrow PHP errors if not handled via ConsoleErrorEvent.');
+        } catch (\Throwable $e) {
+            $this->assertInstanceOf('Error', $e);
+            $this->assertSame($e->getMessage(), 'Class \'UnknownClass\' not found');
+        }
+    }
+
     /**
      * @expectedException        \LogicException
      * @expectedExceptionMessage error

Original file line number	Diff line number	Diff line change
`@@ -138,6 +138,9 @@ public function run(InputInterface $input = null, OutputInterface $output = null`
`138`	`138`	`$e = null;`
`139`	`139`	`$exitCode = 0;`
`140`	`140`	`} else {`
	`141`	`+ if (!$e instanceof \Exception) {`
	`142`	`+ throw $e;`
	`143`	`+ }`
`141`	`144`	`$exitCode = $e->getCode();`
`142`	`145`	`}`
`143`	`146`