Rewrite logic to two keyed lock

Author: antonkomarev

src/LockId/PostgresLockId.php

@@ -17,18 +17,17 @@
 
 final class PostgresLockId
 {
-    private const DB_INT64_VALUE_MIN = -9_223_372_036_854_775_808;
-    private const DB_INT64_VALUE_MAX = 9_223_372_036_854_775_807;
+    private const DB_INT32_VALUE_MIN = -2_147_483_648;
     private const DB_INT32_VALUE_MAX = 2_147_483_647;
 
     public function __construct(
         public readonly int $id,
         public readonly string $humanReadableValue = '',
     ) {
-        if ($id < self::DB_INT64_VALUE_MIN) {
+        if ($id < self::DB_INT32_VALUE_MIN) {
             throw new InvalidArgumentException('Out of bound exception (id is too small)');
         }
-        if ($id > self::DB_INT64_VALUE_MAX) {
+        if ($id > self::DB_INT32_VALUE_MAX) {
             throw new InvalidArgumentException('Out of bound exception (id is too big)');
         }
     }

test/Integration/Locker/PostgresAdvisoryLockerTest.php

@@ -21,8 +21,8 @@
 
 final class PostgresAdvisoryLockerTest extends AbstractIntegrationTestCase
 {
-    private const DB_INT64_VALUE_MIN = 0;
-    private const DB_INT64_VALUE_MAX = 9223372036854775807;
+    private const DB_INT32_VALUE_MIN = -2_147_483_648;
+    private const DB_INT32_VALUE_MAX = 2_147_483_647;
 
     public function test_it_can_acquire_lock(): void
     {
@@ -41,7 +41,7 @@ public function test_it_can_acquire_lock_with_smallest_lock_id(): void
     {
         $locker = $this->initLocker();
         $dbConnection = $this->initPostgresPdoConnection();
-        $postgresLockId = new PostgresLockId(self::DB_INT64_VALUE_MIN);
+        $postgresLockId = new PostgresLockId(self::DB_INT32_VALUE_MIN);
 
         $isLockAcquired = $locker->tryAcquireLock($dbConnection, $postgresLockId);
 
@@ -54,7 +54,7 @@ public function test_it_can_acquire_lock_with_biggest_lock_id(): void
     {
         $locker = $this->initLocker();
         $dbConnection = $this->initPostgresPdoConnection();
-        $postgresLockId = new PostgresLockId(self::DB_INT64_VALUE_MAX);
+        $postgresLockId = new PostgresLockId(self::DB_INT32_VALUE_MAX);
 
         $isLockAcquired = $locker->tryAcquireLock($dbConnection, $postgresLockId);
 

test/Unit/LockId/PostgresLockIdTest.php

@@ -19,21 +19,21 @@
 
 final class PostgresLockIdTest extends AbstractUnitTestCase
 {
-    private const DB_INT64_VALUE_MIN = 0;
-    private const DB_INT64_VALUE_MAX = 9223372036854775807;
+    private const DB_INT32_VALUE_MIN = -2_147_483_648;
+    private const DB_INT32_VALUE_MAX = 2_147_483_647;
 
     public function test_it_can_create_postgres_lock_id_with_min_id(): void
     {
-        $lockId = new PostgresLockId(self::DB_INT64_VALUE_MIN);
+        $lockId = new PostgresLockId(self::DB_INT32_VALUE_MIN);
 
-        $this->assertSame(self::DB_INT64_VALUE_MIN, $lockId->id);
+        $this->assertSame(self::DB_INT32_VALUE_MIN, $lockId->id);
     }
 
     public function test_it_can_create_postgres_lock_id_with_max_id(): void
     {
-        $lockId = new PostgresLockId(self::DB_INT64_VALUE_MAX);
+        $lockId = new PostgresLockId(self::DB_INT32_VALUE_MAX);
 
-        $this->assertSame(self::DB_INT64_VALUE_MAX, $lockId->id);
+        $this->assertSame(self::DB_INT32_VALUE_MAX, $lockId->id);
     }
 
     public function test_it_can_create_postgres_lock_id_from_lock_id(): void