Merge pull request #65 from singlestore-labs/ISSUE-63

Fixed column renaming

Author: AdalbertMemSQL

src/Schema/Grammar.php

@@ -233,4 +233,17 @@ public function compileRename(Blueprint $blueprint, Fluent $command)
 
         return "alter table {$from} rename to ".$this->wrapTable($command->to);
     }
+
+    /**
+     * Compile a rename column command.
+     *
+     * @return array|string
+     */
+    public function compileRenameColumn(Blueprint $blueprint, Fluent $command, Connection $connection)
+    {
+        return sprintf('alter table %s change %s %s',
+            $this->wrapTable($blueprint),
+            $this->wrap($command->from),
+            $this->wrap($command->to));
+    }
 }

tests/Hybrid/RenameTableTest.php renamed to tests/Hybrid/RenameTest.php

@@ -54,4 +54,36 @@ public function rename_table()
         $this->assertCount(1, $statements);
         $this->assertEquals('alter table `test` rename to `test_renamed`', $statements[0]);
     }
+
+    /** @test */
+    public function rename_column()
+    {
+        if ($this->runHybridIntegrations()) {
+            $cached = $this->mockDatabaseConnection;
+
+            $this->mockDatabaseConnection = false;
+
+            $this->createTable(function (Blueprint $table) {
+                $table->id();
+                $table->string('data');
+            });
+
+            Schema::table('test', function (Blueprint $table) {
+                $table->renameColumn('data', 'data1');
+            });
+
+            $columnNames = Schema::getColumnListing('test');
+            $this->assertEquals(['id', 'data1'], $columnNames);
+
+            $this->mockDatabaseConnection = $cached;
+        }
+
+        $blueprint = new Blueprint('test');
+        $blueprint->renameColumn('data', 'data1');
+
+        $statements = $blueprint->toSql($this->getConnection(), $this->getGrammar());
+
+        $this->assertCount(1, $statements);
+        $this->assertEquals('alter table `test` change `data` `data1`', $statements[0]);
+    }
 }