Fix SchemaAggregation when renaming tables through non-static method call

src/Properties/SchemaAggregator.php

@@ -76,7 +76,7 @@ private function addUpMethodStatements(array $stmts): void
                         break;
 
                     case 'rename':
-                        $this->renameTable($stmt->expr);
+                        $this->renameTableThroughStaticCall($stmt->expr);
                 }
             }
         }
@@ -359,6 +359,9 @@ private function processColumnUpdates(string $tableName, string $argName, array
                             break;
 
                         case 'rename':
+                            $this->renameTableThroughMethodCall($table, $stmt->expr);
+                            break;
+
                         case 'renamecolumn':
                             if ($secondArg instanceof PhpParser\Node\Scalar\String_) {
                                 $table->renameColumn($columnName, $secondArg->value);
@@ -405,7 +408,7 @@ private function dropTable(PhpParser\Node\Expr\StaticCall $call): void
         unset($this->tables[$tableName]);
     }
 
-    private function renameTable(PhpParser\Node\Expr\StaticCall $call): void
+    private function renameTableThroughStaticCall(PhpParser\Node\Expr\StaticCall $call): void
     {
         if (! isset($call->args[0], $call->args[1])
             || ! $call->getArgs()[0]->value instanceof PhpParser\Node\Scalar\String_
@@ -417,6 +420,28 @@ private function renameTable(PhpParser\Node\Expr\StaticCall $call): void
         $oldTableName = $call->getArgs()[0]->value->value;
         $newTableName = $call->getArgs()[1]->value->value;
 
+        $this->renameTable($oldTableName, $newTableName);
+    }
+
+    private function renameTableThroughMethodCall(SchemaTable $oldTable, PhpParser\Node\Expr\MethodCall $call): void
+    {
+        if (! isset($call->args[0])
+            || ! $call->getArgs()[0]->value instanceof PhpParser\Node\Scalar\String_
+        ) {
+            return;
+        }
+
+        /** @var PhpParser\Node\Scalar\String_ $methodCallArgument */
+        $methodCallArgument = $call->getArgs()[0]->value;
+
+        $oldTableName = $oldTable->name;
+        $newTableName = $methodCallArgument->value;
+
+        $this->renameTable($oldTableName, $newTableName);
+    }
+
+    private function renameTable(string $oldTableName, string $newTableName): void
+    {
         if (! isset($this->tables[$oldTableName])) {
             return;
         }

tests/Unit/MigrationHelperTest.php

@@ -106,6 +106,20 @@ public function it_can_handle_use_of_after_method_in_migration(): void
         self::assertSame('string', $tables['users']->columns['updated_at']->readableType);
     }
 
+    /** @test */
+    public function it_can_handle_alter_table_rename()
+    {
+        $migrationHelper = new MigrationHelper($this->parser, [
+            __DIR__.'/data/rename_migrations',
+        ], $this->fileHelper);
+
+        $tables = $migrationHelper->initializeTables();
+
+        self::assertCount(1, $tables);
+        self::assertArrayNotHasKey('users', $tables);
+        self::assertArrayHasKey('accounts', $tables);
+    }
+
     /**
      * @param  array<string, SchemaTable>  $tables
      */

tests/Unit/data/rename_migrations/2021_12_08_000000_rename_users_table.php

@@ -0,0 +1,29 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Tests\Unit\RenameMigrations;
+
+use Illuminate\Database\Migrations\Migration;
+use Illuminate\Database\Schema\Blueprint;
+use Illuminate\Support\Facades\Schema;
+
+class RenameUsersTable extends Migration
+{
+    /**
+     * Run the migrations.
+     */
+    public function up(): void
+    {
+        Schema::create('users', static function (Blueprint $table) {
+            $table->bigIncrements('id');
+            $table->string('name')->nullable();
+            $table->string('email')->unique();
+            $table->timestamps();
+        });
+
+        Schema::table('users', function (Blueprint $table) {
+            $table->rename('accounts');
+        });
+    }
+}