Add Configuration::withPrefix() (#629)

Author: theofidry

src/Configuration/Configuration.php

@@ -74,6 +74,21 @@ public function getPath(): ?string
         return $this->path;
     }
 
+    /**
+     * @param non-empty-string $prefix
+     */
+    public function withPrefix(string $prefix): self
+    {
+        return new self(
+            $this->path,
+            $prefix,
+            $this->filesWithContents,
+            $this->excludedFilesWithContents,
+            $this->patcher,
+            $this->symbolsConfiguration,
+        );
+    }
+
     /**
      * @return non-empty-string
      */

src/PhpParser/Node/PrefixedName.php

@@ -16,6 +16,7 @@
 
 use PhpParser\Node\Name\FullyQualified;
 
+// TODO:review
 final class PrefixedName extends FullyQualified
 {
     private FullyQualified $prefixedName;

tests/Configuration/ConfigurationTest.php

@@ -14,8 +14,9 @@
 
 namespace Humbug\PhpScoper\Configuration;
 
+use Humbug\PhpScoper\Patcher\FakePatcher;
+use Humbug\PhpScoper\Patcher\Patcher;
 use Humbug\PhpScoper\Patcher\PatcherChain;
-use Humbug\PhpScoper\Whitelist;
 use InvalidArgumentException;
 use PHPUnit\Framework\TestCase;
 
@@ -42,12 +43,34 @@ public function test_it_validates_the_prefix(
             [],
             new PatcherChain([]),
             SymbolsConfiguration::create(),
-            [],
-            [],
-            [],
         );
     }
 
+    public function test_it_can_create_a_new_instance_with_a_different_prefix(): void
+    {
+        $values = [
+            '/path/to/config',
+            'initialPrefix',
+            ['/path/to/fileA' => ['/path/to/fileA', 'fileAContent']],
+            ['/path/to/fileB' => ['/path/to/fileB', 'fileBContent']],
+            new FakePatcher(),
+            SymbolsConfiguration::create(),
+        ];
+
+        $config = new Configuration(...$values);
+
+        // Sanity check
+        self::assertStateIs($config, ...$values);
+
+        $newConfig = $config->withPrefix('newPrefix');
+
+        $expectedNewConfigValues = $values;
+        $expectedNewConfigValues[1] = 'newPrefix';
+
+        self::assertStateIs($config, ...$values);
+        self::assertStateIs($newConfig, ...$expectedNewConfigValues);
+    }
+
     public static function prefixProvider(): iterable
     {
         yield [
@@ -60,4 +83,30 @@ public static function prefixProvider(): iterable
             'Invalid namespace separator sequence. Got "App\\\\Foo"',
         ];
     }
+
+    private static function assertStateIs(
+        Configuration $configuration,
+        ?string $expectedPath,
+        string $expectedPrefix,
+        array $expectedFilesWithContents,
+        array $expectedExcludedFilesWithContents,
+        Patcher $expectedPatcher,
+        SymbolsConfiguration $expectedSymbolsConfiguration
+    ): void {
+        self::assertSame($expectedPath, $configuration->getPath());
+        self::assertSame($expectedPrefix, $configuration->getPrefix());
+        self::assertEqualsCanonicalizing(
+            $expectedFilesWithContents,
+            $configuration->getFilesWithContents(),
+        );
+        self::assertEqualsCanonicalizing(
+            $expectedExcludedFilesWithContents,
+            $configuration->getExcludedFilesWithContents(),
+        );
+        self::assertEquals($expectedPatcher, $configuration->getPatcher());
+        self::assertEquals(
+            $expectedSymbolsConfiguration,
+            $configuration->getSymbolsConfiguration(),
+        );
+    }
 }