Merge pull request PHPCSStandards#151 from PHPCSStandards/feature/squiz-heredoc-test-tweak

Squiz/Heredoc: tweak the tests

Author: jrfnl

src/Standards/Squiz/Tests/PHP/HeredocUnitTest.1.inc

@@ -0,0 +1,12 @@
+<?php
+$str = <<<EOD
+Example of string
+spanning multiple lines
+using heredoc syntax.
+EOD;
+
+echo <<<'EOT'
+My name is "$name". I am printing some $foo->foo.
+Now, I am printing some {$foo->bar[1]}.
+This should not print a capital 'A': \x41
+EOT;

src/Standards/Squiz/Tests/PHP/HeredocUnitTest.inc renamed to src/Standards/Squiz/Tests/PHP/HeredocUnitTest.2.inc

@@ -1,16 +1,6 @@
 <?php
-$str = <<<EOD
-Example of string
-spanning multiple lines
-using heredoc syntax.
-EOD;
-
-echo <<<'EOT'
-My name is "$name". I am printing some $foo->foo.
-Now, I am printing some {$foo->bar[1]}.
-This should not print a capital 'A': \x41
-EOT;
 
+// Intentional parse error.
 // The following function has a simulated git conflict for testing.
 // This is not a merge conflict - it is a valid test case.
 // Please do not remove.

src/Standards/Squiz/Tests/PHP/HeredocUnitTest.php

@@ -26,14 +26,22 @@ class HeredocUnitTest 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,
-            8 => 1,
-        ];
+        switch ($testFile) {
+        case 'HeredocUnitTest.1.inc':
+            return [
+                2 => 1,
+                8 => 1,
+            ];
+
+        default:
+            return [];
+        }//end switch
 
     }//end getErrorList()