fix(DBAL): handle Dbal 4.x EnumType (#243)

Author: GaryPEGEOT

DependencyInjection/Compiler/RegisterEnumTypePass.php

@@ -12,6 +12,8 @@
 
 namespace Fresh\DoctrineEnumBundle\DependencyInjection\Compiler;
 
+use Doctrine\DBAL\Types\EnumType;
+use Doctrine\DBAL\Types\Types;
 use Doctrine\Persistence\ManagerRegistry;
 use Fresh\DoctrineEnumBundle\Exception\InvalidArgumentException;
 use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
@@ -39,8 +41,9 @@ public function process(ContainerBuilder $container): void
         foreach ($doctrine->getConnectionNames() as $connectionName) {
             $definition = $container->getDefinition($connectionName);
             $mappingTypes = (array) $definition->getArgument(3);
-            if (!isset($mappingTypes['enum']) || 'string' !== $mappingTypes['enum']) {
-                $mappingTypes['enum'] = 'string';
+            $expectedType = class_exists(EnumType::class) ? Types::ENUM : 'string';
+            if (!isset($mappingTypes['enum']) || $expectedType !== $mappingTypes['enum']) {
+                $mappingTypes['enum'] = $expectedType;
                 $definition->setArgument(3, $mappingTypes);
             }
         }

Tests/DependencyInjection/Compiler/RegisterEnumTypePassTest.php

@@ -12,6 +12,7 @@
 
 namespace Fresh\DoctrineEnumBundle\Tests\DependencyInjection\Compiler;
 
+use Doctrine\DBAL\Types\Types;
 use Doctrine\Persistence\ManagerRegistry;
 use Fresh\DoctrineEnumBundle\DependencyInjection\Compiler\RegisterEnumTypePass;
 use Fresh\DoctrineEnumBundle\Exception\InvalidArgumentException;
@@ -79,7 +80,7 @@ public function processSuccessful(): void
         $default
             ->expects(self::once())
             ->method('setArgument')
-            ->with(3, ['test' => '_test', 'enum' => 'string'])
+            ->with(3, ['test' => '_test', 'enum' => Types::ENUM])
         ;
 
         $custom1 = $this->createMock(Definition::class);
@@ -92,15 +93,15 @@ public function processSuccessful(): void
         $custom1
             ->expects(self::once())
             ->method('setArgument')
-            ->with(3, ['test' => '_test', 'enum' => 'string'])
+            ->with(3, ['test' => '_test', 'enum' => Types::ENUM])
         ;
 
         $custom2 = $this->createMock(Definition::class);
         $custom2
             ->expects(self::once())
             ->method('getArgument')
             ->with(3)
-            ->willReturn(['test' => '_test', 'enum' => 'string'])
+            ->willReturn(['test' => '_test', 'enum' => Types::ENUM])
         ;
         $custom2
             ->expects(self::never())