fix(phpunit): add warnings for invalid console_output and simplify renderer logic

Add stderr warnings when invalid values are provided for the
OPENAPI_CONSOLE_OUTPUT env var or console_output phpunit.xml parameter.
Remove arithmetic-derived uncoveredCount in favor of count() on the
actual uncovered array. Add edge-case tests for zero coverage, full
coverage, and percentage rounding.

Author: wadakatu

src/PHPUnit/ConsoleCoverageRenderer.php

@@ -4,6 +4,7 @@
 
 namespace Studio\OpenApiContractTesting\PHPUnit;
 
+use function count;
 use function round;
 use function str_repeat;
 
@@ -27,13 +28,12 @@ public static function render(array $results, ConsoleOutput $consoleOutput = Con
 
         foreach ($results as $spec => $result) {
             $percentage = self::percentage($result['coveredCount'], $result['total']);
-            $uncoveredCount = $result['total'] - $result['coveredCount'];
 
             $output .= "\n[{$spec}] {$result['coveredCount']}/{$result['total']} endpoints ({$percentage}%)\n";
             $output .= str_repeat('-', 50) . "\n";
 
-            $output .= self::renderCovered($result, $consoleOutput, $uncoveredCount);
-            $output .= self::renderUncovered($result, $consoleOutput, $uncoveredCount);
+            $output .= self::renderCovered($result, $consoleOutput);
+            $output .= self::renderUncovered($result, $consoleOutput);
         }
 
         $output .= "\n";
@@ -44,7 +44,7 @@ public static function render(array $results, ConsoleOutput $consoleOutput = Con
     /**
      * @param CoverageResult $result
      */
-    private static function renderCovered(array $result, ConsoleOutput $consoleOutput, int $uncoveredCount): string
+    private static function renderCovered(array $result, ConsoleOutput $consoleOutput): string
     {
         if ($result['covered'] === []) {
             return '';
@@ -66,12 +66,14 @@ private static function renderCovered(array $result, ConsoleOutput $consoleOutpu
     /**
      * @param CoverageResult $result
      */
-    private static function renderUncovered(array $result, ConsoleOutput $consoleOutput, int $uncoveredCount): string
+    private static function renderUncovered(array $result, ConsoleOutput $consoleOutput): string
     {
-        if ($uncoveredCount <= 0) {
+        if ($result['uncovered'] === []) {
             return '';
         }
 
+        $uncoveredCount = count($result['uncovered']);
+
         if ($consoleOutput === ConsoleOutput::DEFAULT) {
             return "Uncovered: {$uncoveredCount} endpoints\n";
         }

src/PHPUnit/ConsoleOutput.php

@@ -4,6 +4,9 @@
 
 namespace Studio\OpenApiContractTesting\PHPUnit;
 
+use const STDERR;
+
+use function fwrite;
 use function getenv;
 use function mb_strtolower;
 use function trim;
@@ -26,12 +29,20 @@ public static function resolve(?string $parameterValue): self
         if ($envValue !== false && trim($envValue) !== '') {
             $resolved = self::tryFrom(mb_strtolower(trim($envValue)));
 
+            if ($resolved === null) {
+                fwrite(STDERR, "[OpenAPI Coverage] WARNING: Invalid OPENAPI_CONSOLE_OUTPUT value '{$envValue}'. Valid values: default, all, uncovered_only. Falling back to 'default'.\n");
+            }
+
             return $resolved ?? self::DEFAULT;
         }
 
         if ($parameterValue !== null && trim($parameterValue) !== '') {
             $resolved = self::tryFrom(mb_strtolower(trim($parameterValue)));
 
+            if ($resolved === null) {
+                fwrite(STDERR, "[OpenAPI Coverage] WARNING: Invalid console_output parameter '{$parameterValue}'. Valid values: default, all, uncovered_only. Falling back to 'default'.\n");
+            }
+
             return $resolved ?? self::DEFAULT;
         }
 

tests/Unit/ConsoleCoverageRendererTest.php

@@ -222,6 +222,62 @@ public function render_header_and_separators_are_present(): void
         $this->assertStringContainsString('--------------------------------------------------', $output);
     }
 
+    #[Test]
+    public function render_zero_coverage_all_mode_shows_uncovered_list_only(): void
+    {
+        $results = [
+            'front' => [
+                'covered' => [],
+                'uncovered' => ['GET /v1/pets', 'POST /v1/pets'],
+                'total' => 2,
+                'coveredCount' => 0,
+            ],
+        ];
+
+        $output = ConsoleCoverageRenderer::render($results, ConsoleOutput::ALL);
+
+        $this->assertStringContainsString('[front] 0/2 endpoints (0%)', $output);
+        $this->assertStringNotContainsString('Covered', $output);
+        $this->assertStringContainsString('Uncovered:', $output);
+        $this->assertStringContainsString('  ✗ GET /v1/pets', $output);
+    }
+
+    #[Test]
+    public function render_full_coverage_uncovered_only_mode_shows_covered_count_only(): void
+    {
+        $results = [
+            'front' => [
+                'covered' => ['GET /v1/pets', 'POST /v1/pets'],
+                'uncovered' => [],
+                'total' => 2,
+                'coveredCount' => 2,
+            ],
+        ];
+
+        $output = ConsoleCoverageRenderer::render($results, ConsoleOutput::UNCOVERED_ONLY);
+
+        $this->assertStringContainsString('Covered: 2 endpoints', $output);
+        $this->assertStringNotContainsString('  ✓', $output);
+        $this->assertStringNotContainsString('Uncovered', $output);
+    }
+
+    #[Test]
+    public function render_percentage_rounds_to_one_decimal_place(): void
+    {
+        $results = [
+            'front' => [
+                'covered' => ['GET /v1/pets'],
+                'uncovered' => ['POST /v1/pets', 'DELETE /v1/pets/{petId}'],
+                'total' => 3,
+                'coveredCount' => 1,
+            ],
+        ];
+
+        $output = ConsoleCoverageRenderer::render($results);
+
+        $this->assertStringContainsString('[front] 1/3 endpoints (33.3%)', $output);
+    }
+
     #[Test]
     public function render_spec_with_zero_endpoints(): void
     {