Merge pull request #544 from biinari/chore/143-Squiz_ForLoopDeclarati…

…on_separate_parse_error Squiz/ForLoopDeclaration: move js parse error to its own file
PHPCSStandards · Jul 14, 2024 · fd065a7 · fd065a7
2 parents ac4987e + 53a5f79
commit fd065a7
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 9 deletions.
diff --git a/...lStructures/ForLoopDeclarationUnitTest.js → ...tructures/ForLoopDeclarationUnitTest.1.js b/...lStructures/ForLoopDeclarationUnitTest.js → ...tructures/ForLoopDeclarationUnitTest.1.js
@@ -120,6 +120,3 @@ for (
 // Test with semicolon not belonging to for.
 for (i = function() {self.widgetLoaded(widget.id)  ;  }; i < function() {self.widgetLoaded(widget.id);}; i++) {}
 for (i = function() {self.widgetLoaded(widget.id);}; i < function() {self.widgetLoaded(widget.id);}  ;   i++) {}
-
-// This test has to be the last one in the file! Intentional parse error check.
-for
diff --git a/...tures/ForLoopDeclarationUnitTest.js.fixed → ...res/ForLoopDeclarationUnitTest.1.js.fixed b/...tures/ForLoopDeclarationUnitTest.js.fixed → ...res/ForLoopDeclarationUnitTest.1.js.fixed
@@ -86,6 +86,3 @@ for ( i = 0; i < 10; i++ ) {}
 // Test with semicolon not belonging to for.
 for (i = function() {self.widgetLoaded(widget.id)  ;  }; i < function() {self.widgetLoaded(widget.id);}; i++) {}
 for (i = function() {self.widgetLoaded(widget.id);}; i < function() {self.widgetLoaded(widget.id);}; i++) {}
-
-// This test has to be the last one in the file! Intentional parse error check.
-for
diff --git a/src/Standards/Squiz/Tests/ControlStructures/ForLoopDeclarationUnitTest.2.js b/src/Standards/Squiz/Tests/ControlStructures/ForLoopDeclarationUnitTest.2.js
@@ -0,0 +1,2 @@
+// Intentional parse error check. Live coding resilience.
+for
diff --git a/src/Standards/Squiz/Tests/ControlStructures/ForLoopDeclarationUnitTest.php b/src/Standards/Squiz/Tests/ControlStructures/ForLoopDeclarationUnitTest.php
@@ -67,7 +67,7 @@ public function getErrorList($testFile='')
                 116 => 2,
             ];
 
-        case 'ForLoopDeclarationUnitTest.js':
+        case 'ForLoopDeclarationUnitTest.1.js':
             return [
                 6   => 2,
                 9   => 2,
@@ -125,8 +125,8 @@ public function getWarningList($testFile='')
         case 'ForLoopDeclarationUnitTest.3.inc':
             return [6 => 1];
 
-        case 'ForLoopDeclarationUnitTest.js':
-            return [125 => 1];
+        case 'ForLoopDeclarationUnitTest.2.js':
+            return [2 => 1];
 
         default:
             return [];