Merge pull request #9443 from weirdan/require-php-tag-in-tests

vimeo · Mar 3, 2023 · b1b2010 · b1b2010
2 parents 6975acf + 7b8a0a6
commit b1b2010
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/tests/Traits/InvalidCodeAnalysisTestTrait.php b/tests/Traits/InvalidCodeAnalysisTestTrait.php
@@ -51,6 +51,11 @@ public function testInvalidCode(
             $this->markTestSkipped('Skipped due to a bug.');
         }
 
+        // sanity check - do we have a PHP tag?
+        if (strpos($code, '<?php') === false) {
+            $this->fail('Test case must have a <?php tag');
+        }
+
         if (strtoupper(substr(PHP_OS, 0, 3)) === 'WIN') {
             $code = str_replace("\n", "\r\n", $code);
         }

diff --git a/tests/Traits/ValidCodeAnalysisTestTrait.php b/tests/Traits/ValidCodeAnalysisTestTrait.php
@@ -55,6 +55,11 @@ public function testValidCode(
             $this->markTestSkipped('Skipped due to a bug.');
         }
 
+        // sanity check - do we have a PHP tag?
+        if (strpos($code, '<?php') === false) {
+            $this->fail('Test case must have a <?php tag');
+        }
+
         foreach ($ignored_issues as $issue_name) {
             Config::getInstance()->setCustomErrorLevel($issue_name, Config::REPORT_SUPPRESS);
         }