danog/amp
1
0
Fork 0
mirror of https://github.com/danog/amp.git synced 2024-11-27 04:24:42 +01:00
Code Issues Projects Releases Wiki Activity

Do not depend on watcher execution order

Browse Source 
Loop driver spec says order is not guaranteed.
This commit is contained in:
Aaron Piotrowski 2017-04-21 12:18:40 -05:00
parent a562d6b20f
commit e75c62a5fd
1 changed files with 11 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
test/Loop/DriverTest.php
View File

@ -1382,7 +1382,6 @@ abstract class DriverTest extends TestCase {
$watcher2 = $this->loop->onReadable($sockets[0], function ($watcher) use (&$invoked, $watcher1) {
$invoked += 10;
$this->loop->disable($watcher);
$this->loop->enable($watcher1);
});
$watcher3 = $this->loop->onWritable($sockets[0], function ($watcher) use (&$invoked) {
$invoked += 100;
@ -1391,6 +1390,10 @@ abstract class DriverTest extends TestCase {
$watcher4 = $this->loop->onWritable($sockets[0], function ($watcher) use (&$invoked, $watcher3) {
$invoked += 1000;
$this->loop->disable($watcher);
});
$this->loop->delay(100, function () use ($watcher1, $watcher3) {
$this->loop->enable($watcher1);
$this->loop->enable($watcher3);
});
@ -1398,8 +1401,13 @@ abstract class DriverTest extends TestCase {
$this->assertSame(1212, $invoked);
$this->loop->enable($watcher1);
$this->loop->enable($watcher3);
$this->loop->delay(100, function () use ($watcher2, $watcher4) {
$this->loop->enable($watcher2);
$this->loop->enable($watcher4);
});
$this->loop->run();