Skip to content

Commit b177d01

Browse files
Do not set TestCase::$status to TestStatus::risky()
TestCase::status() is only used by CodeCoverage::stop() to decide whether success or failure is communicated to php-code-coverage and we do not need to consider "test considered risky" for that.
1 parent 1381c62 commit b177d01

File tree

1 file changed

+0
-3
lines changed

1 file changed

+0
-3
lines changed

src/Framework/TestCase.php

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -722,7 +722,6 @@ final public function runBare(): void
722722
Event\Code\ComparisonFailureBuilder::from($e),
723723
);
724724
} catch (TimeoutException $e) {
725-
$this->status = TestStatus::risky($e->getMessage());
726725
} catch (Throwable $_e) {
727726
if ($this->isRegisteredFailure($_e)) {
728727
$this->status = TestStatus::failure($_e->getMessage());
@@ -1747,8 +1746,6 @@ private function stopOutputBuffering(): bool
17471746
$message,
17481747
);
17491748

1750-
$this->status = TestStatus::risky($message);
1751-
17521749
return false;
17531750
}
17541751

0 commit comments

Comments
 (0)