Merge pull request #9 from WyriHaximus/rework-internals-to-be-more-reliable

Rework internals to be more reliable

Author: WyriHaximus

src/RunTestsInFibersTrait.php

@@ -4,16 +4,15 @@
 
 namespace WyriHaximus\React\PHPUnit;
 
-use React\Promise\PromiseInterface;
+use React\EventLoop\Loop;
+use React\Promise\Deferred;
 use ReflectionClass;
 
 use function assert;
 use function is_string;
 use function React\Async\async;
 use function React\Async\await;
 use function React\Promise\race;
-use function React\Promise\reject;
-use function React\Promise\Timer\sleep;
 
 trait RunTestsInFibersTrait
 {
@@ -69,15 +68,18 @@ final protected function runAsyncTest(mixed ...$args): mixed
             $timeout = $methodTimeout->timeout();
         }
 
+        $sleepingDeferred = new Deferred();
+        $sleepTimer       = Loop::addTimer($timeout, static fn () => $sleepingDeferred->reject(new TimedOut('Test timed out after ' . $timeout . ' second(s)')));
+
         /**
          * @psalm-suppress MixedArgument
          * @psalm-suppress UndefinedInterfaceMethod
          */
         return await(race([
             async(
                 fn (): mixed => ([$this, $this->realTestName])(...$args), /** @phpstan-ignore-line */
-            )(),
-            sleep($timeout)->then(static fn (): PromiseInterface => reject(new TimedOut('Test timed out after ' . $timeout . ' second(s)'))),
+            )()->always(static fn () => Loop::cancelTimer($sleepTimer)),
+            $sleepingDeferred->promise(),
         ]));
     }