Squiz/EmbeddedPhp: prevent tests being skipped

The sniff contains a condition which prevents checks from being run on the last tag set when it is a non-single line tag set.

Adding a final open tag will prevent the _last_ test in the file accidentally being skipped, independently of what type of tag set this is, making the test case file more stable.

Author: jrfnl

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

@@ -119,3 +119,8 @@ function foo()
 <?php echo 'okay'; // phpcs:ignore Standard.Category.Sniff -- reason. ?>
 <?php echo 'too much space before close'; // phpcs:ignore Standard.Category.Sniff -- reason.    ?>
 <?php echo 'no space before close'; // phpcs:ignore Standard.Category.Sniff -- reason.?>
+
+<?php
+// This test case file MUST always end with an unclosed long open PHP tag (with this comment) to prevent
+// the tests running into the "last PHP closing tag excepted" condition breaking tests.
+// Tests related to that "last PHP closing tag excepted" condition should go in separate files.

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

@@ -119,3 +119,8 @@ function foo()
 <?php echo 'okay'; // phpcs:ignore Standard.Category.Sniff -- reason. ?>
 <?php echo 'too much space before close'; // phpcs:ignore Standard.Category.Sniff -- reason. ?>
 <?php echo 'no space before close'; // phpcs:ignore Standard.Category.Sniff -- reason. ?>
+
+<?php
+// This test case file MUST always end with an unclosed long open PHP tag (with this comment) to prevent
+// the tests running into the "last PHP closing tag excepted" condition breaking tests.
+// Tests related to that "last PHP closing tag excepted" condition should go in separate files.