Map integrations config to options

Author: Jean85

src/DependencyInjection/SentryExtension.php

@@ -93,6 +93,15 @@ private function passConfigurationToOptions(ContainerBuilder $container, array $
         if (\array_key_exists('before_breadcrumb', $processedOptions)) {
             $this->mapCallableValue($options, 'setBeforeBreadcrumbCallback', $processedOptions['before_breadcrumb']);
         }
+
+        if (\array_key_exists('integrations', $processedOptions)) {
+            $integrations = [];
+            foreach ($processedOptions['integrations'] as $integrationName) {
+                $integrations[] = new Reference(substr($integrationName, 1));
+            }
+
+            $options->addMethodCall('setIntegrations', [$integrations]);
+        }
     }
 
     /**

test/DependencyInjection/SentryExtensionTest.php

@@ -5,6 +5,7 @@
 use PHPUnit\Framework\TestCase;
 use Sentry\Breadcrumb;
 use Sentry\Event;
+use Sentry\Integration\IntegrationInterface;
 use Sentry\Options;
 use Sentry\SentryBundle\DependencyInjection\SentryExtension;
 use Sentry\SentryBundle\EventListener\ConsoleListener;
@@ -28,8 +29,9 @@ public function testDataProviderIsMappingTheRightNumberOfOptions(): void
         $providerData = $this->optionsValueProvider();
         $supportedOptions = \array_unique(\array_column($providerData, 0));
 
+        // subtracted one is `integration`, which cannot be tested with the provider
         $this->assertCount(
-            ConfigurationTest::SUPPORTED_SENTRY_OPTIONS_COUNT,
+            ConfigurationTest::SUPPORTED_SENTRY_OPTIONS_COUNT - 1,
             $supportedOptions,
             'Provider for configuration options mismatch: ' . PHP_EOL . print_r($supportedOptions, true)
         );
@@ -280,6 +282,19 @@ public function testBeforeBreadcrumbWithInvalidServiceReference(): void
         $this->getOptionsFrom($container)->getBeforeBreadcrumbCallback();
     }
 
+    public function testIntegrations(): void
+    {
+        $container = $this->getContainer([
+            'options' => [
+                'integrations' => ['@integration_mock'],
+            ],
+        ]);
+
+        $integrations = $this->getOptionsFrom($container)->getIntegrations();
+        $this->assertContainsOnlyInstancesOf(IntegrationMock::class, $integrations);
+        $this->assertCount(1, $integrations);
+    }
+
     private function getContainer(array $configuration = []): Container
     {
         $containerBuilder = new ContainerBuilder();
@@ -290,11 +305,9 @@ private function getContainer(array $configuration = []): Container
         }
         $containerBuilder->setParameter('kernel.environment', 'test');
 
-        $mockEventDispatcher = $this
-            ->createMock(EventDispatcherInterface::class);
+        $mockEventDispatcher = $this->createMock(EventDispatcherInterface::class);
 
-        $mockRequestStack = $this
-            ->createMock(RequestStack::class);
+        $mockRequestStack = $this->createMock(RequestStack::class);
 
         $containerBuilder->set('request_stack', $mockRequestStack);
         $containerBuilder->set('event_dispatcher', $mockEventDispatcher);
@@ -306,6 +319,9 @@ private function getContainer(array $configuration = []): Container
         $beforeSend->setFactory([CallbackMock::class, 'createCallback']);
         $containerBuilder->setDefinition('callable_mock', $beforeSend);
 
+        $integration = new Definition(IntegrationMock::class);
+        $containerBuilder->setDefinition('integration_mock', $integration);
+
         $extension = new SentryExtension();
         $extension->load(['sentry' => $configuration], $containerBuilder);
 
@@ -347,3 +363,10 @@ public static function createCallback(): callable
         return [new self(), 'callback'];
     }
 }
+
+class IntegrationMock implements IntegrationInterface
+{
+    public function setupOnce(): void
+    {
+    }
+}