refactor: clean up code style using pint

Author: aspirantzhang

contexts/Authorization/Application/Coordinators/RoleCoordinator.php

@@ -21,8 +21,7 @@ class RoleCoordinator extends BaseCoordinator
     public function __construct(
         private RoleRepository $repository,
         private RoleFactory $factory
-    ) {
-    }
+    ) {}
 
     public function create(CreateRoleDTO $data): Role
     {

contexts/Authorization/Application/Coordinators/UserIdentityCoordinator.php

@@ -25,8 +25,7 @@ class UserIdentityCoordinator extends BaseCoordinator
     public function __construct(
         private UserRepository $repository,
         private UserIdentityFactory $factory
-    ) {
-    }
+    ) {}
 
     public function create(CreateUserDTO $data): UserIdentity
     {

contexts/Authorization/Domain/Factories/RoleFactory.php

@@ -15,8 +15,7 @@ class RoleFactory
 {
     public function __construct(
         private readonly RoleLabelUniquenessService $roleLabelUniquenessService
-    ) {
-    }
+    ) {}
 
     public function create(
         RoleId $id,

contexts/Authorization/Domain/Factories/UserIdentityFactory.php

@@ -4,21 +4,20 @@
 
 namespace Contexts\Authorization\Domain\Factories;
 
+use Carbon\CarbonImmutable;
 use Contexts\Authorization\Domain\Services\UserEmailUniquenessService;
-use Contexts\Authorization\Domain\UserIdentity\Models\UserIdentity;
-use Contexts\Authorization\Domain\UserIdentity\Models\UserId;
+use Contexts\Authorization\Domain\UserIdentity\Events\UserCreatedEvent;
 use Contexts\Authorization\Domain\UserIdentity\Models\Email;
 use Contexts\Authorization\Domain\UserIdentity\Models\Password;
-use Carbon\CarbonImmutable;
+use Contexts\Authorization\Domain\UserIdentity\Models\UserId;
+use Contexts\Authorization\Domain\UserIdentity\Models\UserIdentity;
 use Contexts\Authorization\Domain\UserIdentity\Models\UserStatus;
-use Contexts\Authorization\Domain\UserIdentity\Events\UserCreatedEvent;
 
 class UserIdentityFactory
 {
     public function __construct(
         private readonly UserEmailUniquenessService $userEmailUniquenessService
-    ) {
-    }
+    ) {}
 
     public function create(
         UserId $id,

contexts/Authorization/Domain/Services/UserEmailUniquenessService.php

@@ -11,8 +11,7 @@ class UserEmailUniquenessService
 {
     public function __construct(
         private readonly UserRepository $userRepository
-    ) {
-    }
+    ) {}
 
     public function ensureUnique(string $email)
     {

contexts/Authorization/Domain/UserIdentity/Models/UserIdentity.php

@@ -26,7 +26,7 @@ private function __construct(
         private ?CarbonImmutable $updated_at = null
     ) {
         $this->created_at = $created_at ?? CarbonImmutable::now();
-        $this->roleIdCollection = new RoleIdCollection();
+        $this->roleIdCollection = new RoleIdCollection;
     }
 
     public function hasAnyRole(RoleIdCollection $roleIds): bool
@@ -106,7 +106,7 @@ public static function reconstitute(
         array $events = []
     ): self {
         $user = new self($id, $email, $password, $display_name, $status, $created_at, $updated_at);
-        $user->roleIdCollection = $roleIdCollection ?? new RoleIdCollection();
+        $user->roleIdCollection = $roleIdCollection ?? new RoleIdCollection;
 
         foreach ($events as $event) {
             $user->recordEvent($event);

contexts/Authorization/Tests/Feature/Application/GlobalPermissionServiceCoordinatorTest.php

@@ -4,16 +4,16 @@
 
 use Contexts\Authorization\Application\Coordinators\GlobalPermissionServiceCoordinator;
 use Contexts\Authorization\Contracts\V1\Services\GlobalPermissionService;
+use Contexts\Authorization\Domain\Factories\RoleFactory;
 use Contexts\Authorization\Domain\Policies\RolePolicy;
 use Contexts\Authorization\Domain\Role\Models\Role;
 use Contexts\Authorization\Domain\Role\Models\RoleId;
+use Contexts\Authorization\Domain\Services\RoleLabelUniquenessService;
 use Contexts\Authorization\Domain\UserIdentity\Models\RoleIdCollection;
 use Contexts\Authorization\Infrastructure\Persistence\RolePersistence;
 use Contexts\Authorization\Infrastructure\Persistence\UserPersistence;
 use Contexts\Authorization\Infrastructure\Records\UserRecord;
 use Illuminate\Support\Facades\Config;
-use Contexts\Authorization\Domain\Services\RoleLabelUniquenessService;
-use Contexts\Authorization\Domain\Factories\RoleFactory;
 
 beforeEach(function () {
     Config::set('policies.article_publishing', [
@@ -53,8 +53,8 @@
 
 it('can check permission for admin user', function () {
     // Setup repositories
-    $userPersistence = new UserPersistence();
-    $rolePersistence = new RolePersistence();
+    $userPersistence = new UserPersistence;
+    $rolePersistence = new RolePersistence;
 
     // Create admin user
     $userRecord = UserRecord::factory()->create();
@@ -76,8 +76,8 @@
 
 it('denies permission for users without required roles', function () {
     // Setup repositories
-    $userPersistence = new UserPersistence();
-    $rolePersistence = new RolePersistence();
+    $userPersistence = new UserPersistence;
+    $rolePersistence = new RolePersistence;
 
     // Create regular user
     $userRecord = UserRecord::factory()->create();

contexts/Authorization/Tests/Feature/Infrastructure/Persistence/RolePersistenceTest.php

@@ -2,14 +2,14 @@
 
 declare(strict_types=1);
 use Carbon\CarbonImmutable;
+use Contexts\Authorization\Domain\Factories\RoleFactory;
 use Contexts\Authorization\Domain\Role\Exceptions\RoleNotFoundException;
 use Contexts\Authorization\Domain\Role\Models\Role;
 use Contexts\Authorization\Domain\Role\Models\RoleId;
 use Contexts\Authorization\Domain\Role\Models\RoleStatus;
+use Contexts\Authorization\Domain\Services\RoleLabelUniquenessService;
 use Contexts\Authorization\Infrastructure\Persistence\RolePersistence;
 use Contexts\Authorization\Infrastructure\Records\RoleRecord;
-use Contexts\Authorization\Domain\Factories\RoleFactory;
-use Contexts\Authorization\Domain\Services\RoleLabelUniquenessService;
 
 beforeEach(function () {
     $this->roleLabelUniquenessService = mock(RoleLabelUniquenessService::class);
@@ -19,7 +19,7 @@
 
 it('can persist role data correctly', function () {
     $role = $this->roleFactory->create(RoleId::null(), 'My Role');
-    $rolePersistence = new RolePersistence();
+    $rolePersistence = new RolePersistence;
 
     $rolePersistence->create($role);
 
@@ -32,7 +32,7 @@
 it('can retrieve an role by ID', function () {
     // Create a test role in the database
     $createdRole = $this->roleFactory->create(RoleId::null(), 'Test Role');
-    $rolePersistence = new RolePersistence();
+    $rolePersistence = new RolePersistence;
     $savedRole = $rolePersistence->create($createdRole);
 
     // Retrieve the role using getById
@@ -46,7 +46,7 @@
 
 it('can retrieve multiple roles by IDs', function () {
     // Create multiple test roles in the database
-    $rolePersistence = new RolePersistence();
+    $rolePersistence = new RolePersistence;
 
     $role1 = $this->roleFactory->create(RoleId::null(), 'Role 1');
     $role2 = $this->roleFactory->create(RoleId::null(), 'Role 2');
@@ -71,7 +71,7 @@
 });
 
 it('throws an exception when retrieving a non-existent role', function () {
-    $rolePersistence = new RolePersistence();
+    $rolePersistence = new RolePersistence;
 
     // Attempt to retrieve a non-existent role
     $rolePersistence->getById(RoleId::fromInt(999));
@@ -80,7 +80,7 @@
 it('can update an role', function () {
     // Create a test role in the database
     $createdRole = $this->roleFactory->create(RoleId::null(), 'Original Label');
-    $rolePersistence = new RolePersistence();
+    $rolePersistence = new RolePersistence;
     $savedRole = $rolePersistence->create($createdRole);
 
     // Create an updated version of the role
@@ -105,22 +105,22 @@
 });
 
 it('throws an exception when updating a non-existent role', function () {
-    $rolePersistence = new RolePersistence();
+    $rolePersistence = new RolePersistence;
 
     // Attempt to update a non-existent role
     $rolePersistence->update($this->roleFactory->create(RoleId::fromInt(999), 'Updated Label'));
 })->throws(RoleNotFoundException::class);
 
 it('can paginate roles', function () {
     // Create multiple test roles
-    $rolePersistence = new RolePersistence();
+    $rolePersistence = new RolePersistence;
 
     // Create 5 roles
     for ($i = 1; $i <= 5; $i++) {
         $role = $this->roleFactory->create(
             RoleId::null(),
             "Role $i",
-            new CarbonImmutable()
+            new CarbonImmutable
         );
         $rolePersistence->create($role);
     }
@@ -146,28 +146,28 @@
 });
 
 it('can filter roles with search criteria', function () {
-    $rolePersistence = new RolePersistence();
+    $rolePersistence = new RolePersistence;
 
     // Create roles with specific labels
     $role1 = $this->roleFactory->create(
         RoleId::null(),
         'Laravel Role',
-        new CarbonImmutable()
+        new CarbonImmutable
     );
     $rolePersistence->create($role1);
 
     $role2 = $this->roleFactory->create(
         RoleId::null(),
         'PHP Tutorial',
-        new CarbonImmutable()
+        new CarbonImmutable
     );
     $role2->subspend();
     $rolePersistence->create($role2);
 
     $role3 = $this->roleFactory->create(
         RoleId::null(),
         'Laravel Tips',
-        new CarbonImmutable()
+        new CarbonImmutable
     );
     $role3->subspend();
     $rolePersistence->create($role3);
@@ -205,7 +205,7 @@
 it('can delete an role', function () {
     // Create a test role in the database
     $createdRole = $this->roleFactory->create(RoleId::null(), 'Test Role');
-    $rolePersistence = new RolePersistence();
+    $rolePersistence = new RolePersistence;
     $savedRole = $rolePersistence->create($createdRole);
 
     // Delete the role
@@ -218,17 +218,16 @@
 });
 
 it('throws an exception when deleting a non-existent role', function () {
-    $rolePersistence = new RolePersistence();
+    $rolePersistence = new RolePersistence;
 
     // Attempt to delete a non-existent role
     $rolePersistence->delete($this->roleFactory->create(RoleId::fromInt(999), 'Test Role'));
 })->throws(RoleNotFoundException::class);
 
-
 it('returns true when role exists with given label', function () {
     // Create a test role in the database with a specific label
     $createdRole = $this->roleFactory->create(RoleId::null(), 'Unique Label');
-    $rolePersistence = new RolePersistence();
+    $rolePersistence = new RolePersistence;
     $rolePersistence->create($createdRole);
 
     // Check if the method correctly detects the existing label
@@ -238,7 +237,7 @@
 });
 
 it('returns false when role does not exist with given label', function () {
-    $rolePersistence = new RolePersistence();
+    $rolePersistence = new RolePersistence;
 
     // Test with a label that doesn't exist in the database
     $result = $rolePersistence->existsByLabel('Non-Existent Label');
@@ -248,7 +247,7 @@
 
 it('can retrieve roles by labels', function () {
     // Create roles with specific labels
-    $rolePersistence = new RolePersistence();
+    $rolePersistence = new RolePersistence;
 
     $role1 = $this->roleFactory->create(RoleId::null(), 'Admin Role');
     $role2 = $this->roleFactory->create(RoleId::null(), 'Editor Role');
@@ -270,7 +269,7 @@
 });
 
 it('returns empty collection when no roles match the given labels', function () {
-    $rolePersistence = new RolePersistence();
+    $rolePersistence = new RolePersistence;
 
     // Retrieve with non-existent labels
     $retrievedRoles = $rolePersistence->getByLabels(['Non-Existent Label']);