Merge pull request #23 from rtm-ctrlz/feat-bind-param

feat(FakePdoStatement): implement `$pdoStatement->bindParam()` method

Author: muglug

src/FakePdoStatementTrait.php

@@ -1,6 +1,8 @@
 <?php
 namespace Vimeo\MysqlEngine;
 
+use PDO;
+
 trait FakePdoStatementTrait
 {
     /**
@@ -70,8 +72,9 @@ public function __construct(FakePdoInterface $conn, string $sql, ?\PDO $real)
      * @param string|int $key
      * @param scalar $value
      * @param int $type
+     * @return bool
      */
-    public function bindValue($key, $value, $type = \PDO::PARAM_STR) : void
+    public function bindValue($key, $value, $type = \PDO::PARAM_STR) : bool
     {
         if (\is_string($key) && $key[0] !== ':') {
             $key = ':' . $key;
@@ -83,8 +86,35 @@ public function bindValue($key, $value, $type = \PDO::PARAM_STR) : void
         $this->boundValues[$key] = $value;
 
         if ($this->realStatement) {
-            $this->realStatement->bindValue($key, $value, $type);
+            return $this->realStatement->bindValue($key, $value, $type);
         }
+        return true;
+    }
+
+    /**
+     * @param string|int $key
+     * @param scalar $value
+     * @param int $type
+     * @param int $maxLength
+     * @param mixed $driverOptions
+     * @return bool
+     */
+    public function bindParam($key, &$value, $type = PDO::PARAM_STR, $maxLength = null, $driverOptions = null): bool
+    {
+        if (\is_string($key) && $key[0] !== ':') {
+            $key = ':' . $key;
+        } elseif (\is_int($key)) {
+            // Parameter offsets start at 1, which is weird.
+            --$key;
+        }
+        $this->boundValues[$key] = &$value;
+        if ($this->realStatement) {
+            /**
+             * @psalm-suppress PossiblyNullArgument
+             */
+            return $this->realStatement->bindParam($key, $value, $type, $maxLength, $driverOptions);
+        }
+        return true;
     }
 
     /**

tests/ParameterBindTest.php

@@ -0,0 +1,63 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Vimeo\MysqlEngine\Tests;
+
+use PHPUnit\Framework\TestCase;
+
+class ParameterBindTest extends TestCase
+{
+
+    public function dataSqlWithParameter(): array
+    {
+        return [
+            'numeric' => ['SELECT ?', 1],
+            ':field'  => ['SELECT :field', ':field'],
+            'field'   => ['SELECT :field', 'field'],
+        ];
+    }
+
+    /**
+     * @param string     $sql
+     * @param string|int $key
+     * @dataProvider dataSqlWithParameter
+     */
+    public function testBindValue(string $sql, $key): void
+    {
+        $pdo = self::getPdo('mysql:foo;dbname=test;');
+
+        $statement = $pdo->prepare($sql);
+        $statement->bindValue($key, 999);
+        $statement->execute();
+
+        self::assertEquals(999, $statement->fetchColumn(0));
+    }
+
+    /**
+     * @param string     $sql
+     * @param string|int $key
+     * @dataProvider dataSqlWithParameter
+     */
+    public function testBindParam(string $sql, $key): void
+    {
+        $pdo = self::getPdo('mysql:foo;dbname=test;');
+
+        $var       = 1000;
+        $statement = $pdo->prepare($sql);
+        $statement->bindParam($key, $var);
+
+        $var = 10;
+        $statement->execute();
+        self::assertEquals(10, $statement->fetchColumn(0));
+    }
+
+    private static function getPdo(string $connection_string): \PDO
+    {
+        if (\PHP_MAJOR_VERSION === 8) {
+            return new \Vimeo\MysqlEngine\Php8\FakePdo($connection_string, '', '', []);
+        }
+
+        return new \Vimeo\MysqlEngine\Php7\FakePdo($connection_string, '', '', []);
+    }
+}