From 22233353fdd822b2754d5c82abe34be4786d3938 Mon Sep 17 00:00:00 2001 From: Ruud Kamphuis Date: Tue, 6 Jun 2023 15:20:34 +0200 Subject: [PATCH] Introduce TracingDriverForV3Point2 This driver does not implement the deprecated `VersionAwarePlatformDriver`. It's automatically picked when `doctrine/dbal` version `3.2.0` or higher is installed. Fixes #579 --- .github/workflows/tests.yaml | 12 ++ .../DBAL/TracingDriverForV3Point2.php | 90 ++++++++++++++ src/aliases.php | 9 +- tests/DoctrineTestCase.php | 15 ++- .../DBAL/TracingDriverForV3Point2Test.php | 110 ++++++++++++++++++ .../Doctrine/DBAL/TracingDriverForV3Test.php | 4 + 6 files changed, 235 insertions(+), 5 deletions(-) create mode 100644 src/Tracing/Doctrine/DBAL/TracingDriverForV3Point2.php create mode 100644 tests/Tracing/Doctrine/DBAL/TracingDriverForV3Point2Test.php diff --git a/.github/workflows/tests.yaml b/.github/workflows/tests.yaml index 16c124ff..1e61506d 100644 --- a/.github/workflows/tests.yaml +++ b/.github/workflows/tests.yaml @@ -30,6 +30,8 @@ jobs: - 6.* dependencies: - highest + doctrine-dbal: + - highest exclude: - php: '7.2' symfony-version: 6.* @@ -47,6 +49,12 @@ jobs: - php: '8.0' symfony-version: 6.* dependencies: lowest + - php: '8.0' + symfony-version: 6.* + doctrine-dbal: '^2.13' + - php: '8.0' + symfony-version: 6.* + doctrine-dbal: '<3.2' steps: - name: Checkout @@ -68,6 +76,10 @@ jobs: run: composer require --dev phpunit/phpunit ^9.3.9 --no-update if: matrix.php == '8.0' && matrix.dependencies == 'lowest' + - name: Update Doctrine DBAL + run: composer require --dev doctrine/dbal "${{ matrix.doctrine-dbal }}" --no-update + if: matrix.doctrine-dbal != 'highest' + - name: Install dependencies uses: ramsey/composer-install@v1 with: diff --git a/src/Tracing/Doctrine/DBAL/TracingDriverForV3Point2.php b/src/Tracing/Doctrine/DBAL/TracingDriverForV3Point2.php new file mode 100644 index 00000000..d187dcdf --- /dev/null +++ b/src/Tracing/Doctrine/DBAL/TracingDriverForV3Point2.php @@ -0,0 +1,90 @@ += 3.2. + * + * @internal + * + * @phpstan-import-type Params from \Doctrine\DBAL\DriverManager as ConnectionParams + */ +final class TracingDriverForV3Point2 implements Driver +{ + /** + * @var TracingDriverConnectionFactoryInterface The connection factory + */ + private $connectionFactory; + + /** + * @var Driver|VersionAwarePlatformDriver The instance of the decorated driver + */ + private $decoratedDriver; + + /** + * Constructor. + * + * @param TracingDriverConnectionFactoryInterface $connectionFactory The connection factory + * @param Driver $decoratedDriver The instance of the driver to decorate + */ + public function __construct(TracingDriverConnectionFactoryInterface $connectionFactory, Driver $decoratedDriver) + { + $this->connectionFactory = $connectionFactory; + $this->decoratedDriver = $decoratedDriver; + } + + /** + * {@inheritdoc} + * + * @phpstan-param ConnectionParams $params + */ + public function connect(array $params): TracingDriverConnectionInterface + { + return $this->connectionFactory->create( + $this->decoratedDriver->connect($params), + $this->decoratedDriver->getDatabasePlatform(), + $params + ); + } + + /** + * {@inheritdoc} + */ + public function getDatabasePlatform(): AbstractPlatform + { + return $this->decoratedDriver->getDatabasePlatform(); + } + + /** + * {@inheritdoc} + * + * @phpstan-template T of AbstractPlatform + * + * @phpstan-param T $platform + * + * @phpstan-return AbstractSchemaManager + */ + public function getSchemaManager(Connection $conn, AbstractPlatform $platform): AbstractSchemaManager + { + return $this->decoratedDriver->getSchemaManager($conn, $platform); + } + + /** + * {@inheritdoc} + */ + public function getExceptionConverter(): ExceptionConverter + { + return $this->decoratedDriver->getExceptionConverter(); + } +} diff --git a/src/aliases.php b/src/aliases.php index 839a55c9..68d6145c 100644 --- a/src/aliases.php +++ b/src/aliases.php @@ -4,6 +4,7 @@ namespace Sentry\SentryBundle; +use Composer\InstalledVersions; use Doctrine\DBAL\Result; use Sentry\SentryBundle\Tracing\Cache\TraceableCacheAdapter; use Sentry\SentryBundle\Tracing\Cache\TraceableCacheAdapterForV2; @@ -15,6 +16,7 @@ use Sentry\SentryBundle\Tracing\Doctrine\DBAL\TracingDriverForV2; use Sentry\SentryBundle\Tracing\Doctrine\DBAL\TracingDriverForV3; use Sentry\SentryBundle\Tracing\Doctrine\DBAL\TracingStatement; +use Sentry\SentryBundle\Tracing\Doctrine\DBAL\TracingDriverForV3Point2; use Sentry\SentryBundle\Tracing\Doctrine\DBAL\TracingStatementForV2; use Sentry\SentryBundle\Tracing\Doctrine\DBAL\TracingStatementForV3; use Sentry\SentryBundle\Tracing\HttpClient\TraceableHttpClient; @@ -53,7 +55,12 @@ class_alias(TraceableTagAwareCacheAdapterForV2::class, TraceableTagAwareCacheAda if (!class_exists(TracingStatement::class)) { if (class_exists(Result::class)) { class_alias(TracingStatementForV3::class, TracingStatement::class); - class_alias(TracingDriverForV3::class, TracingDriver::class); + + if (class_exists(InstalledVersions::class) && version_compare(InstalledVersions::getVersion('doctrine/dbal') ?? '0', '3.2', '>=')) { + class_alias(TracingDriverForV3Point2::class, 'Sentry\\SentryBundle\\Tracing\\Doctrine\\DBAL\\TracingDriver'); + } else { + class_alias(TracingDriverForV3::class, TracingDriver::class); + } } elseif (interface_exists(Result::class)) { class_alias(TracingStatementForV2::class, TracingStatement::class); class_alias(TracingDriverForV2::class, TracingDriver::class); diff --git a/tests/DoctrineTestCase.php b/tests/DoctrineTestCase.php index 6d4c7626..d00d8c1e 100644 --- a/tests/DoctrineTestCase.php +++ b/tests/DoctrineTestCase.php @@ -4,6 +4,7 @@ namespace Sentry\SentryBundle\Tests; +use Composer\InstalledVersions; use Doctrine\Bundle\DoctrineBundle\DoctrineBundle; use Doctrine\DBAL\Driver; use Doctrine\DBAL\Driver\ResultStatement; @@ -13,23 +14,29 @@ abstract class DoctrineTestCase extends TestCase { protected static function isDoctrineDBALInstalled(): bool { - return interface_exists(Driver::class); + return InstalledVersions::isInstalled('doctrine/dbal'); } protected static function isDoctrineDBALVersion2Installed(): bool { return self::isDoctrineDBALInstalled() - && interface_exists(ResultStatement::class); + && version_compare(InstalledVersions::getVersion('doctrine/dbal') ?? '0', '3.0', '<'); } protected static function isDoctrineDBALVersion3Installed(): bool { return self::isDoctrineDBALInstalled() - && !self::isDoctrineDBALVersion2Installed(); + && version_compare(InstalledVersions::getVersion('doctrine/dbal') ?? '0', '3.0', '>='); + } + + protected static function isDoctrineDBALVersion3Point2Installed(): bool + { + return self::isDoctrineDBALInstalled() + && version_compare(InstalledVersions::getVersion('doctrine/dbal') ?? '0', '3.2', '>='); } protected static function isDoctrineBundlePackageInstalled(): bool { - return class_exists(DoctrineBundle::class); + return InstalledVersions::isInstalled('doctrine/doctrine-bundle'); } } diff --git a/tests/Tracing/Doctrine/DBAL/TracingDriverForV3Point2Test.php b/tests/Tracing/Doctrine/DBAL/TracingDriverForV3Point2Test.php new file mode 100644 index 00000000..e6a02cdd --- /dev/null +++ b/tests/Tracing/Doctrine/DBAL/TracingDriverForV3Point2Test.php @@ -0,0 +1,110 @@ += 3.2.'); + } + } + + protected function setUp(): void + { + $this->connectionFactory = $this->createMock(TracingDriverConnectionFactoryInterface::class); + } + + public function testConnect(): void + { + $params = ['host' => 'localhost']; + $databasePlatform = $this->createMock(AbstractPlatform::class); + $driverConnection = $this->createMock(DriverConnectionInterface::class); + $tracingDriverConnection = $this->createMock(TracingDriverConnectionInterface::class); + $decoratedDriver = $this->createMock(DriverInterface::class); + + $decoratedDriver->expects($this->once()) + ->method('connect') + ->with($params) + ->willReturn($driverConnection); + + $decoratedDriver->expects($this->once()) + ->method('getDatabasePlatform') + ->willReturn($databasePlatform); + + $this->connectionFactory->expects($this->once()) + ->method('create') + ->with($driverConnection, $databasePlatform, $params) + ->willReturn($tracingDriverConnection); + + $driver = new TracingDriverForV3Point2($this->connectionFactory, $decoratedDriver); + + $this->assertSame($tracingDriverConnection, $driver->connect($params)); + } + + public function testGetDatabasePlatform(): void + { + $databasePlatform = $this->createMock(AbstractPlatform::class); + + $decoratedDriver = $this->createMock(DriverInterface::class); + $decoratedDriver->expects($this->once()) + ->method('getDatabasePlatform') + ->willReturn($databasePlatform); + + $driver = new TracingDriverForV3Point2($this->connectionFactory, $decoratedDriver); + + $this->assertSame($databasePlatform, $driver->getDatabasePlatform()); + } + + public function testGetSchemaManager(): void + { + $connection = $this->createMock(Connection::class); + $databasePlatform = $this->createMock(AbstractPlatform::class); + $schemaManager = $this->createMock(AbstractSchemaManager::class); + + $decoratedDriver = $this->createMock(DriverInterface::class); + $decoratedDriver->expects($this->once()) + ->method('getSchemaManager') + ->with($connection, $databasePlatform) + ->willReturn($schemaManager); + + $driver = new TracingDriverForV3Point2($this->connectionFactory, $decoratedDriver); + + $this->assertSame($schemaManager, $driver->getSchemaManager($connection, $databasePlatform)); + } + + public function testGetExceptionConverter(): void + { + $exceptionConverter = $this->createMock(ExceptionConverter::class); + + $decoratedDriver = $this->createMock(DriverInterface::class); + $decoratedDriver->expects($this->once()) + ->method('getExceptionConverter') + ->willReturn($exceptionConverter); + + $driver = new TracingDriverForV3Point2($this->connectionFactory, $decoratedDriver); + + $this->assertSame($exceptionConverter, $driver->getExceptionConverter()); + } +} diff --git a/tests/Tracing/Doctrine/DBAL/TracingDriverForV3Test.php b/tests/Tracing/Doctrine/DBAL/TracingDriverForV3Test.php index af3a857b..569259c1 100644 --- a/tests/Tracing/Doctrine/DBAL/TracingDriverForV3Test.php +++ b/tests/Tracing/Doctrine/DBAL/TracingDriverForV3Test.php @@ -29,6 +29,10 @@ public static function setUpBeforeClass(): void if (!self::isDoctrineDBALVersion3Installed()) { self::markTestSkipped('This test requires the version of the "doctrine/dbal" Composer package to be >= 3.0.'); } + + if (self::isDoctrineDBALVersion3Point2Installed()) { + self::markTestSkipped('This test requires the version of the "doctrine/dbal" Composer package to be >= 3.0 <= 3.0.'); + } } protected function setUp(): void