bug #39361 [FrameworkBundle] acces public-deprecated services via the private container to remove false-positive deprecations (nicolas-grekas)

This PR was merged into the 5.1 branch.

Discussion
----------

[FrameworkBundle] acces public-deprecated services via the private container to remove false-positive deprecations

| Q             | A
| ------------- | ---
| Branch?       | 5.1
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       | -
| License       | MIT
| Doc PR        | -

Something we missed in #36470 apparently.

Commits
-------

d502ba9e5d [FrameworkBundle] acces public-deprecated services via the private container to remove false-positive deprecations

Author: nicolas-grekas

DependencyInjection/Compiler/TestServiceContainerWeakRefPass.php

@@ -22,6 +22,13 @@
  */
 class TestServiceContainerWeakRefPass implements CompilerPassInterface
 {
+    private $privateTagName;
+
+    public function __construct(string $privateTagName = 'container.private')
+    {
+        $this->privateTagName = $privateTagName;
+    }
+
     public function process(ContainerBuilder $container)
     {
         if (!$container->hasDefinition('test.private_services_locator')) {
@@ -33,7 +40,7 @@ public function process(ContainerBuilder $container)
         $hasErrors = method_exists(Definition::class, 'hasErrors') ? 'hasErrors' : 'getErrors';
 
         foreach ($definitions as $id => $definition) {
-            if ($id && '.' !== $id[0] && (!$definition->isPublic() || $definition->isPrivate()) && !$definition->$hasErrors() && !$definition->isAbstract()) {
+            if ($id && '.' !== $id[0] && (!$definition->isPublic() || $definition->isPrivate() || $definition->hasTag($this->privateTagName)) && !$definition->$hasErrors() && !$definition->isAbstract()) {
                 $privateServices[$id] = new Reference($id, ContainerBuilder::IGNORE_ON_UNINITIALIZED_REFERENCE);
             }
         }

Tests/DependencyInjection/Compiler/TestServiceContainerRefPassesTest.php

@@ -36,6 +36,12 @@ public function testProcess()
             ->setPublic(true)
             ->addArgument(new Reference('Test\private_used_shared_service'))
             ->addArgument(new Reference('Test\private_used_non_shared_service'))
+            ->addArgument(new Reference('Test\soon_private_service'))
+        ;
+
+        $container->register('Test\soon_private_service')
+            ->setPublic(true)
+            ->addTag('container.private', ['package' => 'foo/bar', 'version' => '1.42'])
         ;
 
         $container->register('Test\private_used_shared_service');
@@ -48,6 +54,7 @@ public function testProcess()
         $expected = [
             'Test\private_used_shared_service' => new ServiceClosureArgument(new Reference('Test\private_used_shared_service')),
             'Test\private_used_non_shared_service' => new ServiceClosureArgument(new Reference('Test\private_used_non_shared_service')),
+            'Test\soon_private_service' => new ServiceClosureArgument(new Reference('.container.private.Test\soon_private_service')),
             'Psr\Container\ContainerInterface' => new ServiceClosureArgument(new Reference('service_container')),
             'Symfony\Component\DependencyInjection\ContainerInterface' => new ServiceClosureArgument(new Reference('service_container')),
         ];