revert to no acronym check

Author: klausi

coder_sniffer/Drupal/Sniffs/NamingConventions/ValidClassNameSniff.php

@@ -72,8 +72,11 @@ public function process(File $phpcsFile, $stackPtr)
             $phpcsFile->addError($error, $stackPtr, 'NoUnderscores', $errorData);
         }
 
-        // Ensure the name does not contain acronyms.
-        if (preg_match('|[A-Z]{3}|', $name) === 1) {
+        // Ensure the name is not all uppercase.
+        // @todo We could make this more strict to check if there are more than
+        // 2 upper case characters in a row, but not decided yet.
+        // See https://www.drupal.org/project/coder/issues/3497433
+        if (strtoupper($name) === $name) {
             $error = '%s name must use UpperCamel naming and not contain multiple upper case letters in a row';
             $phpcsFile->addError($error, $stackPtr, 'NoUpperAcronyms', $errorData);
         }

tests/Drupal/NamingConventions/ValidClassNameUnitTest.inc

@@ -2,28 +2,24 @@
 
 class CorrectClassName {}
 class CorrectClassWithAReallyLongName {}
-class IncorrectJSONClassName {}
 class INCORRECT_CLASS_NAME {}
 class INCORRECTCLASSNAME {}
 class incorrectLowercaseClassName {}
 
 interface CorrectInterfaceName {}
 interface CorrectInterfaceWithAReallyLongName {}
-interface IncorrectJSONInterfaceName {}
 interface INCORRECT_INTERFACE_NAME {}
 interface INCORRECTINTERFACENAME {}
 interface incorrectLowercaseInterfaceName {}
 
 trait CorrectTraitName {}
 trait CorrectTraitWithAReallyLongName {}
-trait IncorrectJSONTraitName {}
 trait INCORRECT_TRAIT_NAME {}
 trait INCORRECTTRAITNAME {}
 trait incorrectLowercaseTraitName {}
 
 enum CorrectEnumName {}
 enum CorrectEnumWithAReallyLongName {}
-enum IncorrectJSONEnumName {}
 enum INCORRECT_ENUM_NAME {}
 enum INCORRECTENUMNAME {}
 enum incorrectLowercaseEnumName {}

tests/Drupal/NamingConventions/ValidClassNameUnitTest.php

@@ -21,22 +21,18 @@ class ValidClassNameUnitTest extends CoderSniffUnitTest
     protected function getErrorList(string $testFile): array
     {
         return [
-            5  => 1,
-            6  => 2,
+            5  => 2,
+            6  => 1,
             7  => 1,
-            8  => 1,
+            11 => 2,
             12 => 1,
-            13 => 2,
-            14 => 1,
-            15 => 1,
+            13 => 1,
+            17 => 2,
+            18 => 1,
             19 => 1,
-            20 => 2,
-            21 => 1,
-            22 => 1,
-            26 => 1,
-            27 => 2,
-            28 => 1,
-            29 => 1,
+            23 => 2,
+            24 => 1,
+            25 => 1,
         ];
 
     }//end getErrorList()

tests/Drupal/NamingConventions/ValidEnumCaseUnitTest.inc

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

tests/Drupal/NamingConventions/ValidEnumCaseUnitTest.php

@@ -21,10 +21,9 @@ class ValidEnumCaseUnitTest extends CoderSniffUnitTest
     protected function getErrorList(string $testFile): array
     {
         return [
-            5  => 1,
-            7  => 2,
-            9  => 1,
-            11 => 1,
+            5 => 1,
+            7 => 2,
+            9 => 1,
         ];
 
     }//end getErrorList()