danog/process
1
0
Fork 0
mirror of https://github.com/danog/process.git synced 2024-11-29 20:29:16 +01:00
Code Issues Projects Releases Wiki Activity

Update tests

Browse Source 
Improved some tests and (hopefully) fixed a timing issue on windows in kill test.
This commit is contained in:
Aaron Piotrowski 2017-12-06 20:41:07 -06:00
parent aeb22b9bda
commit a4b1fa64fa
No known key found for this signature in database
GPG Key ID: ADD1EF783EDE9EEB
1 changed files with 21 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
test/ProcessTest.php
View File

@ -2,6 +2,7 @@
namespace Amp\Process\Test;
use Amp\Delayed;
use Amp\Loop;
use Amp\Process\Process;
use Amp\Process\ProcessInputStream;
@ -56,7 +57,7 @@ class ProcessTest extends TestCase {
$process->start();
$this->assertInternalType('int', yield $process->getPid());
yield $process->join();
$this->assertSame(0, yield $process->join());
});
}
@ -70,7 +71,7 @@ class ProcessTest extends TestCase {
$process->start();
$process->signal(0);
$this->assertInstanceOf(Promise::class, $process->getPid());
yield $process->join();
$this->assertSame(0, yield $process->join());
});
}
@ -184,10 +185,26 @@ class ProcessTest extends TestCase {
* @expectedException \Amp\Process\ProcessException
* @expectedExceptionMessage The process was killed
*/
public function testKillSignals() {
public function testKillImmediately() {
Loop::run(function () {
$process = new Process(self::CMD_PROCESS_SLOW);
$process->start();
$process->kill();
yield $process->join();
});
}
/**
* @expectedException \Amp\Process\ProcessException
* @expectedExceptionMessage The process was killed
*/
public function testKillThenReadStdout() {
Loop::run(function () {
$process = new Process(self::CMD_PROCESS_SLOW);
$process->start();
yield new Delayed(100); // Give process a chance to start, otherwise a different error is thrown.
$process->kill();
$this->assertNull(yield $process->getStdout()->read());
@ -196,6 +213,7 @@ class ProcessTest extends TestCase {
});
}
/**
* @expectedException \Amp\Process\StatusError
* @expectedExceptionMessage Process has not been started.