[symfony] Skip symfony security in NoListenerWithoutContractRule

Author: TomasVotruba

src/Enum/SymfonyClass.php

@@ -6,6 +6,8 @@
 
 final class SymfonyClass
 {
+    public const SECURITY_LISTENER = 'Symfony\Component\Security\Http\Firewall\AbstractListener';
+
     public const ROUTE_ATTRIBUTE = 'Symfony\Component\Routing\Attribute\Route';
 
     /**

src/Rules/Symfony/NoListenerWithoutContractRule.php

@@ -79,6 +79,10 @@ public function processNode(Node $node, Scope $scope): array
             return [];
         }
 
+        if ($this->isSecurityListener($classLike)) {
+            return [];
+        }
+
         if ($this->hasAsListenerAttribute($classLike)) {
             return [];
         }
@@ -141,4 +145,13 @@ private function isFormEventsListener(Class_ $class): bool
 
         return false;
     }
+
+    private function isSecurityListener(Class_ $class): bool
+    {
+        if (! $class->extends instanceof Node\Name) {
+            return false;
+        }
+
+        return $class->extends->toString() === SymfonyClass::SECURITY_LISTENER;
+    }
 }

tests/Rules/Symfony/NoListenerWithoutContractRule/Fixture/SkipSecurityListener.php

@@ -0,0 +1,9 @@
+<?php
+
+namespace Symplify\PHPStanRules\Tests\Rules\Symfony\NoListenerWithoutContractRule\Fixture;
+
+final class SkipSecurityListener extends
+    \Symfony\Component\Security\Http\Firewall\AbstractListener
+{
+
+}

tests/Rules/Symfony/NoListenerWithoutContractRule/NoListenerWithoutContractRuleTest.php

@@ -21,6 +21,7 @@ public function testRule(array $filePaths, array $expectedErrorsWithLines): void
 
     public static function provideData(): Iterator
     {
+        yield [[__DIR__ . '/Fixture/SkipSecurityListener.php'], []];
         yield [[__DIR__ . '/Fixture/SkipFormListener.php'], []];
 
         yield [[__DIR__ . '/Fixture/SomeContractedListener.php'], []];