Make it easier to detect we are running tests under a PEAR install so the executable file check can be ignored

Author: gsherwood

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.
      *

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