TraceableHttpClient: increase decorator's priority

adpeyre · Adrien Peyre · commit 1e3f9433ef4a · 2022-10-13T14:17:07.000+02:00
diff --git a/src/Symfony/Component/HttpClient/DependencyInjection/HttpClientPass.php b/src/Symfony/Component/HttpClient/DependencyInjection/HttpClientPass.php
@@ -38,7 +38,7 @@ public function process(ContainerBuilder $container)
             $container->register('.debug.'.$id, TraceableHttpClient::class)
                 ->setArguments([new Reference('.debug.'.$id.'.inner')])
                 ->addTag('kernel.reset', ['method' => 'reset'])
-                ->setDecoratedService($id);
+                ->setDecoratedService($id, null, 100);
             $container->getDefinition('data_collector.http_client')
                 ->addMethodCall('registerClient', [$id, new Reference('.debug.'.$id)]);
         }
diff --git a/src/Symfony/Component/HttpClient/Tests/DependencyInjection/HttpClientPassTest.php b/src/Symfony/Component/HttpClient/Tests/DependencyInjection/HttpClientPassTest.php
@@ -38,7 +38,7 @@ public function testItDecoratesHttpClientWithTraceableHttpClient()
         $sut->process($container);
         $this->assertTrue($container->hasDefinition('.debug.foo'));
         $this->assertSame(TraceableHttpClient::class, $container->getDefinition('.debug.foo')->getClass());
-        $this->assertSame(['foo', null, 0], $container->getDefinition('.debug.foo')->getDecoratedService());
+        $this->assertSame(['foo', null, 100], $container->getDefinition('.debug.foo')->getDecoratedService());
     }
 
     public function testItRegistersDebugHttpClientToCollector()

Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,7 @@ public function process(ContainerBuilder $container)`
`38`	`38`	`$container->register('.debug.'.$id, TraceableHttpClient::class)`
`39`	`39`	`->setArguments([new Reference('.debug.'.$id.'.inner')])`
`40`	`40`	`->addTag('kernel.reset', ['method' => 'reset'])`
`41`		`- ->setDecoratedService($id);`
	`41`	`+ ->setDecoratedService($id, null, 100);`
`42`	`42`	`$container->getDefinition('data_collector.http_client')`
`43`	`43`	`->addMethodCall('registerClient', [$id, new Reference('.debug.'.$id)]);`
`44`	`44`	`}`
Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,7 @@ public function testItDecoratesHttpClientWithTraceableHttpClient()`
`38`	`38`	`$sut->process($container);`
`39`	`39`	`$this->assertTrue($container->hasDefinition('.debug.foo'));`
`40`	`40`	`$this->assertSame(TraceableHttpClient::class, $container->getDefinition('.debug.foo')->getClass());`
`41`		`- $this->assertSame(['foo', null, 0], $container->getDefinition('.debug.foo')->getDecoratedService());`
	`41`	`+ $this->assertSame(['foo', null, 100], $container->getDefinition('.debug.foo')->getDecoratedService());`
`42`	`42`	`}`
`43`	`43`
`44`	`44`	`public function testItRegistersDebugHttpClientToCollector()`