Make it easier to detect we are running tests under a PEAR install so…

… the executable file check can be ignored
squizlabs · Dec 4, 2019 · 68048f3 · 68048f3
1 parent b3c7925
commit 68048f3
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 0 deletions.
diff --git a/src/Standards/Generic/Tests/Files/ExecutableFileUnitTest.php b/src/Standards/Generic/Tests/Files/ExecutableFileUnitTest.php
@@ -15,6 +15,20 @@ class ExecutableFileUnitTest extends AbstractSniffUnitTest
 {
 
 
+    /**
+     * Should this test be skipped for some reason.
+     *
+     * @return void
+     */
+    protected function shouldSkipTest()
+    {
+        // PEAR doesn't preserve the executable flag, so skip
+        // tests when running in a PEAR install.
+        return $GLOBALS['PHP_CODESNIFFER_PEAR'];
+
+    }//end shouldSkipTest()
+
+
     /**
      * Returns the lines where errors should occur.
      *

diff --git a/tests/AllTests.php b/tests/AllTests.php
@@ -9,13 +9,16 @@
 
 namespace PHP_CodeSniffer\Tests;
 
+$GLOBALS['PHP_CODESNIFFER_PEAR'] = false;
+
 if (is_file(__DIR__.'/../autoload.php') === true) {
     include_once 'Core/AllTests.php';
     include_once 'Standards/AllSniffs.php';
 } else {
     include_once 'CodeSniffer/Core/AllTests.php';
     include_once 'CodeSniffer/Standards/AllSniffs.php';
     include_once 'FileList.php';
+    $GLOBALS['PHP_CODESNIFFER_PEAR'] = true;
 }
 
 // PHPUnit 7 made the TestSuite run() method incompatible with