Merge pull request #12 from tienvx/fix-wrong-arguments

Fix wrong arguments
tienvx · Mar 11, 2023 · 59b131e · 59b131e
2 parents 59d6404 + 5cc1dcd
commit 59b131e
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 12 deletions.
diff --git a/src/Resources/config/services.php b/src/Resources/config/services.php
@@ -10,6 +10,8 @@
 use Tienvx\Bundle\PactProviderBundle\Service\StateHandlerManagerInterface;
 
 return static function (ContainerConfigurator $container): void {
+    $namespace = __NAMESPACE__;
+    $service = function_exists("$namespace\\service") ? "$namespace\\service" : "$namespace\\ref";
     $container->services()
         ->set(StateHandlerManager::class)
             ->args([
@@ -25,16 +27,16 @@
 
         ->set(StateChangeRequestListener::class)
             ->args([
-                StateHandlerManagerInterface::class,
+                $service(StateHandlerManagerInterface::class),
                 '',
                 true,
             ])
             // Before Symfony\Component\HttpKernel\EventListener\RouterListener::onKernelRequest
             ->tag('kernel.event_listener', ['priority' => 33])
         ->set(DispatchMessageRequestListener::class)
             ->args([
-                StateHandlerManagerInterface::class,
-                MessageDispatcherManagerInterface::class,
+                $service(StateHandlerManagerInterface::class),
+                $service(MessageDispatcherManagerInterface::class),
             ])
             // Before Symfony\Component\HttpKernel\EventListener\RouterListener::onKernelRequest
             ->tag('kernel.event_listener', ['priority' => 33])

diff --git a/tests/DependencyInjection/TienvxPactProviderExtensionTest.php b/tests/DependencyInjection/TienvxPactProviderExtensionTest.php
@@ -45,18 +45,20 @@ public function testLoad(): void
             ],
             StateChangeRequestListener::class => [
                 'tag' => 'kernel.event_listener',
-                'args' => fn (array $args) => $args === [
-                    StateHandlerManagerInterface::class,
-                    '/path/to/state/change',
-                    false,
-                ],
+                'args' => function (array $args): bool {
+                    return 3 === count($args) &&
+                        StateHandlerManagerInterface::class == $args[0] &&
+                        '/path/to/state/change' === $args[1] &&
+                        false === $args[2];
+                },
             ],
             DispatchMessageRequestListener::class => [
                 'tag' => 'kernel.event_listener',
-                'args' => fn (array $args) => $args === [
-                    StateHandlerManagerInterface::class,
-                    MessageDispatcherManagerInterface::class,
-                ],
+                'args' => function (array $args): bool {
+                    return 2 === count($args) &&
+                        StateHandlerManagerInterface::class == $args[0] &&
+                        MessageDispatcherManagerInterface::class == $args[1];
+                },
             ],
         ];
         foreach ($services as $key => $value) {