1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48
|
From: Daniel Beyer <dabe@deb.ymc.ch>
Date: Thu, 22 Jan 2015 08:46:10 +0100
Subject: Add more tests to '@group tty'
Not all tests using a tty are in @group tty. This should be reported (and
fixed) upstream but needs further investigation:
- There might be more tests needing a tty.
- It could be that some tests in group tty may not need a tty.
Forwarded: no
---
src/Symfony/Component/Process/Tests/ProcessTest.php | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/src/Symfony/Component/Process/Tests/ProcessTest.php b/src/Symfony/Component/Process/Tests/ProcessTest.php
index 17f98a6..37dd95d 100644
--- a/src/Symfony/Component/Process/Tests/ProcessTest.php
+++ b/src/Symfony/Component/Process/Tests/ProcessTest.php
@@ -504,6 +504,9 @@ class ProcessTest extends TestCase
$this->assertGreaterThan(0, $process->getExitCode());
}
+ /**
+ * @group tty
+ */
public function testTTYCommand()
{
if ('\\' === \DIRECTORY_SEPARATOR) {
@@ -523,6 +526,9 @@ class ProcessTest extends TestCase
$this->assertSame(Process::STATUS_TERMINATED, $process->getStatus());
}
+ /**
+ * @group tty
+ */
public function testTTYCommandExitCode()
{
if ('\\' === \DIRECTORY_SEPARATOR) {
@@ -540,6 +546,9 @@ class ProcessTest extends TestCase
$this->assertTrue($process->isSuccessful());
}
+ /**
+ * @group tty
+ */
public function testTTYInWindowsEnvironment()
{
$this->expectException(RuntimeException::class);
|