Fix phpunit

Setono · Jan 13, 2025 · e06887d · e06887d
1 parent 38da9a9
commit e06887d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 7 deletions.
diff --git a/src/DependencyInjection/SetonoConsentExtension.php b/src/DependencyInjection/SetonoConsentExtension.php
@@ -10,7 +10,6 @@
 use Symfony\Component\Config\FileLocator;
 use Symfony\Component\DependencyInjection\Alias;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
-use Symfony\Component\DependencyInjection\Exception\InvalidArgumentException;
 use Symfony\Component\DependencyInjection\Extension\Extension;
 use Symfony\Component\DependencyInjection\Loader\XmlFileLoader;
 
@@ -33,12 +32,6 @@ public function load(array $configs, ContainerBuilder $container): void
 
         $container->setParameter('setono_consent.consents', $consents);
 
-        // validate the consent checker
-        $consentChecker = $container->getDefinition($config['consent_checker']);
-        if (!is_a($consentChecker->getClass(), ConsentCheckerInterface::class, true)) {
-            throw new InvalidArgumentException(sprintf('The consent checker must implement %s', ConsentCheckerInterface::class));
-        }
-
         $alias = new Alias($config['consent_checker']);
         $alias->setDeprecated('setono/consent-bundle', '1.2', sprintf('The "%%alias_id%%" service is deprecated. You should use the "%s" alias instead.', ConsentCheckerInterface::class));
 

diff --git a/tests/DependencyInjection/ConfigurationTest.php b/tests/DependencyInjection/ConfigurationTest.php
@@ -6,6 +6,7 @@
 
 use Matthias\SymfonyConfigTest\PhpUnit\ConfigurationTestCaseTrait;
 use PHPUnit\Framework\TestCase;
+use Setono\ConsentBundle\Checker\StaticConsentChecker;
 use Setono\ConsentBundle\DependencyInjection\Configuration;
 
 /**
@@ -39,6 +40,7 @@ public function it_processes_configuration(): void
                 'statistics' => true,
                 'random' => true,
             ],
+            'consent_checker' => StaticConsentChecker::class,
         ]);
     }
 }