Generic/ConstructorName: bug fix - allow for unconventional spacing and comments

When determining whether or not a "parent" constructor was being called using a call to a PHP4-style constructor, the sniff would only look at the _next_ token after a double colon, while the next token may be whitespace or a comment, which should be ignored.

Fixed now by making the sniff check for the _next non empty_ token instead.

Includes tests.

Author: jrfnl

src/Standards/Generic/Sniffs/NamingConventions/ConstructorNameSniff.php

@@ -15,6 +15,7 @@
 
 use PHP_CodeSniffer\Files\File;
 use PHP_CodeSniffer\Sniffs\AbstractScopeSniff;
+use PHP_CodeSniffer\Util\Tokens;
 
 class ConstructorNameSniff extends AbstractScopeSniff
 {
@@ -104,14 +105,15 @@ protected function processTokenWithinScope(File $phpcsFile, $stackPtr, $currScop
         $endFunctionIndex = $tokens[$stackPtr]['scope_closer'];
         $startIndex       = $stackPtr;
         while (($doubleColonIndex = $phpcsFile->findNext(T_DOUBLE_COLON, $startIndex, $endFunctionIndex)) !== false) {
-            if ($tokens[($doubleColonIndex + 1)]['code'] === T_STRING
-                && strtolower($tokens[($doubleColonIndex + 1)]['content']) === $parentClassNameLc
+            $nextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, ($doubleColonIndex + 1), null, true);
+            if ($tokens[$nextNonEmpty]['code'] === T_STRING
+                && strtolower($tokens[$nextNonEmpty]['content']) === $parentClassNameLc
             ) {
                 $error = 'PHP4 style calls to parent constructors are not allowed; use "parent::__construct()" instead';
-                $phpcsFile->addError($error, ($doubleColonIndex + 1), 'OldStyleCall');
+                $phpcsFile->addError($error, $nextNonEmpty, 'OldStyleCall');
             }
 
-            $startIndex = ($doubleColonIndex + 1);
+            $startIndex = $nextNonEmpty;
         }
 
     }//end processTokenWithinScope()

src/Standards/Generic/Tests/NamingConventions/ConstructorNameUnitTest.inc

@@ -95,3 +95,20 @@ interface CustomChildCreator
 {
     public function customChildCreator($elementName, Project $project);
 }
+
+// Bug: unconventional spacing and unexpected comments were not handled correctly.
+class UnconventionalSpacing extends MyParent
+{
+    public function UnconventionalSpacing() {
+        self   ::   MyParent();
+        static/*comment*/::/*comment*/MyParent();
+    }
+
+    public function __construct() {
+        parent
+        //comment
+        ::
+        // phpcs:ignore Stnd.Cat.SniffName -- for reasons.
+        MyParent();
+    }
+}

src/Standards/Generic/Tests/NamingConventions/ConstructorNameUnitTest.php

@@ -31,11 +31,14 @@ final class ConstructorNameUnitTest extends AbstractSniffUnitTest
     public function getErrorList()
     {
         return [
-            6  => 1,
-            11 => 1,
-            47 => 1,
-            62 => 1,
-            91 => 1,
+            6   => 1,
+            11  => 1,
+            47  => 1,
+            62  => 1,
+            91  => 1,
+            103 => 1,
+            104 => 1,
+            112 => 1,
         ];
 
     }//end getErrorList()