Prevent illogical values for high lower bound and low upper bound

sebastianbergmann · sebastianbergmann · commit 8eaae8d4211d · 2022-02-21T08:55:34.000+01:00
diff --git a/ChangeLog-8.5.md b/ChangeLog-8.5.md
@@ -8,6 +8,10 @@ All notable changes of the PHPUnit 8.5 release series are documented in this fil
 
 * [#4874](https://github.com/sebastianbergmann/phpunit/pull/4874): `PHP_FLOAT_EPSILON` is now used instead of hardcoded `0.0000000001` in `PHPUnit\Framework\Constraint\IsIdentical`
 
+### Fixed
+
+* When the HTML code coverage report's configured low upper bound is larger than the high lower bound then the default values are used instead
+
 ## [8.5.23] - 2022-01-21
 
 ### Fixed
diff --git a/src/TextUI/TestRunner.php b/src/TextUI/TestRunner.php
@@ -1285,6 +1285,11 @@ private function handleConfiguration(array &$arguments): void
         $arguments['timeoutForMediumTests']                           = $arguments['timeoutForMediumTests'] ?? 10;
         $arguments['timeoutForSmallTests']                            = $arguments['timeoutForSmallTests'] ?? 1;
         $arguments['verbose']                                         = $arguments['verbose'] ?? false;
+
+        if ($arguments['reportLowUpperBound'] > $arguments['reportHighLowerBound']) {
+            $arguments['reportLowUpperBound']  = 50;
+            $arguments['reportHighLowerBound'] = 90;
+        }
     }
 
     private function processSuiteFilters(TestSuite $suite, array $arguments): void
