Fix bigint PHP_INT_MIN/PHP_INT_MAX string to int convert

psalm.xml.dist

@@ -131,6 +131,12 @@
                 <file name="tests/Types/DateIntervalTest.php"/>
             </errorLevel>
         </InaccessibleProperty>
+        <InvalidCast>
+            <errorLevel type="suppress">
+                <!-- See https://github.com/vimeo/psalm/issues/10995 -->
+                <file name="src/Types/BigIntType.php"/>
+            </errorLevel>
+        </InvalidCast>
         <InvalidArgument>
             <errorLevel type="suppress">
                 <!-- We're testing with invalid input here. -->
@@ -185,6 +191,8 @@
         </MoreSpecificReturnType>
         <NoValue>
             <errorLevel type="suppress">
+                <!-- See https://github.com/vimeo/psalm/issues/10995 -->
+                <file name="src/Types/BigIntType.php"/>
                 <!--
                     This error looks bogus.
                 -->
@@ -291,6 +299,8 @@
                 <!-- Ignore isset() checks in destructors. -->
                 <file name="src/Driver/PgSQL/Connection.php"/>
                 <file name="src/Driver/PgSQL/Statement.php"/>
+                <!-- See https://github.com/vimeo/psalm/issues/10995 -->
+                <file name="src/Types/BigIntType.php"/>
             </errorLevel>
         </TypeDoesNotContainType>
         <UndefinedClass>

src/Types/BigIntType.php

@@ -47,15 +47,18 @@ public function convertToPHPValue(mixed $value, AbstractPlatform $platform): int
             return $value;
         }
 
-        if ($value > PHP_INT_MIN && $value < PHP_INT_MAX) {
-            return (int) $value;
-        }
-
         assert(
             is_string($value),
             'DBAL assumes values outside of the integer range to be returned as string by the database driver.',
         );
 
+        if (
+            ($value > PHP_INT_MIN && $value < PHP_INT_MAX)
+            || $value === (string) (int) $value
+        ) {
+            return (int) $value;
+        }
+
         return $value;
     }
 }

tests/Functional/Types/BigIntTypeTest.php

@@ -10,8 +10,6 @@
 use Doctrine\DBAL\Types\Types;
 use Generator;
 use PHPUnit\Framework\Attributes\DataProvider;
-use PHPUnit\Framework\Constraint\IsIdentical;
-use PHPUnit\Framework\Constraint\LogicalOr;
 
 use const PHP_INT_MAX;
 use const PHP_INT_MIN;
@@ -30,13 +28,13 @@ public function testSelectBigInt(string $sqlLiteral, int|string|null $expectedVa
 
         $this->connection->executeStatement(<<<SQL
             INSERT INTO bigint_type_test (id, my_integer)
-            VALUES (42, $sqlLiteral)
+            VALUES (1, $sqlLiteral)
             SQL);
 
         self::assertSame(
             $expectedValue,
             $this->connection->convertToPHPValue(
-                $this->connection->fetchOne('SELECT my_integer from bigint_type_test WHERE id = 42'),
+                $this->connection->fetchOne('SELECT my_integer from bigint_type_test'),
                 Types::BIGINT,
             ),
         );
@@ -49,44 +47,10 @@ public static function provideBigIntLiterals(): Generator
         yield 'null' => ['null', null];
         yield 'positive number' => ['42', 42];
         yield 'negative number' => ['-42', -42];
-
-        if (PHP_INT_SIZE < 8) {
-            // The following tests only work on 64bit systems.
-            return;
-        }
-
-        yield 'large positive number' => ['9223372036854775806', PHP_INT_MAX - 1];
-        yield 'large negative number' => ['-9223372036854775807', PHP_INT_MIN + 1];
-    }
-
-    #[DataProvider('provideBigIntEdgeLiterals')]
-    public function testSelectBigIntEdge(int $value): void
-    {
-        $table = new Table('bigint_type_test');
-        $table->addColumn('id', Types::SMALLINT, ['notnull' => true]);
-        $table->addColumn('my_integer', Types::BIGINT, ['notnull' => false]);
-        $table->setPrimaryKey(['id']);
-        $this->dropAndCreateTable($table);
-
-        $this->connection->executeStatement(<<<SQL
-            INSERT INTO bigint_type_test (id, my_integer)
-            VALUES (42, $value)
-            SQL);
-
-        self::assertThat(
-            $this->connection->convertToPHPValue(
-                $this->connection->fetchOne('SELECT my_integer from bigint_type_test WHERE id = 42'),
-                Types::BIGINT,
-            ),
-            LogicalOr::fromConstraints(new IsIdentical($value), new IsIdentical((string) $value)),
-        );
-    }
-
-    /** @return Generator<string, array{int}> */
-    public static function provideBigIntEdgeLiterals(): Generator
-    {
-        yield 'max int' => [PHP_INT_MAX];
-        yield 'min int' => [PHP_INT_MIN];
+        yield 'large positive number' => [PHP_INT_SIZE === 4 ? '2147483646' : '9223372036854775806', PHP_INT_MAX - 1];
+        yield 'large negative number' => [PHP_INT_SIZE === 4 ? '-2147483647' : '-9223372036854775807', PHP_INT_MIN + 1];
+        yield 'largest positive number' => [PHP_INT_SIZE === 4 ? '2147483647' : '9223372036854775807', PHP_INT_MAX];
+        yield 'largest negative number' => [PHP_INT_SIZE === 4 ? '-2147483648' : '-9223372036854775808', PHP_INT_MIN];
     }
 
     public function testUnsignedBigIntOnMySQL(): void
@@ -104,13 +68,13 @@ public function testUnsignedBigIntOnMySQL(): void
         // Insert (2 ** 64) - 1
         $this->connection->executeStatement(<<<'SQL'
             INSERT INTO bigint_type_test (id, my_integer)
-            VALUES (42, 0xFFFFFFFFFFFFFFFF)
+            VALUES (1, 0xFFFFFFFFFFFFFFFF)
             SQL);
 
         self::assertSame(
             '18446744073709551615',
             $this->connection->convertToPHPValue(
-                $this->connection->fetchOne('SELECT my_integer from bigint_type_test WHERE id = 42'),
+                $this->connection->fetchOne('SELECT my_integer from bigint_type_test'),
                 Types::BIGINT,
             ),
         );