Fix current working directory issues

Author: moufmouf

src/Glob/GlobClassExplorer.php

@@ -3,6 +3,7 @@
 
 namespace TheCodingMachine\ClassExplorer\Glob;
 
+use function chdir;
 use DirectoryIterator;
 use GlobIterator;
 use Mouf\Composer\ClassNameMapper;
@@ -47,14 +48,19 @@ class GlobClassExplorer implements ClassExplorerInterface
      * @var bool
      */
     private $recursive;
+    /**
+     * @var string|null
+     */
+    private $rootPath;
 
-    public function __construct(string $namespace, CacheInterface $cache, ?int $cacheTtl = null, ?ClassNameMapper $classNameMapper = null, bool $recursive = true)
+    public function __construct(string $namespace, CacheInterface $cache, ?int $cacheTtl = null, ?ClassNameMapper $classNameMapper = null, bool $recursive = true, ?string $rootPath = null)
     {
         $this->namespace = $namespace;
         $this->cache = $cache;
         $this->cacheTtl = $cacheTtl;
         $this->classNameMapper = $classNameMapper;
         $this->recursive = $recursive;
+        $this->rootPath = ($rootPath === null) ? __DIR__.'/../../../../../' : rtrim($rootPath, '/').'/';
     }
 
     /**
@@ -107,14 +113,18 @@ private function doGetClasses(): array
      */
     private function getPhpFilesForDir(string $directory): \Iterator
     {
+        $oldCwd = getcwd();
+        chdir($this->rootPath);
         if (!\is_dir($directory)) {
             return new \EmptyIterator();
         }
         if ($this->recursive) {
             $allFiles  = new RecursiveIteratorIterator(new RecursiveDirectoryIterator($directory, RecursiveDirectoryIterator::SKIP_DOTS));
-            return new RegexIterator($allFiles, '/\.php$/i'/*, \RecursiveRegexIterator::GET_MATCH*/);
+            $iterator = new RegexIterator($allFiles, '/\.php$/i'/*, \RecursiveRegexIterator::GET_MATCH*/);
         } else {
-            return new GlobIterator($directory.'/*.php');
+            $iterator = new GlobIterator($directory.'/*.php');
         }
+        chdir($oldCwd);
+        return $iterator;
     }
 }

tests/Glob/GlobClassExplorerTest.php

@@ -11,31 +11,31 @@ class GlobClassExplorerTest extends TestCase
 {
     public function testGetClasses()
     {
-        $explorer = new GlobClassExplorer('\\TheCodingMachine\\ClassExplorer\\', new NullCache());
+        $explorer = new GlobClassExplorer('\\TheCodingMachine\\ClassExplorer\\', new NullCache(), null, null, true, __DIR__.'/../..');
         $classes = $explorer->getClasses();
 
         $this->assertSame([GlobClassExplorer::class, ClassExplorerInterface::class], $classes);
     }
 
     public function testGetClassesNonRecursive()
     {
-        $explorer = new GlobClassExplorer('\\TheCodingMachine\\ClassExplorer\\', new NullCache(), null, null, false);
+        $explorer = new GlobClassExplorer('\\TheCodingMachine\\ClassExplorer\\', new NullCache(), null, null, false, __DIR__.'/../..');
         $classes = $explorer->getClasses();
 
         $this->assertSame([ClassExplorerInterface::class], $classes);
     }
 
     public function testGetDevClasses()
     {
-        $explorer = new GlobClassExplorer('\\TheCodingMachine\\ClassExplorer\\Glob\\', new NullCache(), null, ClassNameMapper::createFromComposerFile(null, null, true));
+        $explorer = new GlobClassExplorer('\\TheCodingMachine\\ClassExplorer\\Glob\\', new NullCache(), null, ClassNameMapper::createFromComposerFile(null, null, true), true, __DIR__.'/../..');
         $classes = $explorer->getClasses();
 
         $this->assertSame([GlobClassExplorer::class, GlobClassExplorerTest::class], $classes);
     }
 
     public function testGetNotExistingClasses()
     {
-        $explorer = new GlobClassExplorer('\\TheCodingMachine\\ClassExplorer\\Glob\\Foobar\\', new NullCache());
+        $explorer = new GlobClassExplorer('\\TheCodingMachine\\ClassExplorer\\Glob\\Foobar\\', new NullCache(), null, null, true, __DIR__.'/../..');
         $classes = $explorer->getClasses();
 
         $this->assertSame([], $classes);