Provide constructor defaults

Signed-off-by: Joey Smith <jsmith@webinertia.net>

Author: tyrsson

phpstan-baseline.neon

@@ -1,2 +1,13 @@
 parameters:
 	ignoreErrors:
+		-
+			message: '#^Callable PhpDb\\Sqlite\\Container\\PdoStatementFactory invoked with 1 parameter, 2\-3 required\.$#'
+			identifier: arguments.count
+			count: 1
+			path: test/integration/Container/PdoStatementFactoryTest.php
+
+		-
+			message: '#^Callable PhpDb\\Sqlite\\Container\\PdoStatementFactory invoked with 1 parameter, 2\-3 required\.$#'
+			identifier: arguments.count
+			count: 3
+			path: test/unit/Container/PdoStatementFactoryTest.php

src/Container/PdoStatementFactory.php

@@ -10,17 +10,11 @@
 
 final class PdoStatementFactory
 {
-    public function __invoke(ContainerInterface $container): StatementInterface&Statement
-    {
-        /** @var array $config */
-        $config = $container->get('config');
-
-        /** @var array $dbConfig */
-        $dbConfig = $config['db'] ?? [];
-
-        /** @var array $options */
-        $options = $dbConfig['options'] ?? [];
-
+    public function __invoke(
+        ContainerInterface $container,
+        string $requestedName,
+        ?array $options = null
+    ): StatementInterface&Statement {
         return new Statement(options: $options);
     }
 }

src/Pdo/Connection.php

@@ -14,6 +14,7 @@
 
 use function array_diff_key;
 use function assert;
+use function is_array;
 use function is_int;
 use function is_string;
 use function str_starts_with;
@@ -23,6 +24,16 @@ class Connection extends AbstractPdoConnection
 {
     public final const CURRENT_SCHEMA = 'main';
 
+    public function __construct(
+        PDO|array $connectionParameters
+    ) {
+        if (is_array($connectionParameters)) {
+            $this->setConnectionParameters($connectionParameters);
+        } elseif ($connectionParameters instanceof PDO) {
+            $this->setResource($connectionParameters);
+        }
+    }
+
     /**
      * {@inheritDoc}
      */

src/Pdo/Driver.php

@@ -5,20 +5,46 @@
 namespace PhpDb\Sqlite\Pdo;
 
 use Override;
+use PDO;
 use PDOStatement;
 use PhpDb\Adapter\Driver\Feature\DriverFeatureProviderInterface;
 use PhpDb\Adapter\Driver\Feature\DriverFeatureProviderTrait;
 use PhpDb\Adapter\Driver\Pdo\AbstractPdo;
 use PhpDb\Adapter\Driver\Pdo\Result;
 use PhpDb\Adapter\Driver\Pdo\Statement;
+use PhpDb\Adapter\Driver\PdoConnectionInterface;
+use PhpDb\Adapter\Driver\PdoDriverAwareInterface;
 use PhpDb\Adapter\Driver\ResultInterface;
+use PhpDb\Adapter\Driver\StatementInterface;
 use PhpDb\Sqlite\DatabasePlatformNameTrait;
 
 class Driver extends AbstractPdo implements DriverFeatureProviderInterface
 {
     use DatabasePlatformNameTrait;
     use DriverFeatureProviderTrait;
 
+    public function __construct(
+        (PdoConnectionInterface&PdoDriverAwareInterface)|PDO $connection,
+        (StatementInterface&PdoDriverAwareInterface)|Statement $statementPrototype = new Statement(),
+        ResultInterface $resultPrototype = new Result(),
+        array $features = [],
+    ) {
+        $this->connection         = $connection;
+        $this->statementPrototype = $statementPrototype;
+        $this->resultPrototype    = $resultPrototype;
+
+        if (! $this->connection instanceof PDO) {
+            $this->connection->setDriver($this);
+        }
+
+        $this->statementPrototype->setDriver($this);
+
+        // $features is not constructor promoted because $this->features is defined in the trait
+        if ($features !== []) {
+            $this->addFeatures($features);
+        }
+    }
+
     /**
      * @param PDOStatement|resource $resource
      */

test/unit/Container/PdoStatementFactoryTest.php

@@ -4,6 +4,7 @@
 
 namespace PhpDbTest\Sqlite\Container;
 
+use PhpDb\Adapter\AdapterInterface;
 use PhpDb\Adapter\Driver\Pdo\Statement;
 use PhpDb\Sqlite\Container\PdoStatementFactory;
 use PHPUnit\Framework\Attributes\CoversClass;
@@ -19,7 +20,7 @@ public function testInvokeReturnsStatement(): void
         $containerMock->method('get')
             ->with('config')
             ->willReturn([
-                'db' => [
+                AdapterInterface::class => [
                     'options' => [],
                 ],
             ]);
@@ -36,7 +37,7 @@ public function testInvokeWithoutOptionsConfig(): void
         $containerMock->method('get')
             ->with('config')
             ->willReturn([
-                'db' => [],
+                AdapterInterface::class => [],
             ]);
 
         $factory   = new PdoStatementFactory();