Merge pull request #43 from smartbooster/41_batch_log

#41 add sentry to BatchLogger

composer.json

@@ -29,7 +29,8 @@
     "symfony/yaml": "^4.4 || ^5.4 || ^6.0",
     "symfony/dependency-injection": "^4.4 || ^5.4 || ^6.0",
     "nelmio/security-bundle": "^2.8",
-    "yokai/security-token-bundle": "^3.3"
+    "yokai/security-token-bundle": "^3.3",
+    "sentry/sentry-symfony": "^4.1"
   },
   "require-dev": {
     "squizlabs/php_codesniffer": "^3.7",

config/services.yaml

@@ -137,6 +137,7 @@ services:
     public: true
     arguments:
       - '@Doctrine\ORM\EntityManagerInterface'
+      - '@Sentry\ClientInterface'
 
   Smart\SonataBundle\Logger\BatchLogger:
     public: false

src/Logger/BatchLogger.php

@@ -3,19 +3,26 @@
 namespace Smart\SonataBundle\Logger;
 
 use Doctrine\ORM\EntityManagerInterface;
+use Sentry\ClientInterface;
 use Smart\SonataBundle\Entity\Log\BatchLog;
 
 class BatchLogger
 {
     private EntityManagerInterface $entityManager;
+    private ClientInterface $sentry;
 
-    public function __construct(EntityManagerInterface $entityManager)
+    public function __construct(EntityManagerInterface $entityManager, ClientInterface $sentry)
     {
         $this->entityManager = $entityManager;
+        $this->sentry = $sentry;
     }
 
     public function log(array $parameters): void
     {
+        if (isset($parameters['exception'])) {
+            $this->sentry->captureException($parameters['exception']);
+        }
+
         $batchLog = new BatchLog();
         if (!isset($parameters['context'])) {
             throw new \Exception("'context' must be present in batch log");

tests/AppKernel.php

@@ -31,6 +31,7 @@ public function registerBundles()
             new \Yokai\SecurityTokenBundle\YokaiSecurityTokenBundle(),
             new \Sonata\DoctrineORMAdminBundle\SonataDoctrineORMAdminBundle(),
             new \Knp\Bundle\MenuBundle\KnpMenuBundle(),
+            new \Sentry\SentryBundle\SentryBundle(),
         ];
     }
 

tests/DependencyInjection/DependencyInjectionTest.php

@@ -4,6 +4,7 @@
 
 use Doctrine\ORM\EntityManagerInterface;
 use PHPUnit\Framework\TestCase;
+use Sentry\ClientInterface;
 use Smart\SonataBundle\SmartSonataBundle;
 use Symfony\Component\Config\Definition\Exception\InvalidConfigurationException;
 use Symfony\Component\Config\FileLocator;
@@ -25,6 +26,7 @@ protected function setUp(): void
         $bundle = new SmartSonataBundle();
         $this->container = new ContainerBuilder();
         $this->container->setDefinition('Doctrine\ORM\EntityManagerInterface', new Definition(EntityManagerInterface::class));
+        $this->container->setDefinition('Sentry\ClientInterface', new Definition(ClientInterface::class));
         $this->container->registerExtension($bundle->getContainerExtension());
         $bundle->build($this->container);
     }