Support --skip-file with fnmatch patterns (#99)

* Support `--skip-file` with `fnmatch` patterns

* Update PhpFilesFinderTest.php

* Update PhpFilesFinder.php

* Update PhpFilesFinder.php

Author: staabm

README.md

@@ -52,9 +52,11 @@ No more! Add this command to CI to spot these:
 
 ```bash
 vendor/bin/swiss-knife check-commented-code <directory>
-vendor/bin/swiss-knife check-commented-code packages --line-limit 5
+vendor/bin/swiss-knife check-commented-code packages --line-limit 5 --skip-file '*Controller.php'
 ```
 
+
+
 <br>
 
 ## 3. Reach full PSR-4
@@ -114,6 +116,12 @@ Do you want to skip file or two?
 vendor/bin/swiss-knife finalize-classes src tests --skip-file src/SpecialProxy.php
 ```
 
+Skip is also support with `fnmatch()` patterns:
+
+```bash
+vendor/bin/swiss-knife finalize-classes src tests --skip-file '*Controller.php'
+```
+
 <br>
 
 ## 5. Privatize local class constants

src/Finder/PhpFilesFinder.php

@@ -33,7 +33,13 @@ private static function createFinderForPathsAndExcludedPaths(array $paths, array
         Assert::allFileExists($paths);
 
         Assert::allString($excludedPaths);
-        Assert::allFileExists($excludedPaths);
+        $excludedFileNames = [];
+        foreach ($excludedPaths as $excludedPath) {
+            if (! str_contains($excludedPath, '*')) {
+                $excludedFileNames[] = $excludedPath;
+            }
+        }
+        Assert::allFileExists($excludedFileNames);
 
         return Finder::create()
             ->files()
@@ -46,7 +52,12 @@ private static function createFinderForPathsAndExcludedPaths(array $paths, array
             // exclude paths, as notPaths() does no work
             ->filter(static function (SplFileInfo $splFileInfo) use ($excludedPaths): bool {
                 foreach ($excludedPaths as $excludedPath) {
-                    if (str_contains($splFileInfo->getRealPath(), $excludedPath)) {
+                    $realpath = $splFileInfo->getRealPath();
+                    if (str_contains($realpath, $excludedPath)) {
+                        return false;
+                    }
+
+                    if (str_contains($excludedPath, '*') && \fnmatch($excludedPath, $realpath)) {
                         return false;
                     }
                 }

tests/Finder/Fixture/AController.php

@@ -0,0 +1,3 @@
+<?php
+
+// intentional left empty

tests/Finder/Fixture/AModel.php

@@ -0,0 +1,3 @@
+<?php
+
+// intentional left empty

tests/Finder/Fixture/BController.php

@@ -0,0 +1,3 @@
+<?php
+
+// intentional left empty

tests/Finder/PhpFilesFinderTest.php

@@ -0,0 +1,38 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\SwissKnife\Tests\Finder;
+
+use Rector\SwissKnife\Tests\AbstractTestCase;
+
+final class PhpFilesFinderTest extends AbstractTestCase
+{
+    public function testExcludeByFnMatch(): void
+    {
+        $files = \Rector\SwissKnife\Finder\PhpFilesFinder::find([__DIR__ . '/Fixture'], ['*Controller.php']);
+
+        $this->assertSame(1, count($files));
+
+        sort($files); // make order predictable across operating systems
+        $file = array_pop($files);
+        $this->assertNotNull($file);
+        $this->assertStringContainsString('AModel.php', $file->getRealPath());
+    }
+
+    public function testExcludeByFnMatch2(): void
+    {
+        $files = \Rector\SwissKnife\Finder\PhpFilesFinder::find([__DIR__ . '/Fixture'], ['*Model.php']);
+
+        $this->assertSame(2, count($files));
+
+        sort($files); // make order predictable across operating systems
+        $file = array_pop($files);
+        $this->assertNotNull($file);
+        $this->assertStringContainsString('BController.php', $file->getRealPath());
+
+        $file = array_pop($files);
+        $this->assertNotNull($file);
+        $this->assertStringContainsString('AController.php', $file->getRealPath());
+    }
+}