Add tests

VincentLanglet · Jan 5, 2024 · b8e952d · b8e952d
1 parent 2c309bc
commit b8e952d
Show file tree

Hide file tree

Showing 3 changed files with 62 additions and 1 deletion.
diff --git a/tests/Report/ViolationIdTest.php b/tests/Report/ViolationIdTest.php
@@ -48,4 +48,32 @@ public static function toStringDataProvider(): iterable
         yield ['short', 'id', 'token', 1, null, 'short.id.token:1'];
         yield ['short', 'id', 'token', 1, 1, 'short.id.token:1:1'];
     }
+
+    /**
+     * @dataProvider matchDataProvider
+     */
+    public function testMatch(string $string1, string $string2, bool $expected,): void
+    {
+        $violationId1 = ViolationId::fromString($string1);
+        $violationId2 = ViolationId::fromString($string2);
+        static::assertSame($expected, $violationId1->match($violationId2));
+    }
+
+    /**
+     * @return iterable<array-key, array{string, string, bool}>
+     */
+    public static function matchDataProvider(): iterable
+    {
+        yield ['short', 'short', true];
+        yield ['short', 'short.id.token:1:1', true];
+        yield ['short.id', 'short.id.token:1:1', true];
+        yield ['short.notId', 'short.id.token:1:1', false];
+        yield ['short.id', 'short', false];
+        yield ['SHORT.ID.TOKEN', 'short.id.token:1:1', true];
+        yield ['short.id.token:2:1', 'short.id.token:1:1', false];
+        yield ['short.id.token:1:2', 'short.id.token:1:1', false];
+        yield ['short::1', 'short.id.token:1:1', true];
+        yield ['short.id::1', 'short.id.token:1:1', true];
+        yield ['short.id.token::1', 'short.id.token:1:1', true];
+    }
 }
diff --git a/tests/Rules/RuleTest.php b/tests/Rules/RuleTest.php
@@ -4,7 +4,6 @@
 
 namespace TwigCsFixer\Tests\Rules;
 
-use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 use SplFileInfo;
 use TwigCsFixer\Environment\StubbedEnvironment;

diff --git a/tests/Runner/FixerTest.php b/tests/Runner/FixerTest.php
@@ -206,6 +206,40 @@ protected function process(int $tokenPosition, array $tokens): void
         static::assertSame('test 2 2', $fixer->fixFile('test test test', $ruleset));
     }
 
+    public function testIgnoredViolations(): void
+    {
+        $tokenizer = new Tokenizer(new StubbedEnvironment());
+
+        $rule = new class () extends AbstractRule {
+            public function getShortName(): string
+            {
+                return 'Rule';
+            }
+
+            protected function process(int $tokenPosition, array $tokens): void
+            {
+                $fixer = $this->addFixableWarning('Error', $tokens[$tokenPosition]);
+                if (null !== $fixer) {
+                    $fixer->replaceToken($tokenPosition, 'a');
+                }
+
+                $fixer = $this->addFixableError('Error', $tokens[$tokenPosition]);
+                if (null !== $fixer) {
+                    $fixer->replaceToken($tokenPosition, 'b');
+                }
+            }
+        };
+
+        $ruleset = new Ruleset();
+        $ruleset->addRule($rule);
+
+        $fixer = new Fixer($tokenizer);
+
+        $content = '{# twig-cs-fixer-disable Rule #}';
+        // The rule should produce an infinite loop but the comment disable it
+        static::assertSame($content, $fixer->fixFile('{# twig-cs-fixer-disable Rule #}', $ruleset));
+    }
+
     /**
      * @dataProvider addContentMethodsDataProvider
      */