Disabled and reworked some tests

lib/Doctrine/DBAL/Driver/PDOConnection.php

@@ -104,7 +104,11 @@ public function quote($input, $type = ParameterType::STRING)
  */
  public function lastInsertId($name = null)
  {
- return parent::lastInsertId($name);
+ try {
+ return parent::lastInsertId($name);
+ } catch (\PDOException $exception) {
+ throw new PDOException($exception);
+ }
  }
 
  /**

tests/Doctrine/Tests/DBAL/Functional/BlobTest.php

@@ -3,6 +3,7 @@
 namespace Doctrine\Tests\DBAL\Functional;
 
 use Doctrine\DBAL\Driver\OCI8\Driver as OCI8Driver;
+use Doctrine\DBAL\Driver\PDOOracle\Driver as PDOOracleDriver;
 use Doctrine\DBAL\FetchMode;
 use Doctrine\DBAL\ParameterType;
 use Doctrine\DBAL\Schema\Table;
@@ -21,6 +22,12 @@ protected function setUp()
  {
  parent::setUp();
 
+ if ($this->connection->getDriver() instanceof PDOOracleDriver) {
+ // inserting BLOBs as streams on Oracle requires Oracle-specific SQL syntax which is currently not supported
+ // see http://php.net/manual/en/pdo.lobs.php#example-1035
+ $this->markTestSkipped('DBAL doesn\'t support storing LOBs represented as streams using PDO_OCI');
+ }
+
  $table = new Table('blob_table');
  $table->addColumn('id', 'integer');
  $table->addColumn('clobfield', 'text');

tests/Doctrine/Tests/DBAL/Functional/DataAccessTest.php

@@ -5,6 +5,10 @@
 use DateTime;
 use Doctrine\DBAL\Connection;
 use Doctrine\DBAL\Driver\Mysqli\Driver as MySQLiDriver;
+use Doctrine\DBAL\Driver\OCI8\Driver as Oci8Driver;
+use Doctrine\DBAL\Driver\PDOConnection;
+use Doctrine\DBAL\Driver\PDOOracle\Driver as PDOOracleDriver;
+use Doctrine\DBAL\Driver\PDOSqlsrv\Driver as PDOSqlsrvDriver;
 use Doctrine\DBAL\Driver\SQLSrv\Driver as SQLSrvDriver;
 use Doctrine\DBAL\FetchMode;
 use Doctrine\DBAL\ParameterType;
@@ -14,6 +18,7 @@
 use Doctrine\DBAL\Statement;
 use Doctrine\DBAL\Types\Type;
 use Doctrine\Tests\DbalFunctionalTestCase;
+use PDO;
 use const CASE_LOWER;
 use const PHP_EOL;
 use function array_change_key_case;
@@ -749,7 +754,7 @@ public function testFetchAllStyleObject()
  */
  public function testFetchAllSupportFetchClass()
  {
- $this->skipOci8AndMysqli();
+ $this->beforeFetchClassTest();
  $this->setupFixture();
 
  $sql = 'SELECT test_int, test_string, test_datetime FROM fetch_table';
@@ -791,7 +796,7 @@ public function testFetchAllStyleColumn()
  */
  public function testSetFetchModeClassFetchAll()
  {
- $this->skipOci8AndMysqli();
+ $this->beforeFetchClassTest();
  $this->setupFixture();
 
  $sql = 'SELECT * FROM fetch_table';
@@ -813,7 +818,7 @@ public function testSetFetchModeClassFetchAll()
  */
  public function testSetFetchModeClassFetch()
  {
- $this->skipOci8AndMysqli();
+ $this->beforeFetchClassTest();
  $this->setupFixture();
 
  $sql = 'SELECT * FROM fetch_table';
@@ -908,16 +913,25 @@ private function setupFixture()
  ]);
  }
 
- private function skipOci8AndMysqli()
+ private function beforeFetchClassTest()
  {
- if (isset($GLOBALS['db_type']) && $GLOBALS['db_type'] === 'oci8') {
+ $driver = $this->connection->getDriver();
+
+ if ($driver instanceof Oci8Driver) {
  $this->markTestSkipped('Not supported by OCI8');
  }
- if ($this->connection->getDriver()->getName() !== 'mysqli') {
+
+ if ($driver instanceof MySQLiDriver) {
+ $this->markTestSkipped('Mysqli driver dont support this feature.');
+ }
+
+ if (! $driver instanceof PDOOracleDriver) {
  return;
  }
 
- $this->markTestSkipped('Mysqli driver dont support this feature.');
+ /** @var PDOConnection $connection */
+ $connection = $this->connection->getWrappedConnection();
+ $connection->setAttribute(PDO::ATTR_CASE, PDO::CASE_LOWER);
  }
 }
 

tests/Doctrine/Tests/DBAL/Functional/PDOStatementTest.php

@@ -24,7 +24,7 @@ protected function setUp()
 
  $table = new Table('stmt_test');
  $table->addColumn('id', 'integer');
- $table->addColumn('name', 'text');
+ $table->addColumn('name', 'string');
  $this->connection->getSchemaManager()->dropAndCreateTable($table);
  }
 

tests/Doctrine/Tests/DBAL/Functional/StatementTest.php

@@ -2,6 +2,7 @@
 
 namespace Doctrine\Tests\DBAL\Functional;
 
+use Doctrine\DBAL\Driver\PDOOracle\Driver as PDOOracleDriver;
 use Doctrine\DBAL\Driver\Statement;
 use Doctrine\DBAL\FetchMode;
 use Doctrine\DBAL\ParameterType;
@@ -25,6 +26,10 @@ protected function setUp()
 
  public function testStatementIsReusableAfterClosingCursor()
  {
+ if ($this->connection->getDriver() instanceof PDOOracleDriver) {
+ $this->markTestIncomplete('See https://bugs.php.net/bug.php?id=77181');
+ }
+
  $this->connection->insert('stmt_test', ['id' => 1]);
  $this->connection->insert('stmt_test', ['id' => 2]);
 
@@ -46,6 +51,10 @@ public function testStatementIsReusableAfterClosingCursor()
 
  public function testReuseStatementWithLongerResults()
  {
+ if ($this->connection->getDriver() instanceof PDOOracleDriver) {
+ $this->markTestIncomplete('PDO_OCI doesn\'t support fetching blobs via PDOStatement::fetchAll()');
+ }
+
  $sm = $this->connection->getSchemaManager();
  $table = new Table('stmt_longer_results');
  $table->addColumn('param', 'string');
@@ -79,6 +88,12 @@ public function testReuseStatementWithLongerResults()
 
  public function testFetchLongBlob()
  {
+ if ($this->connection->getDriver() instanceof PDOOracleDriver) {
+ // inserting BLOBs as streams on Oracle requires Oracle-specific SQL syntax which is currently not supported
+ // see http://php.net/manual/en/pdo.lobs.php#example-1035
+ $this->markTestSkipped('DBAL doesn\'t support storing LOBs represented as streams using PDO_OCI');
+ }
+
  // make sure memory limit is large enough to not cause false positives,
  // but is still not enough to store a LONGBLOB of the max possible size
  $this->iniSet('memory_limit', '4G');
@@ -138,6 +153,10 @@ public function testIncompletelyFetchedStatementDoesNotBlockConnection()
 
  public function testReuseStatementAfterClosingCursor()
  {
+ if ($this->connection->getDriver() instanceof PDOOracleDriver) {
+ $this->markTestIncomplete('See https://bugs.php.net/bug.php?id=77181');
+ }
+
  $this->connection->insert('stmt_test', ['id' => 1]);
  $this->connection->insert('stmt_test', ['id' => 2]);
 

tests/Doctrine/Tests/DBAL/Functional/TypeConversionTest.php

@@ -3,6 +3,7 @@
 namespace Doctrine\Tests\DBAL\Functional;
 
 use DateTime;
+use Doctrine\DBAL\Driver\PDOOracle\Driver as PDOOracleDriver;
 use Doctrine\DBAL\Schema\Table;
 use Doctrine\DBAL\Types\Type;
 use Doctrine\Tests\DbalFunctionalTestCase;
@@ -77,6 +78,12 @@ public static function dataIdempotentDataConversion()
  */
  public function testIdempotentDataConversion($type, $originalValue, $expectedPhpType)
  {
+ if ($type === 'text' && $this->connection->getDriver() instanceof PDOOracleDriver) {
+ // inserting BLOBs as streams on Oracle requires Oracle-specific SQL syntax which is currently not supported
+ // see http://php.net/manual/en/pdo.lobs.php#example-1035
+ $this->markTestSkipped('DBAL doesn\'t support storing LOBs represented as streams using PDO_OCI');
+ }
+
  $columnName = 'test_' . $type;
  $typeInstance = Type::getType($type);
  $insertionValue = $typeInstance->convertToDatabaseValue($originalValue, $this->connection->getDatabasePlatform());

tests/Doctrine/Tests/DBAL/Functional/Types/BinaryTest.php

@@ -5,6 +5,7 @@
 namespace Doctrine\Tests\DBAL\Functional\Types;
 
 use Doctrine\DBAL\Driver\IBMDB2\DB2Driver;
+use Doctrine\DBAL\Driver\PDOOracle\Driver as PDOOracleDriver;
 use Doctrine\DBAL\ParameterType;
 use Doctrine\DBAL\Schema\Table;
 use Doctrine\Tests\DbalFunctionalTestCase;
@@ -19,6 +20,10 @@ protected function setUp()
  {
  parent::setUp();
 
+ if ($this->connection->getDriver() instanceof PDOOracleDriver) {
+ $this->markTestSkipped('PDO_OCI doesn\'t support binding binary values');
+ }
+
  $table = new Table('binary_table');
  $table->addColumn('id', 'binary', [
  'length' => 16,

tests/Doctrine/Tests/DBAL/Functional/WriteTest.php

@@ -3,6 +3,7 @@
 namespace Doctrine\Tests\DBAL\Functional;
 
 use DateTime;
+use Doctrine\DBAL\Driver\DriverException;
 use Doctrine\DBAL\ParameterType;
 use Doctrine\DBAL\Schema\Sequence;
 use Doctrine\DBAL\Schema\Table;
@@ -149,7 +150,7 @@ public function testLastInsertId()
  }
 
  self::assertEquals(1, $this->connection->insert('write_table', ['test_int' => 2, 'test_string' => 'bar']));
- $num = $this->connection->lastInsertId();
+ $num = $this->lastInsertId();
 
  self::assertNotNull($num, 'LastInsertId() should not be null.');
  self::assertGreaterThan(0, $num, 'LastInsertId() should be non-negative number.');
@@ -175,7 +176,7 @@ public function testLastInsertIdSequence()
  $stmt = $this->connection->query($this->connection->getDatabasePlatform()->getSequenceNextValSQL('write_table_id_seq'));
  $nextSequenceVal = $stmt->fetchColumn();
 
- $lastInsertId = $this->connection->lastInsertId('write_table_id_seq');
+ $lastInsertId = $this->lastInsertId('write_table_id_seq');
 
  self::assertGreaterThan(0, $lastInsertId);
  self::assertEquals($nextSequenceVal, $lastInsertId);
@@ -187,7 +188,7 @@ public function testLastInsertIdNoSequenceGiven()
  $this->markTestSkipped("Test only works consistently on platforms that support sequences and don't support identity columns.");
  }
 
- self::assertFalse($this->connection->lastInsertId(null));
+ self::assertFalse($this->lastInsertId());
  }
 
  /**
@@ -285,11 +286,11 @@ public function testEmptyIdentityInsert()
 
  $this->connection->exec($sql);
 
- $firstId = $this->connection->lastInsertId($seqName);
+ $firstId = $this->lastInsertId($seqName);
 
  $this->connection->exec($sql);
 
- $secondId = $this->connection->lastInsertId($seqName);
+ $secondId = $this->lastInsertId($seqName);
 
  self::assertGreaterThan($firstId, $secondId);
  }
@@ -334,4 +335,25 @@ public function testDeleteWhereIsNull()
 
  self::assertCount(0, $data);
  }
+
+ /**
+ * Returns the ID of the last inserted row or skips the test if the currently used driver
+ * doesn't support this feature
+ *
+ * @return string
+ *
+ * @throws DriverException
+ */
+ private function lastInsertId(?string $name = null)
+ {
+ try {
+ return $this->connection->lastInsertId($name);
+ } catch (DriverException $e) {
+ if ($e->getCode() === 'IM001') {
+ $this->markTestSkipped($e->getMessage());
+ }
+
+ throw $e;
+ }
+ }
 }