Merge pull request #335 from fredden/parse-error-detection/PSR12.Files.DeclareStatement

`PSR12.Files.DeclareStatement` - bow out when parse error detected

Author: jrfnl

src/Standards/PSR12/Sniffs/Files/DeclareStatementSniff.php

@@ -110,6 +110,12 @@ public function process(File $phpcsFile, $stackPtr)
 
         // There should be no space between equal sign and directive value.
         $value = $phpcsFile->findNext(T_WHITESPACE, ($equals + 1), null, true);
+
+        if ($value === false) {
+            // Live coding / parse error.
+            return;
+        }
+
         if ($equals !== false) {
             if ($tokens[($equals + 1)]['type'] !== 'T_LNUMBER') {
                 $error = 'Expected no space between equal sign and the directive value in a declare statement';

src/Standards/PSR12/Tests/Files/DeclareStatementUnitTest.inc renamed to src/Standards/PSR12/Tests/Files/DeclareStatementUnitTest.1.inc

@@ -0,0 +1,3 @@
+<?php
+// Safeguard sniff handles parse error/live coding correctly.
+declare(strict_types=

src/Standards/PSR12/Tests/Files/DeclareStatementUnitTest.inc.fixed renamed to src/Standards/PSR12/Tests/Files/DeclareStatementUnitTest.1.inc.fixed

@@ -26,35 +26,42 @@ final class DeclareStatementUnitTest 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 [
-            2  => 1,
-            3  => 1,
-            4  => 1,
-            5  => 2,
-            6  => 1,
-            7  => 1,
-            9  => 2,
-            10 => 1,
-            11 => 3,
-            12 => 2,
-            13 => 1,
-            14 => 2,
-            16 => 3,
-            19 => 3,
-            22 => 1,
-            24 => 1,
-            26 => 3,
-            28 => 3,
-            34 => 2,
-            43 => 1,
-            46 => 1,
-            47 => 1,
-            49 => 1,
-        ];
+        switch ($testFile) {
+        case 'DeclareStatementUnitTest.1.inc':
+            return [
+                2  => 1,
+                3  => 1,
+                4  => 1,
+                5  => 2,
+                6  => 1,
+                7  => 1,
+                9  => 2,
+                10 => 1,
+                11 => 3,
+                12 => 2,
+                13 => 1,
+                14 => 2,
+                16 => 3,
+                19 => 3,
+                22 => 1,
+                24 => 1,
+                26 => 3,
+                28 => 3,
+                34 => 2,
+                43 => 1,
+                46 => 1,
+                47 => 1,
+                49 => 1,
+            ];
+        default:
+            return [];
+        }//end switch
 
     }//end getErrorList()