Skip to content

Commit 5939d34

Browse files
romainneutronxabbuh
authored andcommitted
[Process] Fix unit tests in sigchild environment
1 parent eb68662 commit 5939d34

File tree

3 files changed

+27
-0
lines changed

3 files changed

+27
-0
lines changed

src/Symfony/Component/Process/Tests/SigchildDisabledProcessTest.php

+5
Original file line numberDiff line numberDiff line change
@@ -211,6 +211,11 @@ public function testExitCodeIsAvailableAfterSignal()
211211
$this->markTestSkipped('Signal is not supported in sigchild environment');
212212
}
213213

214+
public function testRunProcessWithTimeout()
215+
{
216+
$this->markTestSkipped('Signal (required for timeout) is not supported in sigchild environment');
217+
}
218+
214219
/**
215220
* {@inheritdoc}
216221
*/

src/Symfony/Component/Process/Tests/SigchildEnabledProcessTest.php

+15
Original file line numberDiff line numberDiff line change
@@ -120,6 +120,21 @@ public function testStartAfterATimeout()
120120
parent::testStartAfterATimeout();
121121
}
122122

123+
public function testStopWithTimeoutIsActuallyWorking()
124+
{
125+
$this->markTestSkipped('Stopping with signal is not supported in sigchild environment');
126+
}
127+
128+
public function testRunProcessWithTimeout()
129+
{
130+
$this->markTestSkipped('Signal (required for timeout) is not supported in sigchild environment');
131+
}
132+
133+
public function testCheckTimeoutOnStartedProcess()
134+
{
135+
$this->markTestSkipped('Signal (required for timeout) is not supported in sigchild environment');
136+
}
137+
123138
/**
124139
* {@inheritdoc}
125140
*/

src/Symfony/Component/Process/Tests/SimpleProcessTest.php

+7
Original file line numberDiff line numberDiff line change
@@ -191,6 +191,13 @@ public function testTermSignalTerminatesProcessCleanly()
191191
}
192192
}
193193

194+
public function testStopWithTimeoutIsActuallyWorking()
195+
{
196+
$this->skipIfPHPSigchild();
197+
198+
parent::testStopWithTimeoutIsActuallyWorking();
199+
}
200+
194201
/**
195202
* {@inheritdoc}
196203
*/

0 commit comments

Comments
 (0)