Add DBAL version check to compiler pass

Author: Jean85

src/DependencyInjection/Compiler/DbalTracingPass.php

@@ -4,6 +4,7 @@
 
 namespace Sentry\SentryBundle\DependencyInjection\Compiler;
 
+use Doctrine\DBAL\Driver;
 use Doctrine\DBAL\Driver\ResultStatement;
 use Sentry\SentryBundle\Tracing\Doctrine\DBAL\ConnectionConfigurator;
 use Sentry\SentryBundle\Tracing\Doctrine\DBAL\TracingDriverMiddleware;
@@ -39,6 +40,10 @@ public function process(ContainerBuilder $container): void
             return;
         }
 
+        if (!interface_exists(Driver::class)) {
+            throw new \InvalidArgumentException(sprintf('The Doctrine "%s" interface is missing, DBAL connection cannot be instrumented; check that you have DBAL 2.13+ installed', Driver::class));
+        }
+
         /** @var string[] $connectionsToTrace */
         $connectionsToTrace = $container->getParameter('sentry.tracing.dbal.connections');
 

tests/DependencyInjection/Compiler/DbalTracingPassTest.php

@@ -103,6 +103,21 @@ public function testProcessWithDoctrineDBALVersionLowerThan30(): void
         $this->assertNull($connection2->getConfigurator());
     }
 
+    public function testProcessWithDoctrineDBALVersionLowerThan213OrMissing(): void
+    {
+        if (self::isDoctrineDBALInstalled()) {
+            $this->markTestSkipped('This test requires the version of the "doctrine/dbal" Composer package to be < 2.13 or missing.');
+        }
+
+        $container = $this->createContainerBuilder();
+        $container->setParameter('sentry.tracing.dbal.connections', ['foo', 'baz']);
+
+        $this->expectException(\RuntimeException::class);
+        $this->expectExceptionMessage('DBAL connection cannot be instrumented; check that you have DBAL 2.13+ installed');
+
+        $container->compile();
+    }
+
     /**
      * @dataProvider processDoesNothingIfConditionsForEnablingTracingAreMissingDataProvider
      */

tests/DoctrineTestCase.php

@@ -5,14 +5,20 @@
 namespace Sentry\SentryBundle\Tests;
 
 use Doctrine\Bundle\DoctrineBundle\DoctrineBundle;
-use Doctrine\DBAL\Driver\ResultStatement;
+use Doctrine\DBAL\Driver;
+use Doctrine\DBAL\Driver\Middleware;
 use PHPUnit\Framework\TestCase;
 
 abstract class DoctrineTestCase extends TestCase
 {
+    protected static function isDoctrineDBALInstalled(): bool
+    {
+        return interface_exists(Driver::class);
+    }
+
     protected static function isDoctrineDBALVersion3Installed(): bool
     {
-        return !interface_exists(ResultStatement::class);
+        return interface_exists(Middleware::class);
     }
 
     protected static function isDoctrineBundlePackageInstalled(): bool