fix: tests

Author: me-shaon

database/migrations/create_request_analytics_table.php

@@ -11,28 +11,30 @@ public function up()
         $tableName = config('request-analytics.database.table', 'request_analytics');
         $connection = config('request-analytics.database.connection');
 
-        Schema::connection($connection)->create($tableName, function (Blueprint $table) {
-            $table->id();
-            $table->string('path');
-            $table->string('page_title');
-            $table->string('ip_address');
-            $table->string('operating_system')->nullable();
-            $table->string('browser')->nullable();
-            $table->string('device')->nullable();
-            $table->string('screen')->nullable();
-            $table->string('referrer')->nullable();
-            $table->string('country')->nullable();
-            $table->string('city')->nullable();
-            $table->string('language')->nullable();
-            $table->tinyText('query_params')->nullable();
-            $table->string('session_id');
-            $table->string('visitor_id')->nullable()->index();
-            $table->unsignedBigInteger('user_id')->nullable();
-            $table->string('http_method');
-            $table->string('request_category');
-            $table->bigInteger('response_time')->nullable()->comment('Response time in milliseconds');
-            $table->timestamp('visited_at');
-        });
+        if (! Schema::connection($connection)->hasTable($tableName)) {
+            Schema::connection($connection)->create($tableName, function (Blueprint $table) {
+                $table->id();
+                $table->string('path');
+                $table->string('page_title')->nullable();
+                $table->string('ip_address');
+                $table->string('operating_system')->nullable();
+                $table->string('browser')->nullable();
+                $table->string('device')->nullable();
+                $table->string('screen')->nullable();
+                $table->string('referrer')->nullable();
+                $table->string('country')->nullable();
+                $table->string('city')->nullable();
+                $table->string('language')->nullable();
+                $table->text('query_params')->nullable();
+                $table->string('session_id');
+                $table->string('visitor_id')->nullable();
+                $table->unsignedBigInteger('user_id')->nullable();
+                $table->string('http_method');
+                $table->string('request_category');
+                $table->bigInteger('response_time')->nullable();
+                $table->timestamp('visited_at');
+            });
+        }
     }
 
     public function down()

src/RequestAnalyticsServiceProvider.php

@@ -15,13 +15,10 @@ class RequestAnalyticsServiceProvider extends PackageServiceProvider
 {
     public function configurePackage(Package $package): void
     {
+        // Manually publish migrations with proper timestamp
         $this->publishes([
-            __DIR__.'/../resources/assets' => public_path('/'),
-        ], 'laravel-request-analytics-assets');
-
-        $this->publishes([
-            __DIR__.'/../config/request-analytics.php' => config_path('request-analytics.php'),
-        ], 'laravel-request-analytics-config');
+            __DIR__.'/../database/migrations/create_request_analytics_table.php' => database_path('migrations/'.date('Y_m_d_His').'_create_request_analytics_table.php'),
+        ], 'laravel-request-analytics-migrations');
 
         $package
             ->name('laravel-request-analytics')
@@ -30,7 +27,6 @@ public function configurePackage(Package $package): void
             ->hasRoute('web')
             ->hasRoute('api')
             ->hasAssets()
-            ->hasMigration('create_request_analytics_table')
             ->hasCommand(RequestAnalyticsCommand::class);
 
         $this->registerMiddlewareAsAliases();

tests/Feature/Commands/SetupCommandTest.php

@@ -4,21 +4,33 @@
 
 namespace MeShaon\RequestAnalytics\Tests\Feature\Commands;
 
+use Illuminate\Foundation\Testing\RefreshDatabase;
 use Illuminate\Support\Facades\File;
 use Illuminate\Support\Facades\Schema;
-use MeShaon\RequestAnalytics\Tests\TestCase;
+use MeShaon\RequestAnalytics\RequestAnalyticsServiceProvider;
+use Orchestra\Testbench\TestCase as Orchestra;
 use PHPUnit\Framework\Attributes\Test;
 
-class SetupCommandTest extends TestCase
+class SetupCommandTest extends Orchestra
 {
-    protected function setUp(): void
+    use RefreshDatabase;
+
+    protected function getPackageProviders($app): array
+    {
+        return [RequestAnalyticsServiceProvider::class];
+    }
+
+    public function getEnvironmentSetUp($app): void
     {
-        parent::setUp();
+        config()->set('database.default', 'testing');
+        config()->set('database.connections.testing', [
+            'driver' => 'sqlite',
+            'database' => ':memory:',
+            'prefix' => '',
+        ]);
 
-        // Drop the table if it exists since we want to test the migration process
-        if (Schema::hasTable('request_analytics')) {
-            Schema::dropIfExists('request_analytics');
-        }
+        // Only essential config needed for setup command testing
+        config()->set('request-analytics.database.table', 'request_analytics');
     }
 
     #[Test]