Skip to content

Commit

Permalink
Merge pull request #79 from clue-labs/unhandled-rejections-v4
Browse files Browse the repository at this point in the history
[4.x] Update test suite to avoid unhandled promise rejections
  • Loading branch information
WyriHaximus authored Jul 7, 2023
2 parents b9641ac + 9b58514 commit 307684c
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions tests/CoroutineTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -193,6 +193,8 @@ public function testCoroutineShouldNotCreateAnyGarbageReferencesForPromiseReject
});
});

$promise->then(null, $this->expectCallableOnce()); // avoid reporting unhandled rejection

unset($promise);

$this->assertEquals(0, gc_collect_cycles());
Expand Down Expand Up @@ -232,6 +234,8 @@ public function testCoroutineShouldNotCreateAnyGarbageReferencesWhenGeneratorThr
yield; // @phpstan-ignore-line
});

$promise->then(null, $this->expectCallableOnce()); // avoid reporting unhandled rejection

unset($promise);

$this->assertEquals(0, gc_collect_cycles());
Expand All @@ -249,6 +253,8 @@ public function testCoroutineShouldNotCreateAnyGarbageReferencesWhenGeneratorYie
yield 42;
});

$promise->then(null, $this->expectCallableOnce()); // avoid reporting unhandled rejection

unset($promise);

$this->assertEquals(0, gc_collect_cycles());
Expand Down

0 comments on commit 307684c

Please sign in to comment.