Improve code coverage

lkrms · Jan 7, 2025 · a68e157 · a68e157
1 parent d22210e
commit a68e157
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 5 deletions.
diff --git a/src/Rule/HangingIndentation.php b/src/Rule/HangingIndentation.php
@@ -49,7 +49,7 @@ public static function getPriority(string $method): ?int
     }
 
     /**
-     * @inheritDoc
+     * @codeCoverageIgnore
      */
     public static function needsSortedTokens(): bool
     {
@@ -358,7 +358,6 @@ public function processTokens(array $tokens): void
                 //   context, followed by the precedence value added previously
                 // - the token added to the last context, followed by the
                 //   precedence value added previously (if present)
-                // - `$lastToken` if no token was added to the last context
                 if (
                     TokenUtil::OPERATOR_PRECEDENCE_INDEX[$trigger->id]
                     && ($precedence = TokenUtil::getOperatorPrecedence($trigger)) < 99

diff --git a/src/Rule/StandardIndentation.php b/src/Rule/StandardIndentation.php
@@ -26,7 +26,7 @@ public static function getPriority(string $method): ?int
     }
 
     /**
-     * @inheritDoc
+     * @codeCoverageIgnore
      */
     public static function needsSortedTokens(): bool
     {

diff --git a/tests/unit/Rule/HangingIndentationTest.php b/tests/unit/Rule/HangingIndentationTest.php
@@ -2,6 +2,7 @@
 
 namespace Lkrms\PrettyPHP\Tests\Rule;
 
+use Lkrms\PrettyPHP\Catalog\FormatterFlag;
 use Lkrms\PrettyPHP\Rule\AlignChains;
 use Lkrms\PrettyPHP\Tests\TestCase;
 use Lkrms\PrettyPHP\Formatter;
@@ -25,7 +26,8 @@ public function testOutput(string $expected, string $code, $formatter): void
      */
     public static function outputProvider(): array
     {
-        $formatterB = Formatter::build();
+        $formatterB = Formatter::build()
+                          ->flags(FormatterFlag::DETECT_PROBLEMS);
         $formatter = $formatterB->build();
         $withAlignChains = $formatterB
                                ->enable([AlignChains::class])

diff --git a/tests/unit/TestCase.php b/tests/unit/TestCase.php
@@ -2,6 +2,7 @@
 
 namespace Lkrms\PrettyPHP\Tests;
 
+use Lkrms\PrettyPHP\Catalog\FormatterFlag;
 use Lkrms\PrettyPHP\Contract\Extension;
 use Lkrms\PrettyPHP\Contract\HasTokenNames;
 use Lkrms\PrettyPHP\Formatter;
@@ -43,14 +44,16 @@ public static function assertFormatterOutputIs(string $expected, string $code, $
      * @param array<class-string<Extension>> $enable
      * @param array<class-string<Extension>> $disable
      * @param 2|4|8 $tabSize
+     * @param int-mask-of<FormatterFlag::*> $flags
      */
     public static function assertCodeFormatIs(
         string $expected,
         string $code,
         array $enable = [],
         array $disable = [],
         bool $insertSpaces = true,
-        int $tabSize = 4
+        int $tabSize = 4,
+        int $flags = FormatterFlag::DETECT_PROBLEMS
     ): void {
         self::assertFormatterOutputIs(
             $expected,
@@ -60,6 +63,7 @@ public static function assertCodeFormatIs(
                 ->tabSize($tabSize)
                 ->enable($enable)
                 ->disable($disable)
+                ->flags($flags)
         );
     }
-Original file line number
+Diff line change
@@ Expand Up / @@ -26,7 +26,7 @@ public static function getPriority(string $method): ?int @@
         }
         /**
-         * @inheritDoc
+         * @codeCoverageIgnore
          */
         public static function needsSortedTokens(): bool
         {
@@ Expand Down @@