Closes #4376

Author: sebastianbergmann

ChangeLog-9.2.md

@@ -2,6 +2,12 @@
 
 All notable changes of the PHPUnit 9.2 release series are documented in this file using the [Keep a CHANGELOG](https://keepachangelog.com/) principles.
 
+## [9.2.6] - 2020-MM-DD
+
+### Fixed
+
+* [#4376](https://github.com/sebastianbergmann/phpunit/issues/4376): Risky test that triggers an error is displayed incorrectly
+
 ## [9.2.5] - 2020-06-22
 
 ### Fixed
@@ -50,6 +56,7 @@ All notable changes of the PHPUnit 9.2 release series are documented in this fil
 * [#4258](https://github.com/sebastianbergmann/phpunit/pull/4258): Prevent unpredictable result by raising an exception when multiple matchers can be applied to a test double invocation
 * The test runner no longer relies on `$_SERVER['REQUEST_TIME_FLOAT']` for printing the elapsed time
 
+[9.2.6]: https://github.com/sebastianbergmann/phpunit/compare/9.2.5...9.2
 [9.2.5]: https://github.com/sebastianbergmann/phpunit/compare/9.2.4...9.2.5
 [9.2.4]: https://github.com/sebastianbergmann/phpunit/compare/9.2.3...9.2.4
 [9.2.3]: https://github.com/sebastianbergmann/phpunit/compare/9.2.2...9.2.3

src/Framework/TestResult.php

@@ -776,7 +776,7 @@ public function run(Test $test): void
             $risky = true;
         }
 
-        if ($this->forceCoversAnnotation && !$incomplete && !$skipped) {
+        if ($this->forceCoversAnnotation && !$error && !$failure && !$warning && !$incomplete && !$skipped && !$risky) {
             $annotations = $test->getAnnotations();
 
             if (!isset($annotations['class']['covers']) &&

tests/end-to-end/regression/GitHub/4376.phpt

@@ -0,0 +1,25 @@
+--TEST--
+https://github.com/sebastianbergmann/phpunit/issues/4376
+--FILE--
+<?php declare(strict_types=1);
+$_SERVER['argv'][1] = '--configuration';
+$_SERVER['argv'][2] = __DIR__ . '/4376/';
+
+require __DIR__ . '/../../../bootstrap.php';
+PHPUnit\TextUI\Command::main();
+--EXPECTF--
+PHPUnit %s by Sebastian Bergmann and contributors.
+
+E                                                                   1 / 1 (100%)
+
+Time: %s, Memory: %s
+
+There was 1 error:
+
+1) Test::testOne
+Error: Class 'C' not found
+
+%sTest.php:%d
+
+ERRORS!
+Tests: 1, Assertions: 0, Errors: 1.

tests/end-to-end/regression/GitHub/4376/phpunit.xml

@@ -0,0 +1,10 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/9.3/phpunit.xsd"
+         forceCoversAnnotation="true">
+    <testsuites>
+        <testsuite name="default">
+            <directory suffix="Test.php">tests</directory>
+        </testsuite>
+    </testsuites>
+</phpunit>

tests/end-to-end/regression/GitHub/4376/tests/Test.php

@@ -0,0 +1,11 @@
+<?php declare(strict_types=1);
+use PHPUnit\Framework\TestCase;
+
+final class Test extends TestCase
+{
+    public function testOne(): void
+    {
+        $o = new C;
+    }
+}
+