Laravel rule does not validate if a table is indeed being created

Author: Bellangelo

src/Rules/Laravel/EnforceCollationRule.php

@@ -54,7 +54,7 @@ public function processNode(Node $node, Scope $scope): array
         }
 
         $callName = $this->getStaticCallName($node);
-        if ($callName !== 'create' && $callName !== 'table') {
+        if ($callName !== 'create') {
             return [];
         }
 

tests/Rules/Laravel/EnforceCollationRuleTest.php

@@ -81,4 +81,25 @@ public function testDoesNotReportOutsideLaravelMigration(): void
             ],
         );
     }
+
+    public function testTableUsageIsIgnored(): void
+    {
+        $this->analyse(
+            [__DIR__ . '/fixtures/TableUsageWithoutCollation.php'],
+            []
+        );
+    }
+
+    public function testMixedExistenceCheckAndCreateOnlyReportsCreate(): void
+    {
+        $this->analyse(
+            [__DIR__ . '/fixtures/MixedTableCheckAndCreate.php'],
+            [
+                [
+                    'Required: table collation must be "utf8mb4". Why: prevents environment-dependent defaults and keeps schema consistent. Fix: set the table collation explicitly in the migration.',
+                    16,
+                ],
+            ]
+        );
+    }
 }

tests/Rules/Laravel/fixtures/MixedTableCheckAndCreate.php

@@ -0,0 +1,25 @@
+<?php
+
+declare(strict_types=1);
+
+namespace PhpStanMigrationRules\Tests\Rules\Laravel\Fixtures;
+
+use Illuminate\Database\Migrations\Migration;
+use Illuminate\Database\Schema\Blueprint;
+use Illuminate\Support\Facades\Schema;
+
+final class MixedTableCheckAndCreate extends Migration
+{
+    public function up(): void
+    {
+        if (!Schema::hasTable('users')) {
+            Schema::create('users', function (Blueprint $table): void {
+                $table->string('email');
+            });
+        }
+
+        Schema::table('users', function (Blueprint $table): void {
+            $table->string('name');
+        });
+    }
+}

tests/Rules/Laravel/fixtures/TableUsageWithoutCollation.php

@@ -0,0 +1,19 @@
+<?php
+
+declare(strict_types=1);
+
+namespace PhpStanMigrationRules\Tests\Rules\Laravel\Fixtures;
+
+use Illuminate\Database\Migrations\Migration;
+use Illuminate\Database\Schema\Blueprint;
+use Illuminate\Support\Facades\Schema;
+
+final class TableUsageWithoutCollation extends Migration
+{
+    public function up(): void
+    {
+        Schema::table('users', function (Blueprint $table): void {
+            $table->string('email');
+        });
+    }
+}