Merge pull request #550 from biinari/chore/143-PEAR_ValidDefaultValue_separate_parse_error

PEAR/ValidDefaultValue: move syntax error to its own file

Author: jrfnl

src/Standards/PEAR/Tests/Functions/ValidDefaultValueUnitTest.inc renamed to src/Standards/PEAR/Tests/Functions/ValidDefaultValueUnitTest.1.inc

@@ -114,6 +114,3 @@ class ConstructorPropertyPromotionMixedWithNormalParams {
         mixed $requiredParam,
     ) {}
 }
-
-// Intentional syntax error. Must be last thing in the file.
-function

src/Standards/PEAR/Tests/Functions/ValidDefaultValueUnitTest.2.inc

@@ -0,0 +1,7 @@
+<?php
+
+// Intentional syntax error.
+// This should be the only test in this file.
+// Testing that the sniff is *not* triggered.
+
+function

src/Standards/PEAR/Tests/Functions/ValidDefaultValueUnitTest.php

@@ -26,23 +26,31 @@ final class ValidDefaultValueUnitTest 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 [
-            29  => 1,
-            34  => 1,
-            39  => 1,
-            71  => 1,
-            76  => 1,
-            81  => 1,
-            91  => 1,
-            99  => 1,
-            101 => 1,
-            106 => 1,
-            114 => 1,
-        ];
+        switch ($testFile) {
+        case 'ValidDefaultValueUnitTest.1.inc':
+            return [
+                29  => 1,
+                34  => 1,
+                39  => 1,
+                71  => 1,
+                76  => 1,
+                81  => 1,
+                91  => 1,
+                99  => 1,
+                101 => 1,
+                106 => 1,
+                114 => 1,
+            ];
+
+        default:
+            return [];
+        }
 
     }//end getErrorList()