Change test to ensure that a periodic timer created with an interval of zero fires at least twice

Author: lucasnetau

tests/Timer/AbstractTimerTest.php

@@ -121,28 +121,24 @@ public function testMinimumIntervalOneMicrosecond()
         $this->assertEquals(0.000001, $timer->getInterval());
     }
 
-    public function testAddPeriodicTimerWithZeroIntervalWillBeInvokedWithMaximumAccuracyUntilItIsCancelled()
+    public function testAddPeriodicTimerWithZeroIntervalWillExecuteCallbackFunctionAtLeastTwice()
     {
         $loop = $this->createLoop();
 
+        $timeout = $loop->addTimer(2, $this->expectCallableNever()); //Timeout the test after two seconds if the periodic timer hasn't fired twice
+
         $i = 0;
-        $periodic = $loop->addPeriodicTimer(0, function () use (&$i) {
+        $loop->addPeriodicTimer(0, function ($timer) use (&$i, $loop, $timeout) {
             ++$i;
-        });
-
-        $this->assertEquals($periodic::MIN_INTERVAL, $periodic->getInterval());
-
-        $loop->addTimer(0.001, function () use ($loop, $periodic) {
-            $loop->cancelTimer($periodic);
+            if ($i === 2) {
+                $loop->cancelTimer($timer);
+                $loop->cancelTimer($timeout);
+            }
         });
 
         $loop->run();
 
-        // make no strict assumptions about number of invocations.
-        // we know it must be no more than 1000 times (1000 microseconds in 0.001 seconds) and should at least be
-        // invoked at least 2 times to prove it is repeating
-        $this->assertLessThanOrEqual(1000, $i);
-        $this->assertGreaterThanOrEqual(2, $i);
+        $this->assertEquals(2, $i);
     }
 
     public function testTimerIntervalBelowZeroRunsImmediately()