Squiz/EmbeddedPhp: document the behaviour for closed tag block at end of file [2]

The last tag block in a file will **not** be ignored if it is followed by non-empty inline HTML.

These tests document this behaviour.

Author: jrfnl

src/Standards/Squiz/Tests/PHP/EmbeddedPhpUnitTest.18.inc

@@ -0,0 +1,15 @@
+<?php
+// This test case file MUST always start with a long open PHP tag set (with this comment) to prevent
+// the tests running into the "first PHP open tag excepted" condition breaking the tests.
+// Tests related to that "first PHP open tag excepted" condition should go in separate files.
+?>
+<!--
+The complete tag block will **NOT** be ignored when it is the last (closed) tag block in a file
+as long as it is followed by non-empty inline HTML.
+-->
+<div>
+<?php
+    ?>
+
+</div>
+<p>Some more content after the last PHP tag block.</p>

src/Standards/Squiz/Tests/PHP/EmbeddedPhpUnitTest.18.inc.fixed

@@ -0,0 +1,13 @@
+<?php
+// This test case file MUST always start with a long open PHP tag set (with this comment) to prevent
+// the tests running into the "first PHP open tag excepted" condition breaking the tests.
+// Tests related to that "first PHP open tag excepted" condition should go in separate files.
+?>
+<!--
+The complete tag block will **NOT** be ignored when it is the last (closed) tag block in a file
+as long as it is followed by non-empty inline HTML.
+-->
+<div>
+
+</div>
+<p>Some more content after the last PHP tag block.</p>

src/Standards/Squiz/Tests/PHP/EmbeddedPhpUnitTest.19.inc

@@ -0,0 +1,17 @@
+<?php
+// This test case file MUST always start with a long open PHP tag set (with this comment) to prevent
+// the tests running into the "first PHP open tag excepted" condition breaking the tests.
+// Tests related to that "first PHP open tag excepted" condition should go in separate files.
+?>
+<!--
+The complete tag block will **NOT** be ignored when it is the last (closed) tag block in a file
+as long as it is followed by non-empty inline HTML.
+
+Note: with short open tags, the empty PHP tag set is a parse error.
+-->
+<div>
+<?=
+    ?>
+
+</div>
+<p>Some more content after the last PHP tag block.</p>

src/Standards/Squiz/Tests/PHP/EmbeddedPhpUnitTest.19.inc.fixed

@@ -0,0 +1,15 @@
+<?php
+// This test case file MUST always start with a long open PHP tag set (with this comment) to prevent
+// the tests running into the "first PHP open tag excepted" condition breaking the tests.
+// Tests related to that "first PHP open tag excepted" condition should go in separate files.
+?>
+<!--
+The complete tag block will **NOT** be ignored when it is the last (closed) tag block in a file
+as long as it is followed by non-empty inline HTML.
+
+Note: with short open tags, the empty PHP tag set is a parse error.
+-->
+<div>
+
+</div>
+<p>Some more content after the last PHP tag block.</p>

src/Standards/Squiz/Tests/PHP/EmbeddedPhpUnitTest.20.inc

@@ -0,0 +1,15 @@
+<?php
+// This test case file MUST always start with a long open PHP tag set (with this comment) to prevent
+// the tests running into the "first PHP open tag excepted" condition breaking the tests.
+// Tests related to that "first PHP open tag excepted" condition should go in separate files.
+?>
+<!--
+The complete tag block will **NOT** be ignored when it is the last (closed) tag block in a file
+as long as it is followed by non-empty inline HTML.
+-->
+<div>
+<?php
+    echo 'too much indent and close tag not on own line'; ?>
+
+</div>
+<p>Some more content after the last PHP tag block.</p>

src/Standards/Squiz/Tests/PHP/EmbeddedPhpUnitTest.20.inc.fixed

@@ -0,0 +1,16 @@
+<?php
+// This test case file MUST always start with a long open PHP tag set (with this comment) to prevent
+// the tests running into the "first PHP open tag excepted" condition breaking the tests.
+// Tests related to that "first PHP open tag excepted" condition should go in separate files.
+?>
+<!--
+The complete tag block will **NOT** be ignored when it is the last (closed) tag block in a file
+as long as it is followed by non-empty inline HTML.
+-->
+<div>
+<?php
+echo 'too much indent and close tag not on own line';
+    ?>
+
+</div>
+<p>Some more content after the last PHP tag block.</p>

src/Standards/Squiz/Tests/PHP/EmbeddedPhpUnitTest.21.inc

@@ -0,0 +1,15 @@
+<?php
+// This test case file MUST always start with a long open PHP tag set (with this comment) to prevent
+// the tests running into the "first PHP open tag excepted" condition breaking the tests.
+// Tests related to that "first PHP open tag excepted" condition should go in separate files.
+?>
+<!--
+The complete tag block will **NOT** be ignored when it is the last (closed) tag block in a file
+as long as it is followed by non-empty inline HTML.
+-->
+<div>
+<?=
+    'too much indent and close tag not on own line'; ?>
+
+</div>
+<p>Some more content after the last PHP tag block.</p>

src/Standards/Squiz/Tests/PHP/EmbeddedPhpUnitTest.21.inc.fixed

@@ -0,0 +1,16 @@
+<?php
+// This test case file MUST always start with a long open PHP tag set (with this comment) to prevent
+// the tests running into the "first PHP open tag excepted" condition breaking the tests.
+// Tests related to that "first PHP open tag excepted" condition should go in separate files.
+?>
+<!--
+The complete tag block will **NOT** be ignored when it is the last (closed) tag block in a file
+as long as it is followed by non-empty inline HTML.
+-->
+<div>
+<?=
+'too much indent and close tag not on own line';
+    ?>
+
+</div>
+<p>Some more content after the last PHP tag block.</p>

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

@@ -158,6 +158,16 @@ public function getErrorList($testFile='')
                 12 => 1,
             ];
 
+        case 'EmbeddedPhpUnitTest.18.inc':
+            return [11 => 1];
+
+        case 'EmbeddedPhpUnitTest.19.inc':
+            return [13 => 1];
+
+        case 'EmbeddedPhpUnitTest.20.inc':
+        case 'EmbeddedPhpUnitTest.21.inc':
+            return [12 => 2];
+
         default:
             return [];
         }//end switch