Generic/InlineControlStructure: improve code coverage

Author: rodrigoprimo

src/Standards/Generic/Tests/ControlStructures/InlineControlStructureUnitTest.1.inc

@@ -297,3 +297,22 @@ foreach ($array as $sniffShouldBailEarly)
 	/* some comment */;
 
 do ; while ($sniffShouldBailEarly > 5);
+
+do echo $i++; while ($i < 5);
+
+// phpcs:set Generic.ControlStructures.InlineControlStructure error false
+if ($something) echo 'hello';
+// phpcs:set Generic.ControlStructures.InlineControlStructure error true
+
+for ($i = 0; $i < 5; $i++)
+    if ($i === 3) {
+        echo 'three';
+    } elseif ($i === 4) {
+        echo 'four';
+    } elseif ($i === 5) {
+        echo 'five';
+    }
+
+if ($noSpaceAfterClosingParenthesis)echo 'hello'; ?>
+
+<?php if ( $model->scenario == 'simple' ) $widget->renderPager() ?>

src/Standards/Generic/Tests/ControlStructures/InlineControlStructureUnitTest.1.inc.fixed

@@ -336,3 +336,26 @@ foreach ($array as $sniffShouldBailEarly)
 	/* some comment */;
 
 do ; while ($sniffShouldBailEarly > 5);
+
+do { echo $i++;
+} while ($i < 5);
+
+// phpcs:set Generic.ControlStructures.InlineControlStructure error false
+if ($something) { echo 'hello';
+}
+// phpcs:set Generic.ControlStructures.InlineControlStructure error true
+
+for ($i = 0; $i < 5; $i++) {
+    if ($i === 3) {
+        echo 'three';
+    } elseif ($i === 4) {
+        echo 'four';
+    } elseif ($i === 5) {
+        echo 'five';
+    }
+}
+
+if ($noSpaceAfterClosingParenthesis) { echo 'hello';
+} ?>
+
+<?php if ( $model->scenario == 'simple' ) { $widget->renderPager(); } ?>

src/Standards/Generic/Tests/ControlStructures/InlineControlStructureUnitTest.10.inc

@@ -0,0 +1,9 @@
+<?php
+
+// This should be the only test in this file.
+// There should be only empty tokens after the scope closer of the "if" token.
+
+for ($i = 0; $i < 5; $i++)
+    if ($i === 3) {
+        echo $i;
+    }

src/Standards/Generic/Tests/ControlStructures/InlineControlStructureUnitTest.10.inc.fixed

@@ -0,0 +1,10 @@
+<?php
+
+// This should be the only test in this file.
+// There should be only empty tokens after the scope closer of the "if" token.
+
+for ($i = 0; $i < 5; $i++) {
+    if ($i === 3) {
+        echo $i;
+    }
+}

src/Standards/Generic/Tests/ControlStructures/InlineControlStructureUnitTest.11.inc

@@ -0,0 +1,8 @@
+<?php
+
+// Intentional parse error (missing semicolon). Should still trigger the sniff error.
+// This should be the only test in this file.
+// This test should explicitly NOT contain a new line at the end of the file.
+
+if ($bar)
+    doSomething()

src/Standards/Generic/Tests/ControlStructures/InlineControlStructureUnitTest.11.inc.fixed

@@ -0,0 +1,9 @@
+<?php
+
+// Intentional parse error (missing semicolon). Should still trigger the sniff error.
+// This should be the only test in this file.
+// This test should explicitly NOT contain a new line at the end of the file.
+
+if ($bar) {
+    doSomething(); 
+}

src/Standards/Generic/Tests/ControlStructures/InlineControlStructureUnitTest.12.inc

@@ -0,0 +1,9 @@
+<?php
+
+// Intentional parse error (nothing after else).
+// This should be the only test in this file.
+// Testing that the sniff is *not* triggered.
+
+if ($bar) {
+    doSomething();
+} else

src/Standards/Generic/Tests/ControlStructures/InlineControlStructureUnitTest.13.inc

@@ -0,0 +1,13 @@
+<?php
+
+// Intentional parse error (missing closing brace for the first if).
+// This should be the only test in this file.
+// Testing that the sniff is *not* triggered.
+// Test for https://github.com/squizlabs/PHP_CodeSniffer/pull/2989.
+
+if ( true ) {
+
+// This second if is here to ensure that the sniff completely bows out of this file after
+// encountering the first one without a closing brace.
+if ( true )
+    echo 'foo';

src/Standards/Generic/Tests/ControlStructures/InlineControlStructureUnitTest.8.inc

@@ -0,0 +1,7 @@
+<?php
+
+// Intentional parse error (nothing after for closing parenthesis).
+// This should be the only test in this file.
+// Testing that the sniff is *not* triggered.
+
+for ($i = 0; $i < 5; $i++)

src/Standards/Generic/Tests/ControlStructures/InlineControlStructureUnitTest.9.inc

@@ -0,0 +1,7 @@
+<?php
+
+// Intentional parse error (nothing after while closing parenthesis).
+// This should be the only test in this file.
+// Testing that the sniff is *not* triggered.
+
+while ($i < 5)