Add cache and vendor dirs to default in app exclusions

Author: Jean85

src/DependencyInjection/Configuration.php

@@ -61,7 +61,10 @@ public function getConfigTreeBuilder()
             ->scalarNode('error_types')
             ->end()
             ->arrayNode('in_app_exclude')
-                ->defaultValue($defaultValues->getInAppExcludedPaths())
+                ->defaultValue([
+                    '%kernel.cache_dir%',
+                    $this->getProjectRoot() . '/vendor',
+                ])
                 ->scalarPrototype()->end()
             ->end()
             ->arrayNode('excluded_exceptions')

test/DependencyInjection/ConfigurationTest.php

@@ -49,7 +49,10 @@ public function testConfigurationDefaults(): void
             ],
             'options' => [
                 'environment' => '%kernel.environment%',
-                'in_app_exclude' => $defaultSdkValues->getInAppExcludedPaths(),
+                'in_app_exclude' => [
+                    '%kernel.cache_dir%',
+                    '%kernel.root_dir%/../vendor',
+                ],
                 'excluded_exceptions' => $defaultSdkValues->getExcludedExceptions(),
                 'prefixes' => $defaultSdkValues->getPrefixes(),
                 'project_root' => '%kernel.root_dir%/..',
@@ -59,6 +62,7 @@ public function testConfigurationDefaults(): void
 
         if (method_exists(Kernel::class, 'getProjectDir')) {
             $expectedDefaults['options']['project_root'] = '%kernel.project_dir%';
+            $expectedDefaults['options']['in_app_exclude'][1] = '%kernel.project_dir%/vendor';
         }
 
         $this->assertEquals($expectedDefaults, $processed);

test/DependencyInjection/SentryExtensionTest.php

@@ -38,12 +38,16 @@ public function testOptionsDefaultValues(): void
         $options = $this->getOptionsFrom($container);
 
         if (method_exists(Kernel::class, 'getProjectDir')) {
-            $this->assertSame('/dir/project/root/', $options->getProjectRoot());
+            $vendorDir = '/dir/project/root/vendor';
+            $this->assertSame('/dir/project/root', $options->getProjectRoot());
         } else {
+            $vendorDir = 'kernel/root/../vendor';
             $this->assertSame('kernel/root/..', $options->getProjectRoot());
         }
+
         $this->assertNull($options->getDsn());
         $this->assertSame('test', $options->getEnvironment());
+        $this->assertSame(['var/cache', $vendorDir], $options->getInAppExcludedPaths());
 
         $this->assertSame(1, $container->getParameter('sentry.listener_priorities.request'));
         $this->assertSame(1, $container->getParameter('sentry.listener_priorities.console'));
@@ -144,9 +148,10 @@ public function emptyDsnValueProvider(): array
     private function getContainer(array $configuration = []): Container
     {
         $containerBuilder = new ContainerBuilder();
+        $containerBuilder->setParameter('kernel.cache_dir', 'var/cache');
         $containerBuilder->setParameter('kernel.root_dir', 'kernel/root');
         if (method_exists(Kernel::class, 'getProjectDir')) {
-            $containerBuilder->setParameter('kernel.project_dir', '/dir/project/root/');
+            $containerBuilder->setParameter('kernel.project_dir', '/dir/project/root');
         }
         $containerBuilder->setParameter('kernel.environment', 'test');