Merge pull request #32 from Icinga/require-php-event-ext

Suggest PHP `ext-ev`

.github/workflows/php.yml

@@ -27,6 +27,7 @@ jobs:
         with:
           php-version: ${{ matrix.php }}
           tools: phpcs
+          extensions: ev
 
       - name: Setup dependencies
         run: composer require -n --no-progress overtrue/phplint
@@ -61,6 +62,7 @@ jobs:
         with:
           php-version: ${{ matrix.php }}
           tools: phpunit:${{ env.PHPUNIT_VERSION }}
+          extensions: ev
 
       - name: Setup dependencies
         run: composer install -n --no-progress

composer.json

@@ -19,6 +19,9 @@
     "simshaun/recurr": "^5",
     "ipl/stdlib": ">=0.12.0"
   },
+  "suggest": {
+    "ext-ev": "Improves performance, efficiency and avoids system limitations. Highly recommended! (See https://www.php.net/manual/en/intro.ev.php for details)"
+  },
   "autoload": {
     "files": ["src/register_cron_aliases.php"],
     "psr-4": {

tests/SchedulerTest.php

@@ -114,14 +114,14 @@ function (Task $_, array $promises) use (&$canceledPromises): void {
                 }
             );
 
-        // Tick callbacks are guaranteed to be executed in the order they are enqueued,
-        // thus we will not be able to remove the task before it is finished. Therefore,
-        // we need to use timers with 0 intervals.
-        Loop::addTimer(0, function () use ($task): void {
+        // Wait .001ms for the scheduler to run the task at least 2x before removing it und stopping the event loop.
+        Loop::addTimer(.001, function () use ($task): void {
             $this->scheduler->remove($task);
+
+            Loop::stop();
         });
 
-        $this->runAndStopEventLoop();
+        Loop::run();
 
         // When a task is due before the ongoing promise gets resolved, the scheduler will
         // not cancel it. Instead, it will start a new one for the new operations.