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
|
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 | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/src/Symfony/Component/Process/Tests/ProcessTest.php b/src/Symfony/Component/Process/Tests/ProcessTest.php
index a9ecbe8..1972868 100644
--- a/src/Symfony/Component/Process/Tests/ProcessTest.php
+++ b/src/Symfony/Component/Process/Tests/ProcessTest.php
@@ -447,6 +447,9 @@ class ProcessTest extends TestCase
$this->assertGreaterThan(0, $process->getExitCode());
}
+ /**
+ * @group tty
+ */
public function testTTYCommand()
{
if ('\\' === \DIRECTORY_SEPARATOR) {
@@ -462,6 +465,9 @@ class ProcessTest extends TestCase
$this->assertSame(Process::STATUS_TERMINATED, $process->getStatus());
}
+ /**
+ * @group tty
+ */
public function testTTYCommandExitCode()
{
if ('\\' === \DIRECTORY_SEPARATOR) {
@@ -479,6 +485,7 @@ class ProcessTest extends TestCase
/**
* @expectedException \Symfony\Component\Process\Exception\RuntimeException
* @expectedExceptionMessage TTY mode is not supported on Windows platform.
+ * @group tty
*/
public function testTTYInWindowsEnvironment()
{
|