Allow d in short names (#116)

Author: TomasVotruba

src/ObjectCalisthenics/Rules/NoShortNameRule.php

@@ -36,7 +36,7 @@ final class NoShortNameRule extends AbstractSymplifyRule implements Configurable
      */
     public function __construct(
         private readonly int $minNameLength,
-        private readonly array $allowedShortNames = ['i', 'j', 'y', 'z']
+        private readonly array $allowedShortNames = ['i', 'j', 'y', 'z', 'd', 'dd']
     ) {
     }
 
@@ -130,6 +130,9 @@ private function isNameValid(string $name): bool
             return true;
         }
 
+        dump($name);
+        dump(in_array($name, $this->allowedShortNames, true));
+
         return in_array($name, $this->allowedShortNames, true);
     }
 }

tests/ObjectCalisthenics/Rules/NoShortNameRule/Fixture/AllowD.php

@@ -0,0 +1,8 @@
+<?php
+
+declare(strict_types=1);
+
+function d()
+{
+    // handle dump
+}

tests/ObjectCalisthenics/Rules/NoShortNameRule/NoShortNameRuleTest.php

@@ -26,13 +26,15 @@ public static function provideData(): Iterator
         yield [__DIR__ . '/Fixture/SkipId.php', []];
 
         $errorMessage = sprintf(NoShortNameRule::ERROR_MESSAGE, 'em', 3);
-        $yeErrorMEssage = sprintf(NoShortNameRule::ERROR_MESSAGE, 'YE', 3);
-        yield [__DIR__ . '/Fixture/ShortNamingClass.php', [[$errorMessage, 9], [$yeErrorMEssage, 11]]];
+        $yeErrorMessage = sprintf(NoShortNameRule::ERROR_MESSAGE, 'YE', 3);
+        yield [__DIR__ . '/Fixture/ShortNamingClass.php', [[$errorMessage, 9], [$yeErrorMessage, 11]]];
 
         $errorMessage = sprintf(NoShortNameRule::ERROR_MESSAGE, 'n', 3);
         yield [__DIR__ . '/Fixture/ShortClosureParam.php', [[$errorMessage, 11]]];
         yield [__DIR__ . '/Fixture/ShortParam.php', [[$errorMessage, 9]]];
 
+        yield [__DIR__ . '/Fixture/AllowD.php', []];
+
         $errorMessage = sprintf(NoShortNameRule::ERROR_MESSAGE, 'n', 3);
         yield [
             __DIR__ . '/Fixture/ShortAssignParameter.php',

tests/ObjectCalisthenics/Rules/NoShortNameRule/config/configured_rule.neon

@@ -9,3 +9,4 @@ services:
             minNameLength: 3
             allowedShortNames:
                 - id
+                - d