[Scoped] Clean up SmartFileInfo (#2889)

* [Scoped] Clean up SmartFileInfo * [ci-review] Rector Rectify Co-authored-by: GitHub Action <action@github.com>
rectorphp · Sep 1, 2022 · dc22a95 · dc22a95
1 parent 2c454d2
commit dc22a95
Show file tree

Hide file tree

Showing 4 changed files with 1 addition and 47 deletions.
diff --git a/build/target-repository/bootstrap.php b/build/target-repository/bootstrap.php
@@ -23,12 +23,4 @@
             $composerAutoloader->loadClass($class);
         }
     }
-
-    // @deprecated, to be removed
-    if ($class === 'Symplify\SmartFileSystem\SmartFileInfo') {
-        $filePath = __DIR__ . '/vendor/symplify/smart-file-system/src/SmartFileInfo.php';
-        if (file_exists($filePath)) {
-            require_once $filePath;
-        }
-    }
 });
diff --git a/packages/Testing/Fixture/FixtureFileFinder.php b/packages/Testing/Fixture/FixtureFileFinder.php
@@ -7,7 +7,6 @@
 use Iterator;
 use Symfony\Component\Finder\Finder;
 use Symfony\Component\Finder\SplFileInfo;
-use Symplify\SmartFileSystem\SmartFileInfo;
 
 final class FixtureFileFinder
 {
@@ -23,20 +22,6 @@ public static function yieldFilePathsFromDirectory(string $directory, string $su
         }
     }
 
-    /**
-     * @deprecated
-     */
-    public static function yieldDirectory(string $directory, string $suffix = '*.php.inc'): Iterator
-    {
-        $fileInfos = self::findFilesInDirectory($directory, $suffix);
-
-        foreach ($fileInfos as $fileInfo) {
-            // @todo is this one needed? maybe symfony is good enough :)
-            $smartFileInfo = new SmartFileInfo($fileInfo->getRealPath());
-            yield [$smartFileInfo];
-        }
-    }
-
     /**
      * @return SplFileInfo[]
      */

diff --git a/packages/Testing/PHPUnit/AbstractRectorTestCase.php b/packages/Testing/PHPUnit/AbstractRectorTestCase.php
@@ -25,7 +25,6 @@
 use Rector\Testing\Fixture\FixtureSplitter;
 use Rector\Testing\Fixture\FixtureTempFileDumper;
 use Rector\Testing\PHPUnit\Behavior\MovingFilesTrait;
-use Symplify\SmartFileSystem\SmartFileInfo;
 
 abstract class AbstractRectorTestCase extends AbstractTestCase implements RectorTestInterface
 {
@@ -128,15 +127,6 @@ protected function doTestFile(string $fixtureFilePath): void
         $this->doTestFileMatchesExpectedContent($inputFilePath, $expectedFilePath, $fixtureFilePath);
     }
 
-    /**
-     * @deprecated Use doTestFile() with file path instead
-     */
-    protected function doTestFileInfo(SmartFileInfo $fixtureFileInfo): void
-    {
-        $fixtureFileRealPath = $fixtureFileInfo->getRealPath();
-        $this->doTestFile($fixtureFileRealPath);
-    }
-
     protected function getFixtureTempDirectory(): string
     {
         return FixtureTempFileDumper::getTempDirectory();

diff --git a/scoper.php b/scoper.php
@@ -61,12 +61,7 @@
     ],
 
     // expose
-    'expose-classes' => [
-        'Normalizer',
-        // used by public API
-        // @deprecated, to be removed
-        'Symplify\SmartFileSystem\SmartFileInfo',
-    ],
+    'expose-classes' => ['Normalizer'],
     'expose-functions' => ['u', 'b', 's', 'trigger_deprecation'],
     'expose-constants' => ['__RECTOR_RUNNING__', '#^SYMFONY\_[\p{L}_]+$#'],
 
@@ -78,14 +73,6 @@
             $content
         ),
 
-        // @deprecated - to be removed
-        // unprefixed SmartFileInfo - needed in AbstractTestCase
-        static fn (string $filePath, string $prefix, string $content): string => Strings::replace(
-            $content,
-            '#' . $prefix . '\\\\Symplify\\\\SmartFileSystem\\\\SmartFileInfo#',
-            'Symplify\SmartFileSystem\SmartFileInfo'
-        ),
-
         static function (string $filePath, string $prefix, string $content): string {
             if (! \str_ends_with($filePath, 'src/Application/VersionResolver.php')) {
                 return $content;