Merge pull request #579 from PHPCSStandards/feature/abstractpatternsniff-fix-bug

AbstractPatternSniff: prevent PHP notice

Author: jrfnl

src/Sniffs/AbstractPatternSniff.php

@@ -416,6 +416,11 @@ protected function processPattern($patternInfo, File $phpcsFile, $stackPtr)
         $lastAddedStackPtr = null;
         $patternLen        = count($pattern);
 
+        if (($stackPtr + $patternLen - $patternInfo['listen_pos']) > $phpcsFile->numTokens) {
+            // Pattern can never match as there are not enough tokens left in the file.
+            return false;
+        }
+
         for ($i = $patternInfo['listen_pos']; $i < $patternLen; $i++) {
             if (isset($tokens[$stackPtr]) === false) {
                 break;

src/Standards/Squiz/Tests/Functions/FunctionDeclarationUnitTest.inc renamed to src/Standards/Squiz/Tests/Functions/FunctionDeclarationUnitTest.1.inc

@@ -0,0 +1,5 @@
+<?php
+
+// Intentional parse error. This has to be the last (and only) test in the file.
+// Live coding test. The sniff should stay silent.
+function

src/Standards/Squiz/Tests/Functions/FunctionDeclarationUnitTest.2.inc

@@ -0,0 +1,5 @@
+<?php
+
+// Intentional parse error. This has to be the last (and only) test in the file.
+// Live coding test. The sniff should stay silent.
+function // Comment.

src/Standards/Squiz/Tests/Functions/FunctionDeclarationUnitTest.3.inc

@@ -26,14 +26,22 @@ final class FunctionDeclarationUnitTest extends AbstractSniffUnitTest
      * 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>
      */
-    public function getErrorList()
+    public function getErrorList($testFile='')
     {
-        return [
-            55 => 1,
-            68 => 1,
-        ];
+        switch ($testFile) {
+        case 'FunctionDeclarationUnitTest.1.inc':
+            return [
+                55 => 1,
+                68 => 1,
+            ];
+
+        default:
+            return [];
+        }//end switch
 
     }//end getErrorList()