Merge pull request #103 from TheDragonCode/3.x

Fixed `ColumnDoesNotExist`

Author: andrey-helldar

src/Database/BaseChangeMigrationColumn.php

@@ -22,7 +22,9 @@ public function up(): void
     {
         if ($this->hasTable()) {
             Schema::table($this->table(), function (Blueprint $table) {
-                $table->renameColumn('migration', 'action');
+                if ($this->hasColumn('migration') && $this->doesntHaveColumn('action')) {
+                    $table->renameColumn('migration', 'action');
+                }
 
                 $table->unsignedInteger('batch')->change();
             });
@@ -45,6 +47,16 @@ protected function hasTable(): bool
         return Schema::hasTable($this->table());
     }
 
+    protected function hasColumn(string $column): bool
+    {
+        return Schema::hasColumn($this->table(), $column);
+    }
+
+    protected function doesntHaveColumn(string $column): bool
+    {
+        return ! $this->hasColumn($column);
+    }
+
     protected function table(): string
     {
         return $this->config->table();

tests/Migrations/MigrationTest.php

@@ -0,0 +1,40 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Tests\Migrations;
+
+use Doctrine\DBAL\Driver\PDOSqlite\Driver;
+use DragonCode\LaravelActions\Constants\Names;
+use Illuminate\Support\Facades\DB;
+use Illuminate\Support\Facades\Schema;
+use Tests\TestCase;
+
+class MigrationTest extends TestCase
+{
+    public function testRunMigrationAfterInstall(): void
+    {
+        if (! class_exists(Driver::class)) {
+            $this->assertTrue(true);
+
+            return;
+        }
+
+        DB::table('migrations')->truncate();
+
+        Schema::connection($this->database)->dropIfExists($this->table);
+
+        $this->assertDatabaseCount('migrations', 0);
+        $this->assertDatabaseDoesntTable($this->table);
+
+        $this->artisan(Names::INSTALL)->run();
+
+        $this->assertDatabaseCount('migrations', 0);
+        $this->assertDatabaseHasTable($this->table);
+
+        $this->artisan('migrate')->run();
+
+        $this->assertDatabaseCount('migrations', 1);
+        $this->assertDatabaseHas('migrations', ['migration' => '2022_08_18_180137_change_migration_actions_table']);
+    }
+}