Improve test to use artisan command instead of accessing static property

Co-authored-by: arietimmerman <[email protected]>

Author: Copilot

tests/ServiceProviderTest.php

@@ -3,6 +3,7 @@
 namespace ArieTimmerman\Laravel\SCIMServer\Tests;
 
 use ArieTimmerman\Laravel\SCIMServer\ServiceProvider;
+use Illuminate\Support\Facades\Artisan;
 use Orchestra\Testbench\TestCase as BaseTestCase;
 
 class ServiceProviderTest extends BaseTestCase
@@ -12,42 +13,35 @@ protected function getPackageProviders($app)
         return [ServiceProvider::class];
     }
 
-    public function testMigrationsArePublishable()
+    public function testMigrationsCanBePublished()
     {
-        // Get the publishable groups
-        $published = ServiceProvider::$publishes[ServiceProvider::class] ?? [];
-
-        // Check that the migration publishing is configured
-        $migrationPublishes = collect($published)
-            ->filter(function ($destination, $source) {
-                return str_contains($source, 'database/migrations');
-            });
-
-        $this->assertNotEmpty($migrationPublishes, 'Migrations should be publishable');
+        // Test that the vendor:publish command can find the migration tag
+        $result = Artisan::call('vendor:publish', [
+            '--tag' => 'laravel-scim-migrations',
+            '--provider' => ServiceProvider::class,
+            '--dry-run' => true,
+        ]);
+
+        // The command should succeed (return 0)
+        $this->assertEquals(0, $result, 'Migrations should be publishable via vendor:publish command');
     }
 
-    public function testConfigIsPublishable()
+    public function testConfigCanBePublished()
     {
-        // Get the publishable groups
-        $published = ServiceProvider::$publishes[ServiceProvider::class] ?? [];
-
-        // Check that the config publishing is configured
-        $configPublishes = collect($published)
-            ->filter(function ($destination, $source) {
-                return str_contains($source, 'config/scim.php');
-            });
-
-        $this->assertNotEmpty($configPublishes, 'Config should be publishable');
+        // Test that the vendor:publish command can find the config tag
+        $result = Artisan::call('vendor:publish', [
+            '--tag' => 'laravel-scim',
+            '--provider' => ServiceProvider::class,
+            '--dry-run' => true,
+        ]);
+
+        // The command should succeed (return 0)
+        $this->assertEquals(0, $result, 'Config should be publishable via vendor:publish command');
     }
 
-    public function testMigrationsAreNotAutoLoaded()
+    public function testMigrationFileExistsInPackage()
     {
-        // Create an instance of the service provider
-        $provider = new ServiceProvider($this->app);
-
-        // Use reflection to check if loadMigrationsFrom was called
-        // Since we can't directly check if a migration path is loaded,
-        // we just verify the migration file exists in the package
+        // Verify the migration file exists in the package
         $migrationPath = realpath(__DIR__ . '/../database/migrations/2021_01_01_000003_add_scim_fields_to_users_table.php');
 
         $this->assertFileExists($migrationPath, 'Migration file should exist in the package');