-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
Showing
1 changed file
with
100 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,100 @@ | ||
<?php | ||
|
||
declare(strict_types=1); | ||
|
||
namespace Doctrine\Tests\DBAL\Functional; | ||
|
||
use Doctrine\DBAL\Connection; | ||
use Doctrine\DBAL\Driver\OCI8; | ||
use Doctrine\DBAL\Exception; | ||
use Doctrine\DBAL\LockMode; | ||
use Doctrine\DBAL\Platforms\SqlitePlatform; | ||
use Doctrine\DBAL\Platforms\SQLServerPlatform; | ||
use Doctrine\DBAL\Schema\Table; | ||
use Doctrine\DBAL\TransactionIsolationLevel; | ||
use Doctrine\Tests\DbalFunctionalTestCase; | ||
use Doctrine\Tests\TestUtil; | ||
|
||
class LockModeTest extends DbalFunctionalTestCase | ||
{ | ||
/** @var Connection */ | ||
private $connection2; | ||
|
||
public function setUp(): void | ||
{ | ||
parent::setUp(); | ||
|
||
if ($this->connection->getDriver() instanceof OCI8\Driver) { | ||
// https://github.com/doctrine/dbal/issues/4417 | ||
self::markTestSkipped('This test fails on OCI8 for a currently unknown reason'); | ||
} | ||
|
||
if ($this->connection->getDatabasePlatform() instanceof SQLServerPlatform) { | ||
// Use row versioning instead of locking on SQL Server (if we don't, the second connection will block when | ||
// attempting to read the row created by the first connection, instead of reading the previous version); | ||
// for some reason we cannot set READ_COMMITTED_SNAPSHOT ON when not running this test in isolation, | ||
// there may be another connection active at this point; temporarily forcing to SINGLE_USER does the trick. | ||
$name = $this->connection->fetchOne('SELECT db_name()'); | ||
$this->connection->executeStatement('ALTER DATABASE ' . $name . ' SET SINGLE_USER WITH ROLLBACK IMMEDIATE'); | ||
$this->connection->executeStatement('ALTER DATABASE ' . $name . ' SET READ_COMMITTED_SNAPSHOT ON'); | ||
$this->connection->executeStatement('ALTER DATABASE ' . $name . ' SET MULTI_USER'); | ||
} | ||
|
||
$table = new Table('users'); | ||
$table->addColumn('id', 'integer'); | ||
$table->setPrimaryKey(['id']); | ||
|
||
$this->connection->getSchemaManager()->createTable($table); | ||
|
||
$this->connection2 = TestUtil::getConnection(); | ||
|
||
if ($this->connection2->getSchemaManager()->tablesExist('users')) { | ||
return; | ||
} | ||
|
||
if ($this->connection2->getDatabasePlatform() instanceof SqlitePlatform) { | ||
self::markTestSkipped('This test cannot run on SQLite using an in-memory database'); | ||
} | ||
|
||
self::fail('Separate connections do not seem to talk to the same database'); | ||
} | ||
|
||
public function tearDown(): void | ||
{ | ||
parent::tearDown(); | ||
|
||
$this->connection2->close(); | ||
|
||
$this->connection->getSchemaManager()->dropTable('users'); | ||
|
||
if (! $this->connection->getDatabasePlatform() instanceof SQLServerPlatform) { | ||
return; | ||
} | ||
|
||
$name = $this->connection->fetchOne('SELECT db_name()'); | ||
$this->connection->executeStatement('ALTER DATABASE ' . $name . ' SET READ_COMMITTED_SNAPSHOT OFF'); | ||
} | ||
|
||
public function testLockModeNoneDoesNotBreakTransactionIsolation(): void | ||
{ | ||
try { | ||
$this->connection->setTransactionIsolation(TransactionIsolationLevel::READ_COMMITTED); | ||
$this->connection2->setTransactionIsolation(TransactionIsolationLevel::READ_COMMITTED); | ||
} catch (Exception $e) { | ||
self::markTestSkipped('This test must be able to set a transaction isolation level'); | ||
} | ||
|
||
$this->connection->beginTransaction(); | ||
$this->connection2->beginTransaction(); | ||
|
||
$this->connection->insert('users', ['id' => 1]); | ||
|
||
$query = 'SELECT id FROM users'; | ||
$query = $this->connection2->getDatabasePlatform()->appendLockHint($query, LockMode::NONE); | ||
|
||
self::assertSame([], $this->connection2->fetchAllNumeric($query)); | ||
|
||
$this->connection->commit(); | ||
$this->connection2->commit(); | ||
} | ||
} |