Replace assertExitCode(0) with assertSuccessful() in tests

Updated EmulatedImportTest to use assertSuccessful() instead of assertExitCode(0) for better readability and alignment with Laravel's testing conventions.

Author: stevebauman

tests/Feature/Emulator/EmulatedImportTest.php

@@ -48,7 +48,7 @@ public function test_ldap_users_are_imported()
             'provider' => 'ldap-database',
             '--attributes' => 'cn,mail,objectguid',
             '--no-interaction',
-        ])->assertExitCode(0);
+        ])->assertSuccessful();
 
         $imported = TestUserModelStub::get();
 
@@ -74,7 +74,7 @@ public function test_import_fails_when_required_attributes_are_missing()
         $this->artisan('ldap:import', [
             'provider' => 'ldap-database',
             '--no-interaction',
-        ])->assertExitCode(0);
+        ])->assertSuccessful();
 
         Event::assertDispatched(ImportFailed::class);
     }
@@ -92,7 +92,7 @@ public function test_disabled_users_are_soft_deleted_when_flag_is_set()
             'provider' => 'ldap-database',
             '--no-interaction',
             '--delete' => true,
-        ])->assertExitCode(0);
+        ])->assertSuccessful();
 
         $created = TestUserModelStub::withTrashed()->first();
 
@@ -123,7 +123,7 @@ public function test_enabled_users_who_are_soft_deleted_are_restored_when_flag_i
             '--no-interaction',
             '--delete' => true,
             '--restore' => true,
-        ])->assertExitCode(0);
+        ])->assertSuccessful();
 
         $this->assertFalse($database->fresh()->trashed());
     }
@@ -180,7 +180,7 @@ public function test_missing_users_are_soft_deleted_when_flag_is_enabled()
             'provider' => 'ldap-database',
             '--no-interaction',
             '--delete-missing',
-        ])->assertExitCode(0);
+        ])->assertSuccessful();
 
         $this->assertFalse($missingLdapUser->fresh()->trashed());
 
@@ -193,7 +193,7 @@ public function test_missing_users_are_soft_deleted_when_flag_is_enabled()
             'provider' => 'ldap-database',
             '--no-interaction',
             '--delete-missing' => true,
-        ])->assertExitCode(0);
+        ])->assertSuccessful();
 
         Event::assertDispatched(DeletedMissing::class, function (DeletedMissing $event) use ($missingLdapUser) {
             return $event->deleted->count() == 1
@@ -236,7 +236,7 @@ public function test_existing_users_are_synchronized_when_enabled()
         $this->artisan('ldap:import', [
             'provider' => 'ldap-database',
             '--no-interaction',
-        ])->assertExitCode(0);
+        ])->assertSuccessful();
 
         $database->refresh();
 
@@ -274,7 +274,7 @@ public function test_can_use_array_syntax_for_sync_existing_configuration()
         $this->artisan('ldap:import', [
             'provider' => 'ldap-database',
             '--no-interaction',
-        ])->assertExitCode(0);
+        ])->assertSuccessful();
 
         $database->refresh();
 
@@ -308,7 +308,7 @@ public function test_existing_users_can_be_synchronized_using_raw_attributes()
         $this->artisan('ldap:import', [
             'provider' => 'ldap-database',
             '--no-interaction',
-        ])->assertExitCode(0);
+        ])->assertSuccessful();
 
         $database->refresh();
 
@@ -341,7 +341,7 @@ public function test_scopes_can_be_applied_to_command()
             'provider' => 'ldap-database',
             '--scopes' => TestImportUserModelScope::class,
             '--no-interaction',
-        ])->assertExitCode(0);
+        ])->assertSuccessful();
 
         $this->assertCount(1, $users = TestUserModelStub::get());
         $this->assertEquals('Bar', $users->first()->name);