From 1be266fea0df1f10ec09bd465d0b41148528138b Mon Sep 17 00:00:00 2001 From: Daniel Beyer Date: Mon, 19 Jan 2015 14:26:55 +0100 Subject: [PATCH] [Process] Make test AbstractProcessTest::testStartAfterATimeout useful again --- src/Symfony/Component/Process/Tests/AbstractProcessTest.php | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/Symfony/Component/Process/Tests/AbstractProcessTest.php b/src/Symfony/Component/Process/Tests/AbstractProcessTest.php index 1286c107aa..3638e47ac4 100644 --- a/src/Symfony/Component/Process/Tests/AbstractProcessTest.php +++ b/src/Symfony/Component/Process/Tests/AbstractProcessTest.php @@ -724,12 +724,12 @@ abstract class AbstractProcessTest extends \PHPUnit_Framework_TestCase public function testStartAfterATimeout() { - $process = $this->getProcess('php -r "$n = 1000; while ($n--) {echo \'\'; usleep(1000); }"'); + $process = $this->getProcess(sprintf('php -r %s', escapeshellarg('$n = 1000; while ($n--) {echo \'\'; usleep(1000); }'))); $process->setTimeout(0.1); try { $process->run(); - $this->fail('An exception should have been raised.'); - } catch (\Exception $e) { + $this->fail('A RuntimeException should have been raised.'); + } catch (RuntimeException $e) { } $process->start(); usleep(10000);