feat(ValidClassName): Check traits and enums for valid upperCamel names

Author: klausi

coder_sniffer/Drupal/Sniffs/NamingConventions/ValidClassNameSniff.php

@@ -15,7 +15,7 @@
 /**
  * \Drupal\Sniffs\NamingConventions\ValidClassNameSniff.
  *
- * Ensures class and interface names start with a capital letter
+ * Ensures class, enum, interface and trait names start with a capital letter
  * and do not use _ separators.
  *
  * @category PHP
@@ -35,7 +35,9 @@ public function register()
     {
         return [
             T_CLASS,
+            T_ENUM,
             T_INTERFACE,
+            T_TRAIT,
         ];
 
     }//end register()
@@ -60,7 +62,7 @@ public function process(File $phpcsFile, $stackPtr)
 
         // Make sure the first letter is a capital.
         if (preg_match('|^[A-Z]|', $name) === 0) {
-            $error = '%s name must begin with a capital letter';
+            $error = '%s name must use UpperCamel naming and begin with a capital letter';
             $phpcsFile->addError($error, $stackPtr, 'StartWithCapital', $errorData);
         }
 
@@ -70,6 +72,12 @@ public function process(File $phpcsFile, $stackPtr)
             $phpcsFile->addError($error, $stackPtr, 'NoUnderscores', $errorData);
         }
 
+        // Ensure the name is not all uppercase.
+        if (strtoupper($name) === $name) {
+            $error = '%s name must use UpperCamel naming and not be all uppercase';
+            $phpcsFile->addError($error, $stackPtr, 'NotAllUppercase', $errorData);
+        }
+
     }//end process()
 
 

tests/Drupal/NamingConventions/ValidClassNameUnitTest.inc

@@ -0,0 +1,25 @@
+<?php
+
+class CorrectClassName {}
+class CorrectJSONClassName {}
+class INCORRECT_CLASS_NAME {}
+class INCORRECTCLASSNAME {}
+class incorrectLowercaseClassName {}
+
+interface CorrectInterfaceName {}
+interface CorrectJSONInterfaceName {}
+interface INCORRECT_INTERFACE_NAME {}
+interface INCORRECTINTERFACENAME {}
+interface incorrectLowercaseInterfaceName {}
+
+trait CorrectTraitName {}
+trait CorrectJSONTraitName {}
+trait INCORRECT_TRAIT_NAME {}
+trait INCORRECTTRAITNAME {}
+trait incorrectLowercaseTraitName {}
+
+enum CorrectEnumName {}
+enum CorrectJSONEnumName {}
+enum INCORRECT_ENUM_NAME {}
+enum INCORRECTENUMNAME {}
+enum incorrectLowercaseEnumName {}

tests/Drupal/NamingConventions/ValidClassNameUnitTest.php

@@ -0,0 +1,58 @@
+<?php
+
+namespace Drupal\Test\NamingConventions;
+
+use Drupal\Test\CoderSniffUnitTest;
+
+class ValidClassNameUnitTest extends CoderSniffUnitTest
+{
+
+
+    /**
+     * Returns the lines where errors should occur.
+     *
+     * The key of the array should represent the line number and the value
+     * should represent the number of errors that should occur on that line.
+     *
+     * @param string $testFile The name of the file being tested.
+     *
+     * @return array<int, int>
+     */
+    protected function getErrorList(string $testFile): array
+    {
+        return [
+            5 => 2,
+            6 => 1,
+            7 => 1,
+            11 => 2,
+            12 => 1,
+            13 => 1,
+            17 => 2,
+            18 => 1,
+            19 => 1,
+            23 => 2,
+            24 => 1,
+            25 => 1,
+        ];
+
+    }//end getErrorList()
+
+
+    /**
+     * Returns the lines where warnings should occur.
+     *
+     * The key of the array should represent the line number and the value
+     * should represent the number of warnings that should occur on that line.
+     *
+     * @param string $testFile The name of the file being tested.
+     *
+     * @return array<int, int>
+     */
+    protected function getWarningList(string $testFile): array
+    {
+        return [];
+
+    }//end getWarningList()
+
+
+}//end class

tests/Drupal/NamingConventions/ValidEnumCaseUnitTest.inc

@@ -5,4 +5,8 @@ enum Test: int {
   case one = 1;
   // Must not contain underscores.
   case TWO_TEST = 2;
+  // Must not contain only upper case.
+  case THREE = 3;
+  // Some upper case parts are allowed for now.
+  case FourJSONCase = 4;
 }

tests/Drupal/NamingConventions/ValidEnumCaseUnitTest.php

@@ -22,7 +22,8 @@ protected function getErrorList(string $testFile): array
     {
         return [
             5 => 1,
-            7 => 1,
+            7 => 2,
+            9 => 1,
         ];
 
     }//end getErrorList()

tests/Drupal/bad/BadUnitTest.php

@@ -380,7 +380,7 @@ protected function getErrorList(string $testFile): array
                 827 => 1,
                 829 => 1,
                 836 => 1,
-                838 => 1,
+                838 => 3,
                 849 => 2,
                 860 => 2,
                 867 => 1,