fix #34

Insolita · Apr 14, 2021 · 88f3111 · 88f3111
1 parent 21e66e9
commit 88f3111
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 4 deletions.
diff --git a/Lib/Config.php b/Lib/Config.php
@@ -35,9 +35,9 @@ public function __construct(string $composerJsonPath, string $vendorPath, array
         }
         $this->composerJsonPath = $composerJsonPath;
         $this->vendorPath = realpath(rtrim($vendorPath, DIRECTORY_SEPARATOR)) . DIRECTORY_SEPARATOR;
-        $this->scanDirectories = array_map(static function ($path) {
-            return realpath(rtrim($path, DIRECTORY_SEPARATOR)) . DIRECTORY_SEPARATOR;
-        }, $scanDirectories);
+        $this->scanDirectories = array_filter(array_map(static function ($path) {
+            return realpath(rtrim($path, DIRECTORY_SEPARATOR). DIRECTORY_SEPARATOR);
+        }, $scanDirectories));
     }
 
     public function getComposerJsonPath(): string

diff --git a/tests/ConfigTest.php b/tests/ConfigTest.php
@@ -0,0 +1,19 @@
+<?php
+
+namespace tests;
+
+use insolita\Scanner\Lib\Config;
+use PHPUnit\Framework\TestCase;
+
+class ConfigTest extends TestCase
+{
+    public  function testConfigShouldSkipUnexistedDirs()
+    {
+        $params = require __DIR__ . '/scanner_test_config_dev.php';
+        $config = Config::create($params);
+        $dirs = $config->getScanDirectories();
+        $this->assertContains(__DIR__.'/stubs/', $dirs);
+        $this->assertNotContains(__DIR__.'/not_existed/', $dirs);
+        $this->assertNotContains(__DIR__.'/missing/', $dirs);
+    }
+}
diff --git a/tests/scanner_test_config_dev.php b/tests/scanner_test_config_dev.php
@@ -3,6 +3,6 @@
 return [
     'composerJsonPath' => $projectPath . '/stub_composer.json',
     'vendorPath' => $projectPath . '/../vendor/',
-    'scanDirectories' => [$projectPath . '/stubs/'],
+    'scanDirectories' => [$projectPath . '/stubs/', $projectPath.'/not_existed/', $projectPath.'/missing'],
     'requireDev' => true
 ];