Merge pull request #51 from andrewminerd/timer_fix

StreamSelectLoop: Use fresh time so Timers added during stream events are accurate

Author: clue

src/Timer/Timers.php

@@ -30,7 +30,7 @@ public function getTime()
     public function add(TimerInterface $timer)
     {
         $interval = $timer->getInterval();
-        $scheduledAt = $interval + $this->getTime();
+        $scheduledAt = $interval + microtime(true);
 
         $this->timers->attach($timer, $scheduledAt);
         $this->scheduler->insert($timer, -$scheduledAt);

tests/Timer/TimersTest.php

@@ -0,0 +1,29 @@
+<?php
+
+namespace React\Tests\EventLoop\Timer;
+
+use React\Tests\EventLoop\TestCase;
+use React\EventLoop\Timer\Timer;
+use React\EventLoop\Timer\Timers;
+
+class TimersTest extends TestCase
+{
+    public function testBlockedTimer()
+    {
+        $loop = $this
+            ->getMockBuilder('React\EventLoop\LoopInterface')
+            ->getMock();
+        
+        $timers = new Timers();
+        $timers->tick();
+        
+        // simulate a bunch of processing on stream events,
+        // part of which schedules a future timer...
+        sleep(1);
+        $timers->add(new Timer($loop, 0.5, function () {
+            $this->fail("Timer shouldn't be called");
+        }));
+
+        $timers->tick();
+    }
+}