Introduced type checks for SqlPlatform for code safety

Signed-off-by: Simon Mundy <simon.mundy@peptolab.com>

Author: simon-mundy

src/Sql/Platform/AbstractPlatform.php

@@ -83,7 +83,7 @@ public function getSqlString(?PlatformInterface $adapterPlatform = null): string
         if (! $this->subject instanceof SqlInterface) {
             throw new Exception\RuntimeException(
                 'The subject does not appear to implement PhpDb\Sql\SqlInterface, thus calling '
-                . 'prepareStatement() has no effect'
+                . 'getSqlString() has no effect'
             );
         }
 

src/Sql/Platform/PlatformDecoratorInterface.php

@@ -7,7 +7,7 @@
 use PhpDb\Sql\PreparableSqlInterface;
 use PhpDb\Sql\SqlInterface;
 
-interface PlatformDecoratorInterface extends PreparableSqlInterface, SqlInterface
+interface PlatformDecoratorInterface
 {
     public function setSubject(
         SqlInterface|PreparableSqlInterface|null $subject

src/Sql/Sql.php

@@ -109,10 +109,17 @@ public function prepareStatementForSqlObject(
         ?StatementInterface $statement = null,
         ?AdapterInterface $adapter = null
     ): StatementInterface {
+        if (! $this->sqlPlatform instanceof PreparableSqlInterface) {
+            throw new Exception\RuntimeException(
+                'The subject does not implement PreparableSqlInterface'
+            );
+        }
+
         $adapter   ??= $this->adapter;
         $statement ??= $adapter->getDriver()->createStatement();
 
-        $this->sqlPlatform->setSubject($sqlObject)->prepareStatement($adapter, $statement);
+        $this->sqlPlatform->setSubject($sqlObject);
+        $this->sqlPlatform->prepareStatement($adapter, $statement);
 
         return $statement;
     }
@@ -122,11 +129,16 @@ public function prepareStatementForSqlObject(
      */
     public function buildSqlString(SqlInterface $sqlObject, ?AdapterInterface $adapter = null): string
     {
-        return $this
-            ->sqlPlatform
-            ->setSubject($sqlObject)
-            ->getSqlString(
-                $adapter instanceof AdapterInterface ? $adapter->getPlatform() : $this->adapter->getPlatform()
+        if (! $this->sqlPlatform instanceof SqlInterface) {
+            throw new Exception\RuntimeException(
+                'The subject does not implement SqlInterface'
             );
+        }
+
+        $this->sqlPlatform->setSubject($sqlObject);
+
+        return $this->sqlPlatform->getSqlString(
+            $adapter instanceof AdapterInterface ? $adapter->getPlatform() : $this->adapter->getPlatform()
+        );
     }
 }

test/unit/RowGateway/AbstractRowGatewayTest.php

@@ -10,14 +10,14 @@
 use PhpDb\Adapter\Driver\DriverInterface;
 use PhpDb\Adapter\Driver\ResultInterface;
 use PhpDb\Adapter\Driver\StatementInterface;
-use PhpDb\Adapter\Platform\PlatformInterface;
 use PhpDb\RowGateway\AbstractRowGateway;
 use PhpDb\RowGateway\Exception\InvalidArgumentException;
 use PhpDb\RowGateway\Exception\RuntimeException;
 use PhpDb\RowGateway\Feature\FeatureSet;
 use PhpDb\RowGateway\RowGateway;
 use PhpDb\Sql\Select;
 use PhpDb\Sql\Sql;
+use PhpDbTest\TestAsset\TrustingSql92Platform;
 use PHPUnit\Framework\Attributes\CoversMethod;
 use PHPUnit\Framework\Attributes\IgnoreDeprecations;
 use PHPUnit\Framework\Attributes\RequiresPhp;
@@ -77,7 +77,7 @@ protected function setUp(): void
             ->setConstructorArgs(
                 [
                     $mockDriver,
-                    $this->getMockBuilder(PlatformInterface::class)->getMock(),
+                    new TrustingSql92Platform(),
                 ]
             )->getMock();