Also use Thresholds value object to configure Text report

Author: sebastianbergmann

src/Report/Html/Facade.php

@@ -16,6 +16,7 @@
 use function str_ends_with;
 use SebastianBergmann\CodeCoverage\CodeCoverage;
 use SebastianBergmann\CodeCoverage\Node\Directory as DirectoryNode;
+use SebastianBergmann\CodeCoverage\Report\Thresholds;
 use SebastianBergmann\CodeCoverage\Util\Filesystem;
 use SebastianBergmann\Template\Template;
 

src/Report/Html/Renderer.php

@@ -17,6 +17,7 @@
 use SebastianBergmann\CodeCoverage\Node\AbstractNode;
 use SebastianBergmann\CodeCoverage\Node\Directory as DirectoryNode;
 use SebastianBergmann\CodeCoverage\Node\File as FileNode;
+use SebastianBergmann\CodeCoverage\Report\Thresholds;
 use SebastianBergmann\CodeCoverage\Version;
 use SebastianBergmann\Environment\Runtime;
 use SebastianBergmann\Template\Template;

src/Report/Text.php

@@ -53,18 +53,15 @@ final class Text
      */
     private const COLOR_EOL = "\x1b[2K";
 
-    private int $lowUpperBound;
-
-    private int $highLowerBound;
+    private Thresholds $thresholds;
 
     private bool $showUncoveredFiles;
 
     private bool $showOnlySummary;
 
-    public function __construct(int $lowUpperBound = 50, int $highLowerBound = 90, bool $showUncoveredFiles = false, bool $showOnlySummary = false)
+    public function __construct(Thresholds $thresholds, bool $showUncoveredFiles = false, bool $showOnlySummary = false)
     {
-        $this->lowUpperBound      = $lowUpperBound;
-        $this->highLowerBound     = $highLowerBound;
+        $this->thresholds         = $thresholds;
         $this->showUncoveredFiles = $showUncoveredFiles;
         $this->showOnlySummary    = $showOnlySummary;
     }
@@ -294,11 +291,11 @@ private function coverageColor(int $numberOfCoveredElements, int $totalNumberOfE
             $totalNumberOfElements
         );
 
-        if ($coverage->asFloat() >= $this->highLowerBound) {
+        if ($coverage->asFloat() >= $this->thresholds->highLowerBound()) {
             return self::COLOR_GREEN;
         }
 
-        if ($coverage->asFloat() > $this->lowUpperBound) {
+        if ($coverage->asFloat() > $this->thresholds->lowUpperBound()) {
             return self::COLOR_YELLOW;
         }
 

src/Report/Html/Thresholds.php renamed to src/Report/Thresholds.php

@@ -7,7 +7,7 @@
  * For the full copyright and license information, please view the LICENSE
  * file that was distributed with this source code.
  */
-namespace SebastianBergmann\CodeCoverage\Report\Html;
+namespace SebastianBergmann\CodeCoverage\Report;
 
 use SebastianBergmann\CodeCoverage\InvalidArgumentException;
 

tests/tests/Report/Html/ThresholdsTest.php

@@ -13,6 +13,7 @@
 use PHPUnit\Framework\Attributes\Small;
 use PHPUnit\Framework\TestCase;
 use SebastianBergmann\CodeCoverage\InvalidArgumentException;
+use SebastianBergmann\CodeCoverage\Report\Thresholds;
 
 #[CoversClass(Thresholds::class)]
 #[Small]

tests/tests/Report/TextTest.php

@@ -19,7 +19,7 @@ final class TextTest extends TestCase
 {
     public function testLineCoverageForBankAccountTest(): void
     {
-        $text = new Text(50, 90, false, false);
+        $text = new Text(Thresholds::default(), false, false);
 
         $this->assertStringMatchesFormatFile(
             TEST_FILES_PATH . 'BankAccount-text-line.txt',
@@ -29,7 +29,7 @@ public function testLineCoverageForBankAccountTest(): void
 
     public function testPathCoverageForBankAccountTest(): void
     {
-        $text = new Text(50, 90, false, false);
+        $text = new Text(Thresholds::default(), false, false);
 
         $this->assertStringMatchesFormatFile(
             TEST_FILES_PATH . 'BankAccount-text-path.txt',
@@ -39,7 +39,7 @@ public function testPathCoverageForBankAccountTest(): void
 
     public function testTextOnlySummaryForBankAccountTest(): void
     {
-        $text = new Text(50, 90, false, true);
+        $text = new Text(Thresholds::default(), false, true);
 
         $this->assertStringMatchesFormatFile(
             TEST_FILES_PATH . 'BankAccount-text-summary.txt',
@@ -49,7 +49,7 @@ public function testTextOnlySummaryForBankAccountTest(): void
 
     public function testTextForNamespacedBankAccountTest(): void
     {
-        $text = new Text(50, 90, true, false);
+        $text = new Text(Thresholds::default(), true, false);
 
         $this->assertStringMatchesFormatFile(
             TEST_FILES_PATH . 'NamespacedBankAccount-text.txt',
@@ -59,7 +59,7 @@ public function testTextForNamespacedBankAccountTest(): void
 
     public function testTextForFileWithIgnoredLines(): void
     {
-        $text = new Text(50, 90, false, false);
+        $text = new Text(Thresholds::default(), false, false);
 
         $this->assertStringMatchesFormatFile(
             TEST_FILES_PATH . 'ignored-lines-text.txt',
@@ -69,7 +69,7 @@ public function testTextForFileWithIgnoredLines(): void
 
     public function testTextForClassWithAnonymousFunction(): void
     {
-        $text = new Text(50, 90, false, false);
+        $text = new Text(Thresholds::default(), false, false);
 
         $this->assertStringMatchesFormatFile(
             TEST_FILES_PATH . 'class-with-anonymous-function-text.txt',
@@ -79,7 +79,7 @@ public function testTextForClassWithAnonymousFunction(): void
 
     public function testUncoveredFilesAreIncludedWhenConfiguredTest(): void
     {
-        $text = new Text(50, 90, false, false);
+        $text = new Text(Thresholds::default(), false, false);
 
         $this->assertStringMatchesFormatFile(
             TEST_FILES_PATH . 'BankAccountWithUncovered-text-line.txt',
@@ -89,7 +89,7 @@ public function testUncoveredFilesAreIncludedWhenConfiguredTest(): void
 
     public function testUncoveredFilesAreExcludedWhenConfiguredTest(): void
     {
-        $text = new Text(50, 90, false, false);
+        $text = new Text(Thresholds::default(), false, false);
 
         $this->assertStringMatchesFormatFile(
             TEST_FILES_PATH . 'BankAccountWithoutUncovered-text-line.txt',