Add file contents to the Scoper

Continuing some work for box-project/box#31. The Scoper now takes the file
contents as an argument. This allows the scoper to be used with
non-exitent files (e.g. virtual files) or files for which we are
modifying the content like in Box.

Another benefit is that it removes the Scopers dependency to the file
system layer.

src/Console/Command/AddPrefixCommand.php

@@ -246,10 +246,8 @@ private function scopeFile(
         bool $stopOnFailure,
         ConsoleLogger $logger
     ): void {
-        // TODO: use $inputContents instead of doing file_get_contents in the Scopers
-
         try {
-            $scoppedContent = $this->scoper->scope($inputFilePath, $prefix, $patchers, $whitelist, $globalWhitelister);
+            $scoppedContent = $this->scoper->scope($inputFilePath, $inputContents, $prefix, $patchers, $whitelist, $globalWhitelister);
         } catch (Throwable $error) {
             $exception = new ParsingException(
                 sprintf(

src/Scoper.php

@@ -22,6 +22,7 @@ interface Scoper
      * Scope AKA. apply the given prefix to the file in the appropriate way.
      *
      * @param string     $filePath          File to scope
+     * @param string     $contents          File contents
      * @param string     $prefix            Prefix to apply to the file
      * @param callable[] $patchers
      * @param string[]   $whitelist         List of classes to exclude from the scoping.
@@ -31,7 +32,7 @@ interface Scoper
      *
      * @throws ParsingException
      *
-     * @return string Content of the file with the prefix applied
+     * @return string Contents of the file with the prefix applied
      */
-    public function scope(string $filePath, string $prefix, array $patchers, array $whitelist, callable $globalWhitelister): string;
+    public function scope(string $filePath, string $contents, string $prefix, array $patchers, array $whitelist, callable $globalWhitelister): string;
 }

src/Scoper/Composer/InstalledPackagesScoper.php

@@ -32,16 +32,13 @@ public function __construct(Scoper $decoratedScoper)
      *
      * {@inheritdoc}
      */
-    public function scope(string $filePath, string $prefix, array $patchers, array $whitelist, callable $globalWhitelister): string
+    public function scope(string $filePath, string $contents, string $prefix, array $patchers, array $whitelist, callable $globalWhitelister): string
     {
         if (1 !== preg_match(self::$filePattern, $filePath)) {
-            return $this->decoratedScoper->scope($filePath, $prefix, $patchers, $whitelist, $globalWhitelister);
+            return $this->decoratedScoper->scope($filePath, $contents, $prefix, $patchers, $whitelist, $globalWhitelister);
         }
 
-        $decodedJson = json_decode(
-            file_get_contents($filePath),
-            true
-        );
+        $decodedJson = json_decode($contents, true);
 
         $decodedJson = $this->prefixLockPackages($decodedJson, $prefix);
 

src/Scoper/Composer/JsonFileScoper.php

@@ -30,16 +30,13 @@ public function __construct(Scoper $decoratedScoper)
      *
      * {@inheritdoc}
      */
-    public function scope(string $filePath, string $prefix, array $patchers, array $whitelist, callable $globalWhitelister): string
+    public function scope(string $filePath, string $contents, string $prefix, array $patchers, array $whitelist, callable $globalWhitelister): string
     {
         if (1 !== preg_match('/composer\.json$/', $filePath)) {
-            return $this->decoratedScoper->scope($filePath, $prefix, $patchers, $whitelist, $globalWhitelister);
+            return $this->decoratedScoper->scope($filePath, $contents, $prefix, $patchers, $whitelist, $globalWhitelister);
         }
 
-        $decodedJson = json_decode(
-            file_get_contents($filePath),
-            true
-        );
+        $decodedJson = json_decode($contents, true);
 
         $decodedJson = AutoloadPrefixer::prefixPackageAutoloads($decodedJson, $prefix);
 

src/Scoper/NullScoper.php

@@ -21,8 +21,8 @@ final class NullScoper implements Scoper
     /**
      * @inheritdoc
      */
-    public function scope(string $filePath, string $prefix, array $patchers, array $whitelist, callable $globalWhitelister): string
+    public function scope(string $filePath, string $contents, string $prefix, array $patchers, array $whitelist, callable $globalWhitelister): string
     {
-        return file_get_contents($filePath);
+        return $contents;
     }
 }

src/Scoper/PatchScoper.php

@@ -28,16 +28,16 @@ public function __construct(Scoper $decoratedScoper)
     /**
      * @inheritdoc
      */
-    public function scope(string $filePath, string $prefix, array $patchers, array $whitelist, callable $globalWhitelister): string
+    public function scope(string $filePath, string $contents, string $prefix, array $patchers, array $whitelist, callable $globalWhitelister): string
     {
-        $content = $this->decoratedScoper->scope($filePath, $prefix, $patchers, $whitelist, $globalWhitelister);
+        $contents = $this->decoratedScoper->scope($filePath, $contents, $prefix, $patchers, $whitelist, $globalWhitelister);
 
         return array_reduce(
             $patchers,
-            function (string $content, callable $patcher) use ($filePath, $prefix): string {
-                return $patcher($filePath, $prefix, $content);
+            function (string $contents, callable $patcher) use ($filePath, $prefix): string {
+                return $patcher($filePath, $prefix, $contents);
             },
-            $content
+            $contents
         );
     }
 }

src/Scoper/PhpScoper.php

@@ -19,7 +19,7 @@
 use PhpParser\Parser;
 use PhpParser\PrettyPrinter\Standard;
 
-final class PhpScoper implements Scoper
+final class  PhpScoper implements Scoper
 {
     private const FILE_PATH_PATTERN = '/.*\.php$/';
     private const NOT_FILE_BINARY = '/\..+?$/';
@@ -43,15 +43,13 @@ public function __construct(Parser $parser, Scoper $decoratedScoper, TraverserFa
      *
      * @throws PhpParserError
      */
-    public function scope(string $filePath, string $prefix, array $patchers, array $whitelist, callable $globalWhitelister): string
+    public function scope(string $filePath, string $contents, string $prefix, array $patchers, array $whitelist, callable $globalWhitelister): string
     {
-        if (false === $this->isPhpFile($filePath)) {
-            return $this->decoratedScoper->scope($filePath, $prefix, $patchers, $whitelist, $globalWhitelister);
+        if (false === $this->isPhpFile($filePath, $contents)) {
+            return $this->decoratedScoper->scope($filePath, $contents, $prefix, $patchers, $whitelist, $globalWhitelister);
         }
 
-        $content = file_get_contents($filePath);
-
-        $statements = $this->parser->parse($content);
+        $statements = $this->parser->parse($contents);
 
         $traverser = $this->traverserFactory->create($prefix, $whitelist, $globalWhitelister);
 
@@ -62,7 +60,7 @@ public function scope(string $filePath, string $prefix, array $patchers, array $
         return $prettyPrinter->prettyPrintFile($statements)."\n";
     }
 
-    private function isPhpFile(string $filePath): bool
+    private function isPhpFile(string $filePath, string $contents): bool
     {
         if (1 === preg_match(self::FILE_PATH_PATTERN, $filePath)) {
             return true;
@@ -72,8 +70,6 @@ private function isPhpFile(string $filePath): bool
             return false;
         }
 
-        $content = file_get_contents($filePath);
-
-        return 1 === preg_match(self::PHP_BINARY, $content);
+        return 1 === preg_match(self::PHP_BINARY, $contents);
     }
 }

tests/Console/Command/AddPrefixCommandTest.php

@@ -15,6 +15,7 @@
 namespace Humbug\PhpScoper\Console\Command;
 
 use Closure;
+use function file_get_contents;
 use Humbug\PhpScoper\Console\Application;
 use Humbug\PhpScoper\Scoper;
 use InvalidArgumentException;
@@ -195,9 +196,12 @@ public function test_scope_the_given_paths()
             $inputPath = escape_path($root.'/'.$expectedFile);
             $outputPath = escape_path($this->tmp.'/'.$expectedFile);
 
+            $inputContents = file_get_contents($inputPath);
+
             $this->scoperProphecy
                 ->scope(
                     $inputPath,
+                    $inputContents,
                     'MyPrefix',
                     [],
                     [],
@@ -255,10 +259,13 @@ public function test_let_the_file_unchanged_when_cannot_scope_a_file()
             $inputPath = escape_path($root.'/'.$expectedFile);
             $outputPath = escape_path($this->tmp.'/'.$expectedFile);
 
+            $inputContents = file_get_contents($inputPath);
+
             if (null !== $prefixedContents) {
                 $this->scoperProphecy
                     ->scope(
                         $inputPath,
+                        $inputContents,
                         'MyPrefix',
                         [],
                         [],
@@ -272,6 +279,7 @@ public function test_let_the_file_unchanged_when_cannot_scope_a_file()
                 $this->scoperProphecy
                     ->scope(
                         $inputPath,
+                        $inputContents,
                         'MyPrefix',
                         [],
                         [],
@@ -280,7 +288,7 @@ public function test_let_the_file_unchanged_when_cannot_scope_a_file()
                     ->willThrow(new \RuntimeException('Scoping of the file failed'))
                 ;
 
-                $this->fileSystemProphecy->dumpFile($outputPath, file_get_contents($inputPath))->shouldBeCalled();
+                $this->fileSystemProphecy->dumpFile($outputPath, $inputContents)->shouldBeCalled();
             }
         }
 
@@ -327,9 +335,12 @@ public function test_do_not_scope_duplicated_given_paths()
             $inputPath = escape_path($root.'/'.$expectedFile);
             $outputPath = escape_path($this->tmp.'/'.$expectedFile);
 
+            $inputContents = file_get_contents($inputPath);
+
             $this->scoperProphecy
                 ->scope(
                     $inputPath,
+                    $inputContents,
                     'MyPrefix',
                     [],
                     [],
@@ -385,11 +396,13 @@ public function test_scope_the_given_paths_and_the_ones_found_by_the_finder()
             $inputPath = realpath($inputPath);
             $outputPath = escape_path($outputPath);
 
+            $inputContents = file_get_contents($inputPath);
             $prefixedFileContents = 'Random string';
 
             $this->scoperProphecy
                 ->scope(
                     $inputPath,
+                    $inputContents,
                     'MyPrefix',
                     [],
                     [],
@@ -434,6 +447,7 @@ public function test_applies_a_random_prefix_when_none_given()
 
         $this->scoperProphecy
             ->scope(
+                Argument::any(),
                 Argument::any(),
                 Argument::that(
                     function (string $prefix): bool {
@@ -497,9 +511,12 @@ public function test_scope_the_current_working_directory_if_no_path_given()
             $inputPath = escape_path($root.'/'.$expectedFile);
             $outputPath = escape_path($this->tmp.'/'.$expectedFile);
 
+            $inputContents = file_get_contents($inputPath);
+
             $this->scoperProphecy
                 ->scope(
                     $inputPath,
+                    $inputContents,
                     'MyPrefix',
                     [],
                     [],
@@ -545,6 +562,7 @@ public function test_prefix_can_end_by_a_backslash()
 
         $this->scoperProphecy
             ->scope(
+                Argument::any(),
                 Argument::any(),
                 'MyPrefix',
                 [],
@@ -588,6 +606,7 @@ public function test_prefix_can_end_by_multiple_backslashes()
 
         $this->scoperProphecy
             ->scope(
+                Argument::any(),
                 Argument::any(),
                 'MyPrefix',
                 [],
@@ -642,9 +661,12 @@ public function test_an_output_directory_can_be_given()
             $inputPath = escape_path($root.'/'.$expectedFile);
             $outputPath = escape_path($outDir.'/'.$expectedFile);
 
+            $inputContents = file_get_contents($inputPath);
+
             $this->scoperProphecy
                 ->scope(
                     $inputPath,
+                    $inputContents,
                     'MyPrefix',
                     [],
                     [],
@@ -702,9 +724,12 @@ public function test_relative_output_directory_are_made_absolute()
             $inputPath = escape_path($root.'/'.$expectedFile);
             $outputPath = escape_path($this->tmp.DIRECTORY_SEPARATOR.$outDir.'/'.$expectedFile);
 
+            $inputContents = file_get_contents($inputPath);
+
             $this->scoperProphecy
                 ->scope(
                     $inputPath,
+                    $inputContents,
                     'MyPrefix',
                     [],
                     [],
@@ -822,9 +847,12 @@ public function test_attempts_to_use_patch_file_in_current_directory()
             $inputPath = escape_path($root.'/'.$expectedFile);
             $outputPath = escape_path($this->tmp.'/'.$expectedFile);
 
+            $inputContents = file_get_contents($inputPath);
+
             $this->scoperProphecy
                 ->scope(
                     $inputPath,
+                    $inputContents,
                     'MyPrefix',
                     Argument::that(function ($arg) use (&$patchersFound) {
                         $patchersFound = $arg;
@@ -914,6 +942,7 @@ public function test_can_scope_projects_with_invalid_files()
             $this->scoperProphecy
                 ->scope(
                     $inputPath,
+                    $fileContents,
                     'MyPrefix',
                     [],
                     [],