From 531cd497c50f1aaa47408aa772d67414b1bd2020 Mon Sep 17 00:00:00 2001
From: Simon Podlipsky <simon@podlipsky.net>
Date: Mon, 16 Sep 2024 10:12:27 +0200
Subject: [PATCH] test: cover false positive where types are not considered
 during Twig analysis

---
 .../Fixture/DontSkipTwigEntityWithMethods.php        | 12 ++++++++++++
 .../TemplatePathsTest.php                            |  9 ++++++++-
 2 files changed, 20 insertions(+), 1 deletion(-)
 create mode 100644 tests/Rules/UnusedPublicClassMethodRule/Fixture/DontSkipTwigEntityWithMethods.php

diff --git a/tests/Rules/UnusedPublicClassMethodRule/Fixture/DontSkipTwigEntityWithMethods.php b/tests/Rules/UnusedPublicClassMethodRule/Fixture/DontSkipTwigEntityWithMethods.php
new file mode 100644
index 0000000..caab957
--- /dev/null
+++ b/tests/Rules/UnusedPublicClassMethodRule/Fixture/DontSkipTwigEntityWithMethods.php
@@ -0,0 +1,12 @@
+<?php
+
+declare(strict_types=1);
+
+namespace TomasVotruba\UnusedPublic\Tests\Rules\UnusedPublicClassMethodRule\Fixture;
+
+final class DontSkipTwigEntityWithMethods
+{
+    public function getComplicatedValue()
+    {
+    }
+}
diff --git a/tests/Rules/UnusedPublicClassMethodRule/TemplatePathsTest.php b/tests/Rules/UnusedPublicClassMethodRule/TemplatePathsTest.php
index f09b0d7..9be38a2 100644
--- a/tests/Rules/UnusedPublicClassMethodRule/TemplatePathsTest.php
+++ b/tests/Rules/UnusedPublicClassMethodRule/TemplatePathsTest.php
@@ -14,6 +14,7 @@
 use TomasVotruba\UnusedPublic\Collectors\PublicClassMethodCollector;
 use TomasVotruba\UnusedPublic\Collectors\StaticCall\StaticMethodCallCollector;
 use TomasVotruba\UnusedPublic\Rules\UnusedPublicClassMethodRule;
+use TomasVotruba\UnusedPublic\Tests\Rules\UnusedPublicClassMethodRule\Fixture\DontSkipTwigEntityWithMethods;
 
 final class TemplatePathsTest extends RuleTestCase
 {
@@ -26,7 +27,13 @@ public function testRule(array $filePaths, array $expectedErrorMessagesWithLines
 
     public static function provideDataWithTwigTemplates(): Iterator
     {
-        yield [[__DIR__ . '/Fixture/SkipTwigEntityWithMethods.php'], []];
+        $errorMessage = sprintf(
+            UnusedPublicClassMethodRule::ERROR_MESSAGE,
+            DontSkipTwigEntityWithMethods::class,
+            'getComplicatedValue'
+        );
+
+        yield [[__DIR__ . '/Fixture/SkipTwigEntityWithMethods.php', __DIR__ . '/Fixture/DontSkipTwigEntityWithMethods.php'], [[$errorMessage, 9]]];
 
         // this method is required by parent contract, and should be skipped
         yield [[