feat: add support for excludedDirs into FileSymbolLoader

composer-unused · Dec 9, 2022 · 02d3660 · 02d3660
1 parent 3c9b6c5
commit 02d3660
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 4 deletions.
diff --git a/src/Symbol/Loader/FileSymbolLoader.php b/src/Symbol/Loader/FileSymbolLoader.php
@@ -11,22 +11,29 @@
 
 use function array_map;
 use function array_merge;
+use function is_array;
 use function preg_match;
 
+use const DIRECTORY_SEPARATOR;
+
 final class FileSymbolLoader implements SymbolLoaderInterface
 {
     private FileSymbolProvider $fileSymbolProvider;
     /** @var array<string> */
     private array $autoloadTypes;
     private ?string $baseDir;
+    /** @var list<string> */
+    private array $excludedDirs;
 
     /**
      * @param array<string> $autoloadTypes
+     * @param list<string> $excludedDirs
      */
-    public function __construct(FileSymbolProvider $fileSymbolProvider, array $autoloadTypes)
+    public function __construct(FileSymbolProvider $fileSymbolProvider, array $autoloadTypes, array $excludedDirs = [])
     {
         $this->fileSymbolProvider = $fileSymbolProvider;
         $this->autoloadTypes = $autoloadTypes;
+        $this->excludedDirs = $excludedDirs;
     }
 
     public function load(PackageInterface $package): Generator
@@ -54,7 +61,7 @@ public function load(PackageInterface $package): Generator
             ->ignoreVCS(true)
             ->ignoreUnreadableDirs()
             ->followLinks()
-            ->exclude(['vendor']);
+            ->exclude($this->excludedDirs);
 
         $this->fileSymbolProvider->appendFiles($files->getIterator());
 

diff --git a/tests/Integration/AbstractIntegrationTestCase.php b/tests/Integration/AbstractIntegrationTestCase.php
@@ -82,8 +82,9 @@ protected function loadDefinedFileSymbols(
     /**
      * @param list<string> $autoloadTypes
      * @return array<SymbolInterface>
+     * @param list<string> $excludedDirs
      */
-    protected function loadConsumedFileSymbols(string $baseDir, array $autoloadTypes = null): array
+    protected function loadConsumedFileSymbols(string $baseDir, array $autoloadTypes = null, array $excludedDirs = []): array
     {
         $rootPackage = $this->loadPackage($baseDir);
 
@@ -111,7 +112,8 @@ protected function loadConsumedFileSymbols(string $baseDir, array $autoloadTypes
                 ),
                 new FileContentProvider()
             ),
-            $autoloadTypes
+            $autoloadTypes,
+            $excludedDirs
         );
 
         return iterator_to_array(

diff --git a/tests/Integration/Symbol/Loader/FileSymbolLoaderTest.php b/tests/Integration/Symbol/Loader/FileSymbolLoaderTest.php
@@ -12,6 +12,7 @@ class FileSymbolLoaderTest extends AbstractIntegrationTestCase
     private const ONLY_FILE_DEPS = __DIR__ . '/../../../assets/TestProjects/OnlyFileDependencies';
     private const AUTOLOAD_FILES_REQUIRE = __DIR__ . '/../../../assets/TestProjects/AutoloadFilesWithRequire';
     private const ARRAY_NAMESPACE = __DIR__ . '/../../../assets/TestProjects/ArrayNamespace';
+    private const CLASSMAP_AUTOLOAD = __DIR__ . '/../../../assets/TestProjects/ClassmapAutoload';
 
     /**
      * @test
@@ -61,4 +62,14 @@ public function itFindsArraySymbols(): void
         self::assertArrayHasKey('json_encode', $symbols);
         self::assertArrayHasKey('array_keys', $symbols);
     }
+
+    /**
+     * @test
+     */
+    public function itSkipsSymbolsInExcludedDirs(): void
+    {
+        $symbols = $this->loadConsumedFileSymbols(self::CLASSMAP_AUTOLOAD, [AutoloadType::CLASSMAP], ['Redis']);
+
+        self::assertCount(0, $symbols);
+    }
 }