Added rule to use proper suffix/prefix

Author: ViniTou

extension.neon

@@ -7,3 +7,5 @@ parameters:
 		- stubs/Money/MoneyParser.stub
 rules:
     - Ibexa\PHPStan\Rules\NoConfigResolverParametersInConstructorRule
+    - Ibexa\PHPStan\Rules\RequireInterfaceInDependenciesRule
+    - Ibexa\PHPStan\Rules\NamingConventionRule

phpstan-baseline.neon

@@ -0,0 +1,16 @@
+parameters:
+	ignoreErrors:
+		-
+			message: "#^Property Ibexa\\\\Tests\\\\PHPStan\\\\Rules\\\\Fixtures\\\\RequireInterfaceInDependenciesFixture\\:\\:\\$classWithoutInterface is never read, only written\\.$#"
+			count: 1
+			path: tests/rules/Fixtures/RequireInterfaceInDependenciesFixture.php
+
+		-
+			message: "#^Property Ibexa\\\\Tests\\\\PHPStan\\\\Rules\\\\Fixtures\\\\RequireInterfaceInDependenciesFixture\\:\\:\\$concreteClass is never read, only written\\.$#"
+			count: 1
+			path: tests/rules/Fixtures/RequireInterfaceInDependenciesFixture.php
+
+		-
+			message: "#^Property Ibexa\\\\Tests\\\\PHPStan\\\\Rules\\\\Fixtures\\\\RequireInterfaceInDependenciesFixture\\:\\:\\$testInterface is never read, only written\\.$#"
+			count: 1
+			path: tests/rules/Fixtures/RequireInterfaceInDependenciesFixture.php

phpstan.neon

@@ -1,3 +1,6 @@
+includes:
+    - phpstan-baseline.neon
+
 parameters:
     level: 8
     paths:

rules/NamingConventionRule.php

@@ -0,0 +1,75 @@
+<?php
+
+/**
+ * @copyright Copyright (C) Ibexa AS. All rights reserved.
+ * @license For full copyright and license information view LICENSE file distributed with this source code.
+ */
+declare(strict_types=1);
+
+namespace Ibexa\PHPStan\Rules;
+
+use PhpParser\Node;
+use PHPStan\Analyser\Scope;
+use PHPStan\Rules\Rule;
+use PHPStan\Rules\RuleErrorBuilder;
+
+/**
+ * @implements \PHPStan\Rules\Rule<\PhpParser\Node\Stmt\ClassLike>
+ */
+final class NamingConventionRule implements Rule
+{
+    public function getNodeType(): string
+    {
+        return Node\Stmt\ClassLike::class;
+    }
+
+    public function processNode(Node $node, Scope $scope): array
+    {
+        if (!$this->isRelevantNode($node)) {
+            return [];
+        }
+
+        if ($node->name === null) {
+            return [];
+        }
+
+        $className = $node->name->toString();
+        $errors = [];
+
+        if ($node instanceof Node\Stmt\Interface_ && substr($className, -9) !== 'Interface') {
+            $errors[] = RuleErrorBuilder::message(
+                sprintf(
+                    'Interface "%s" should have "Interface" suffix',
+                    $className
+                )
+            )->build();
+        }
+
+        if ($node instanceof Node\Stmt\Trait_ && substr($className, -5) !== 'Trait') {
+            $errors[] = RuleErrorBuilder::message(
+                sprintf(
+                    'Trait "%s" should have "Trait" suffix',
+                    $className
+                )
+            )->build();
+        }
+
+        if ($node instanceof Node\Stmt\Class_ && $node->isAbstract() && strpos($className, 'Abstract') !== 0) {
+            $errors[] = RuleErrorBuilder::message(
+                sprintf(
+                    'Abstract class "%s" should have "Abstract" prefix',
+                    $className
+                )
+            )->build();
+        }
+
+        return $errors;
+    }
+
+    private function isRelevantNode(Node $node): bool
+    {
+        return $node instanceof Node\Stmt\Interface_
+            || $node instanceof Node\Stmt\Trait_
+            || ($node instanceof Node\Stmt\Class_ && $node->isAbstract());
+    }
+}

tests/rules/Fixtures/NamingConventionFixture.php

@@ -0,0 +1,33 @@
+<?php
+
+/**
+ * @copyright Copyright (C) Ibexa AS. All rights reserved.
+ * @license For full copyright and license information view LICENSE file distributed with this source code.
+ */
+declare(strict_types=1);
+
+namespace Ibexa\Tests\PHPStan\Rules\Fixtures;
+
+interface WrongName
+{
+}
+
+interface CorrectNameInterface
+{
+}
+
+trait SimpleThing
+{
+}
+
+trait CorrectNameTrait
+{
+}
+
+abstract class SimpleClass
+{
+}
+
+abstract class AbstractCorrectClass
+{
+}

tests/rules/NamingConventionRuleTest.php

@@ -0,0 +1,47 @@
+<?php
+
+/**
+ * @copyright Copyright (C) Ibexa AS. All rights reserved.
+ * @license For full copyright and license information view LICENSE file distributed with this source code.
+ */
+declare(strict_types=1);
+
+namespace Ibexa\Tests\PHPStan\Rules;
+
+use Ibexa\PHPStan\Rules\NamingConventionRule;
+use PHPStan\Rules\Rule;
+use PHPStan\Testing\RuleTestCase;
+
+/**
+ * @extends \PHPStan\Testing\RuleTestCase<\Ibexa\PHPStan\Rules\NamingConventionRule>
+ */
+final class NamingConventionRuleTest extends RuleTestCase
+{
+    protected function getRule(): Rule
+    {
+       return new NamingConventionRule();
+    }
+
+    public function testRule(): void
+    {
+        $this->analyse(
+            [
+                __DIR__ . '/Fixtures/NamingConventionFixture.php',
+            ],
+            [
+                [
+                    'Interface "WrongName" should have "Interface" suffix',
+                    11,
+                ],
+                [
+                    'Trait "SimpleThing" should have "Trait" suffix',
+                    19,
+                ],
+                [
+                    'Abstract class "SimpleClass" should have "Abstract" prefix',
+                    27,
+                ],
+            ]
+        );
+    }
+}