feat(upgrade): automate map namespace change

Author: innocenzi

packages/upgrade/config/sets/level/up-to-tempest-30.php

@@ -0,0 +1,14 @@
+<?php
+
+declare(strict_types=1);
+
+use Rector\Config\RectorConfig;
+use Tempest\Upgrade\Set\TempestSetList;
+
+return static function (RectorConfig $config): void {
+    $config->sets([
+        TempestSetList::TEMPEST_20,
+        TempestSetList::TEMPEST_28,
+        TempestSetList::TEMPEST_30,
+    ]);
+};

packages/upgrade/config/sets/tempest30.php

@@ -0,0 +1,11 @@
+<?php
+
+use Rector\Config\RectorConfig;
+use Tempest\Upgrade\Tempest3\UpdateMapperFunctionImportsRector;
+
+return static function (RectorConfig $config): void {
+    $config->importNames();
+    $config->importShortClasses();
+
+    $config->rule(UpdateMapperFunctionImportsRector::class);
+};

packages/upgrade/src/Set/TempestLevelSetList.php

@@ -8,4 +8,5 @@ final class TempestLevelSetList
 {
     public const string UP_TO_TEMPEST_20 = __DIR__ . '/../../config/sets/level/up-to-tempest-20.php';
     public const string UP_TO_TEMPEST_28 = __DIR__ . '/../../config/sets/level/up-to-tempest-28.php';
+    public const string UP_TO_TEMPEST_30 = __DIR__ . '/../../config/sets/level/up-to-tempest-30.php';
 }

packages/upgrade/src/Set/TempestSetList.php

@@ -8,4 +8,5 @@ final class TempestSetList
 {
     public const string TEMPEST_20 = __DIR__ . '/../../config/sets/tempest20.php';
     public const string TEMPEST_28 = __DIR__ . '/../../config/sets/tempest28.php';
+    public const string TEMPEST_30 = __DIR__ . '/../../config/sets/tempest30.php';
 }

packages/upgrade/src/Tempest3/UpdateMapperFunctionImportsRector.php

@@ -0,0 +1,50 @@
+<?php
+
+namespace Tempest\Upgrade\Tempest3;
+
+use PhpParser\Node;
+use Rector\Rector\AbstractRector;
+
+final class UpdateMapperFunctionImportsRector extends AbstractRector
+{
+    public function getNodeTypes(): array
+    {
+        return [
+            Node\UseItem::class,
+            Node\Expr\FuncCall::class,
+        ];
+    }
+
+    public function refactor(Node $node): ?int
+    {
+        if ($node instanceof Node\UseItem) {
+            if ($node->name->toString() === 'Tempest\map') {
+                $node->name = new Node\Name('Tempest\Mapper\map');
+            }
+
+            if ($node->name->toString() === 'Tempest\make') {
+                $node->name = new Node\Name('Tempest\Mapper\make');
+            }
+
+            return null;
+        }
+
+        if ($node instanceof Node\Expr\FuncCall && $node->name instanceof Node\Name) {
+            $functionName = $node->name->toString();
+
+            if ($functionName === 'Tempest\map') {
+                $node->name = new Node\Name\FullyQualified('Tempest\Mapper\map');
+
+                return null;
+            }
+
+            if ($functionName === 'Tempest\make') {
+                $node->name = new Node\Name\FullyQualified('Tempest\Mapper\make');
+
+                return null;
+            }
+        }
+
+        return null;
+    }
+}

packages/upgrade/tests/Tempest3/Fixtures/FullyQualifiedMakeCall.input.php

@@ -0,0 +1,9 @@
+<?php
+
+final class FullyQualifiedMakeCall
+{
+    public function __invoke()
+    {
+        return \Tempest\make(Author::class)->from(['name' => 'Jon Doe']);
+    }
+}

packages/upgrade/tests/Tempest3/Fixtures/FullyQualifiedMapCall.input.php

@@ -0,0 +1,9 @@
+<?php
+
+final class FullyQualifiedMapCall
+{
+    public function __invoke(array $data)
+    {
+        return \Tempest\map($data)->to(Author::class);
+    }
+}

packages/upgrade/tests/Tempest3/Fixtures/MakeNamespaceChange.input.php

@@ -0,0 +1,11 @@
+<?php
+
+use function Tempest\make;
+
+final class MakeNamespaceChange
+{
+    public function __invoke()
+    {
+        return make(Author::class)->from(['name' => 'Jon Doe']);
+    }
+}

packages/upgrade/tests/Tempest3/Fixtures/MapNamespaceChange.input.php

@@ -0,0 +1,11 @@
+<?php
+
+use function Tempest\map;
+
+final class MapNamespaceChange
+{
+    public function __invoke(array $data)
+    {
+        return map($data)->to(Author::class);
+    }
+}

packages/upgrade/tests/Tempest3/Tempest3RectorTest.php

@@ -0,0 +1,45 @@
+<?php
+
+namespace Tempest\Upgrade\Tests\Tempest3;
+
+use PHPUnit\Framework\TestCase;
+use Tempest\Upgrade\Tests\RectorTester;
+
+final class Tempest3RectorTest extends TestCase
+{
+    private RectorTester $rector {
+        get => new RectorTester(__DIR__ . '/tempest30_rector.php');
+    }
+
+    public function test_map_namespace_change(): void
+    {
+        $this->rector
+            ->runFixture(__DIR__ . '/Fixtures/MapNamespaceChange.input.php')
+            ->assertContains('use function Tempest\Mapper\map;')
+            ->assertNotContains('use function Tempest\map;');
+    }
+
+    public function test_make_namespace_change(): void
+    {
+        $this->rector
+            ->runFixture(__DIR__ . '/Fixtures/MakeNamespaceChange.input.php')
+            ->assertContains('use function Tempest\Mapper\make;')
+            ->assertNotContains('use function Tempest\make;');
+    }
+
+    public function test_fully_qualified_map_call(): void
+    {
+        $this->rector
+            ->runFixture(__DIR__ . '/Fixtures/FullyQualifiedMapCall.input.php')
+            ->assertContains('use Tempest\Mapper\map;')
+            ->assertContains('return map($data)->to(Author::class);');
+    }
+
+    public function test_fully_qualified_make_call(): void
+    {
+        $this->rector
+            ->runFixture(__DIR__ . '/Fixtures/FullyQualifiedMakeCall.input.php')
+            ->assertContains('use Tempest\Mapper\make;')
+            ->assertContains('return make(Author::class)');
+    }
+}