[DependencyInjection] Add tests for repeating #[Autoconfigure] attributes

alexandre-daubois · nicolas-grekas · commit 960d2596145a · 2024-08-20T11:51:49.000+02:00
diff --git a/Tests/Compiler/RegisterAutoconfigureAttributesPassTest.php b/Tests/Compiler/RegisterAutoconfigureAttributesPassTest.php
@@ -19,6 +19,12 @@
 use Symfony\Component\DependencyInjection\Reference;
 use Symfony\Component\DependencyInjection\Tests\Fixtures\AutoconfigureAttributed;
 use Symfony\Component\DependencyInjection\Tests\Fixtures\AutoconfiguredInterface;
+use Symfony\Component\DependencyInjection\Tests\Fixtures\AutoconfigureRepeated;
+use Symfony\Component\DependencyInjection\Tests\Fixtures\AutoconfigureRepeatedBindings;
+use Symfony\Component\DependencyInjection\Tests\Fixtures\AutoconfigureRepeatedCalls;
+use Symfony\Component\DependencyInjection\Tests\Fixtures\AutoconfigureRepeatedOverwrite;
+use Symfony\Component\DependencyInjection\Tests\Fixtures\AutoconfigureRepeatedProperties;
+use Symfony\Component\DependencyInjection\Tests\Fixtures\AutoconfigureRepeatedTag;
 use Symfony\Component\DependencyInjection\Tests\Fixtures\ParentNotExists;
 
 /**
@@ -77,6 +83,99 @@ public function testAutoconfiguredTag()
         $this->assertEquals([AutoconfiguredInterface::class => $expected], $container->getAutoconfiguredInstanceof());
     }
 
+    public function testAutoconfiguredRepeated()
+    {
+        $container = new ContainerBuilder();
+        $container->register('foo', AutoconfigureRepeated::class)
+            ->setAutoconfigured(true);
+
+        (new RegisterAutoconfigureAttributesPass())->process($container);
+
+        $expected = (new ChildDefinition(''))
+            ->setLazy(true)
+            ->setPublic(true)
+            ->setShared(false);
+
+        $this->assertEquals([AutoconfigureRepeated::class => $expected], $container->getAutoconfiguredInstanceof());
+    }
+
+    public function testAutoconfiguredRepeatedOverwrite()
+    {
+        $container = new ContainerBuilder();
+        $container->register('foo', AutoconfigureRepeatedOverwrite::class)
+            ->setAutoconfigured(true);
+
+        (new RegisterAutoconfigureAttributesPass())->process($container);
+
+        $expected = (new ChildDefinition(''))
+            ->setLazy(true)
+            ->setPublic(false)
+            ->setShared(true);
+
+        $this->assertEquals([AutoconfigureRepeatedOverwrite::class => $expected], $container->getAutoconfiguredInstanceof());
+    }
+
+    public function testAutoconfiguredRepeatedTag()
+    {
+        $container = new ContainerBuilder();
+        $container->register('foo', AutoconfigureRepeatedTag::class)
+            ->setAutoconfigured(true);
+
+        (new RegisterAutoconfigureAttributesPass())->process($container);
+
+        $expected = (new ChildDefinition(''))
+            ->addTag('foo', ['priority' => 2])
+            ->addTag('bar');
+
+        $this->assertEquals([AutoconfigureRepeatedTag::class => $expected], $container->getAutoconfiguredInstanceof());
+    }
+
+    public function testAutoconfiguredRepeatedCalls()
+    {
+        $container = new ContainerBuilder();
+        $container->register('foo', AutoconfigureRepeatedCalls::class)
+            ->setAutoconfigured(true);
+
+        (new RegisterAutoconfigureAttributesPass())->process($container);
+
+        $expected = (new ChildDefinition(''))
+            ->addMethodCall('setBar', ['arg2'])
+            ->addMethodCall('setFoo', ['arg1']);
+
+        $this->assertEquals([AutoconfigureRepeatedCalls::class => $expected], $container->getAutoconfiguredInstanceof());
+    }
+
+    public function testAutoconfiguredRepeatedBindingsOverwrite()
+    {
+        $container = new ContainerBuilder();
+        $container->register('foo', AutoconfigureRepeatedBindings::class)
+            ->setAutoconfigured(true);
+
+        (new RegisterAutoconfigureAttributesPass())->process($container);
+
+        $expected = (new ChildDefinition(''))
+            ->setBindings(['$arg' => new BoundArgument('bar', false, BoundArgument::INSTANCEOF_BINDING, realpath(__DIR__.'/../Fixtures/AutoconfigureRepeatedBindings.php'))]);
+
+        $this->assertEquals([AutoconfigureRepeatedBindings::class => $expected], $container->getAutoconfiguredInstanceof());
+    }
+
+    public function testAutoconfiguredRepeatedPropertiesOverwrite()
+    {
+        $container = new ContainerBuilder();
+        $container->register('foo', AutoconfigureRepeatedProperties::class)
+            ->setAutoconfigured(true);
+
+        (new RegisterAutoconfigureAttributesPass())->process($container);
+
+        $expected = (new ChildDefinition(''))
+            ->setProperties([
+                '$foo' => 'bar',
+                '$bar' => 'baz',
+            ]);
+
+        $this->assertEquals([AutoconfigureRepeatedProperties::class => $expected], $container->getAutoconfiguredInstanceof());
+    }
+
     public function testMissingParent()
     {
         $container = new ContainerBuilder();
diff --git a/Tests/Fixtures/AutoconfigureRepeated.php b/Tests/Fixtures/AutoconfigureRepeated.php
@@ -0,0 +1,11 @@
+<?php
+
+namespace Symfony\Component\DependencyInjection\Tests\Fixtures;
+
+use Symfony\Component\DependencyInjection\Attribute\Autoconfigure;
+
+#[Autoconfigure(public: true, shared: false)]
+#[Autoconfigure(lazy: true)]
+class AutoconfigureRepeated
+{
+}
diff --git a/Tests/Fixtures/AutoconfigureRepeatedBindings.php b/Tests/Fixtures/AutoconfigureRepeatedBindings.php
@@ -0,0 +1,11 @@
+<?php
+
+namespace Symfony\Component\DependencyInjection\Tests\Fixtures;
+
+use Symfony\Component\DependencyInjection\Attribute\Autoconfigure;
+
+#[Autoconfigure(bind: ['$arg' => 'foo'])]
+#[Autoconfigure(bind: ['$arg' => 'bar'])]
+class AutoconfigureRepeatedBindings
+{
+}
diff --git a/Tests/Fixtures/AutoconfigureRepeatedCalls.php b/Tests/Fixtures/AutoconfigureRepeatedCalls.php
@@ -0,0 +1,18 @@
+<?php
+
+namespace Symfony\Component\DependencyInjection\Tests\Fixtures;
+
+use Symfony\Component\DependencyInjection\Attribute\Autoconfigure;
+
+#[Autoconfigure(calls: [['setBar', ['arg2']]])]
+#[Autoconfigure(calls: [['setFoo', ['arg1']]])]
+class AutoconfigureRepeatedCalls
+{
+    public function setFoo(string $arg)
+    {
+    }
+
+    public function setBar(string $arg)
+    {
+    }
+}
diff --git a/Tests/Fixtures/AutoconfigureRepeatedOverwrite.php b/Tests/Fixtures/AutoconfigureRepeatedOverwrite.php
@@ -0,0 +1,11 @@
+<?php
+
+namespace Symfony\Component\DependencyInjection\Tests\Fixtures;
+
+use Symfony\Component\DependencyInjection\Attribute\Autoconfigure;
+
+#[Autoconfigure(public: true, shared: false)]
+#[Autoconfigure(lazy: true, shared: true, public: false)]
+class AutoconfigureRepeatedOverwrite
+{
+}
diff --git a/Tests/Fixtures/AutoconfigureRepeatedProperties.php b/Tests/Fixtures/AutoconfigureRepeatedProperties.php
@@ -0,0 +1,11 @@
+<?php
+
+namespace Symfony\Component\DependencyInjection\Tests\Fixtures;
+
+use Symfony\Component\DependencyInjection\Attribute\Autoconfigure;
+
+#[Autoconfigure(properties: ['$replaced' => 'to be replaced', '$bar' => 'existing to be replaced'])]
+#[Autoconfigure(properties: ['$foo' => 'bar', '$bar' => 'baz'])]
+class AutoconfigureRepeatedProperties
+{
+}
diff --git a/Tests/Fixtures/AutoconfigureRepeatedTag.php b/Tests/Fixtures/AutoconfigureRepeatedTag.php
@@ -0,0 +1,11 @@
+<?php
+
+namespace Symfony\Component\DependencyInjection\Tests\Fixtures;
+
+use Symfony\Component\DependencyInjection\Attribute\AutoconfigureTag;
+
+#[AutoconfigureTag('foo', ['priority' => 2])]
+#[AutoconfigureTag('bar')]
+class AutoconfigureRepeatedTag
+{
+}
