Add test for NoDocumentMockingRule and skip interface/abstract classes

Author: TomasVotruba

src/Rules/Doctrine/NoDocumentMockingRule.php

@@ -7,18 +7,25 @@
 use PhpParser\Node;
 use PhpParser\Node\Expr\MethodCall;
 use PHPStan\Analyser\Scope;
+use PHPStan\Reflection\ReflectionProvider;
 use PHPStan\Rules\Rule;
 use PHPStan\Rules\RuleErrorBuilder;
 use Symplify\PHPStanRules\Enum\RuleIdentifier\PHPUnitRuleIdentifier;
 use Symplify\PHPStanRules\Helper\NamingHelper;
 
 /**
  * @implements Rule<MethodCall>
+ * @see \Symplify\PHPStanRules\Tests\Rules\Doctrine\NoDocumentMockingRule\NoDocumentMockingRuleTest
  */
-final class NoDocumentMockingRule implements Rule
+final readonly class NoDocumentMockingRule implements Rule
 {
     public const string ERROR_MESSAGE = 'Instead of document mocking, create object directly to get better type support';
 
+    public function __construct(
+        private ReflectionProvider $reflectionProvider
+    ) {
+    }
+
     public function getNodeType(): string
     {
         return MethodCall::class;
@@ -40,10 +47,20 @@ public function processNode(Node $node, Scope $scope): array
         $firstArg = $node->getArgs()[0];
         $mockedClassType = $scope->getType($firstArg->value);
         foreach ($mockedClassType->getConstantStrings() as $constantStringType) {
-            if (! str_contains($constantStringType->getValue(), '\\Document\\')) {
+            if (! str_contains($constantStringType->getValue(), '\\Document\\') && ! str_contains($constantStringType->getValue(), '\\Entity\\')) {
                 continue;
             }
 
+            if ($this->reflectionProvider->hasClass($constantStringType->getValue())) {
+                $classReflection = $this->reflectionProvider->getClass($constantStringType->getValue());
+                if ($classReflection->isAbstract()) {
+                    continue;
+                }
+                if ($classReflection->isInterface()) {
+                    continue;
+                }
+            }
+
             $ruleError = RuleErrorBuilder::message(self::ERROR_MESSAGE)
                 ->identifier(PHPUnitRuleIdentifier::NO_DOCUMENT_MOCKING)
                 ->build();

tests/Rules/Doctrine/NoDocumentMockingRule/NoDocumentMockingRuleTest.php

@@ -4,14 +4,12 @@
 
 namespace Symplify\PHPStanRules\Tests\Rules\Doctrine\NoDocumentMockingRule;
 
+use Override;
 use Iterator;
-use PHPStan\Reflection\ReflectionProvider;
 use PHPStan\Rules\Rule;
 use PHPStan\Testing\RuleTestCase;
 use PHPUnit\Framework\Attributes\DataProvider;
 use Symplify\PHPStanRules\Rules\Doctrine\NoDocumentMockingRule;
-use Symplify\PHPStanRules\Rules\Doctrine\NoGetRepositoryOnServiceRepositoryEntityRule;
-use Symplify\PHPStanRules\Tests\Rules\Doctrine\NoGetRepositoryOnServiceRepositoryEntityRule\Source\Repository\SomeServiceRepository;
 
 final class NoDocumentMockingRuleTest extends RuleTestCase
 {
@@ -29,17 +27,25 @@ public function testRule(string $filePath, array $expectedErrorMessagesWithLines
      */
     public static function provideData(): Iterator
     {
-        $errorMessage = sprintf(NoGetRepositoryOnServiceRepositoryEntityRule::ERROR_MESSAGE, 'SomeEntity', SomeServiceRepository::class);
         yield [__DIR__ . '/Fixture/SomeEntityMocking.php', [[
-            $errorMessage,
+            NoDocumentMockingRule::ERROR_MESSAGE,
             14,
         ]]];
 
         yield [__DIR__ . '/Fixture/SomeAbstractEntityMocking.php', []];
     }
 
+    /**
+     * @return string[]
+     */
+    #[Override]
+    public static function getAdditionalConfigFiles(): array
+    {
+        return [__DIR__ . '/config/configured_rule.neon'];
+    }
+
     protected function getRule(): Rule
     {
-        return new NoDocumentMockingRule();
+        return self::getContainer()->getByType(NoDocumentMockingRule::class);
     }
 }

tests/Rules/Doctrine/NoDocumentMockingRule/config/configured_rule.neon

@@ -0,0 +1,5 @@
+includes:
+    - ../../../../config/included_services.neon
+
+rules:
+    - Symplify\PHPStanRules\Rules\Doctrine\NoDocumentMockingRule