Fixes #62 - Test for file manager to ignore files

bmitch · Aug 24, 2017 · 181c574 · 181c574
1 parent 5cf8720
commit 181c574
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 1 deletion.
diff --git a/src/Managers/FileManager.php b/src/Managers/FileManager.php
@@ -7,6 +7,7 @@
 use RecursiveDirectoryIterator;
 use RecursiveIteratorIterator;
 use Churn\Values\File;
+use SplFileInfo;
 
 class FileManager
 {
@@ -34,13 +35,31 @@ public function getPhpFiles(string $path): FileCollection
                 continue;
             }
 
-            if (in_array($file->getPathname(), $this->config->getFilesToIgnore())) {
+            if ($this->fileShouldBeIgnored($file)) {
                 continue;
             }
 
+
             $files->push(new File(['displayPath' => $file->getPathName(), 'fullPath' => $file->getRealPath()]));
         }
 
         return $files;
     }
+
+    /**
+     * Determines if a file should be ignored.
+     * @param \SplFileInfo $file File.
+     * @return boolean
+     */
+    private function fileShouldBeIgnored(SplFileInfo $file): bool
+    {
+        foreach ($this->config->getFilesToIgnore() as $fileToIgnore) {
+            $fileToIgnore = str_replace('/', '\/', $fileToIgnore);
+            if (preg_match("/{$fileToIgnore}/", $file->getPathName())) {
+                return true;
+            }
+        }
+
+        return false;
+    }
 }
diff --git a/tests/Unit/Managers/FileManagerTest.php b/tests/Unit/Managers/FileManagerTest.php
@@ -26,6 +26,13 @@ public function it_can_get_the_php_files_in_a_filter()
         $this->assertCount(3, $this->fileManager->getPhpFiles(__DIR__ . '/../Assets'));
     }
 
+    /** @test **/
+    public function it_ignores_files_specified_to_ignore_in_the_config()
+    {
+        $fileManager = new FileManager(new Config(['filesToIgnore' => ['Assets/Baz.php']]));
+        $this->assertCount(2, $fileManager->getPhpFiles(__DIR__ . '/../Assets'));
+    }
+
     public function setup()
     {
         $this->fileManager = new FileManager(new Config);