TraceableHttpClient: increase decorator's priority

adpeyre · nicolas-grekas · commit 772129f800fc · 2022-12-13T12:07:37.000+01:00
diff --git a/DependencyInjection/HttpClientPass.php b/DependencyInjection/HttpClientPass.php
@@ -43,7 +43,7 @@ public function process(ContainerBuilder $container)
             $container->register('.debug.'.$id, TraceableHttpClient::class)
                 ->setArguments([new Reference('.debug.'.$id.'.inner'), new Reference('debug.stopwatch', ContainerInterface::IGNORE_ON_INVALID_REFERENCE)])
                 ->addTag('kernel.reset', ['method' => 'reset'])
-                ->setDecoratedService($id);
+                ->setDecoratedService($id, null, 5);
             $container->getDefinition('data_collector.http_client')
                 ->addMethodCall('registerClient', [$id, new Reference('.debug.'.$id)]);
         }
diff --git a/Tests/DependencyInjection/HttpClientPassTest.php b/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, 5], $container->getDefinition('.debug.foo')->getDecoratedService());
     }
 
     public function testItRegistersDebugHttpClientToCollector()

Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ public function process(ContainerBuilder $container)`
`43`	`43`	`$container->register('.debug.'.$id, TraceableHttpClient::class)`
`44`	`44`	`->setArguments([new Reference('.debug.'.$id.'.inner'), new Reference('debug.stopwatch', ContainerInterface::IGNORE_ON_INVALID_REFERENCE)])`
`45`	`45`	`->addTag('kernel.reset', ['method' => 'reset'])`
`46`		`- ->setDecoratedService($id);`
	`46`	`+ ->setDecoratedService($id, null, 5);`
`47`	`47`	`$container->getDefinition('data_collector.http_client')`
`48`	`48`	`->addMethodCall('registerClient', [$id, new Reference('.debug.'.$id)]);`
`49`	`49`	`}`
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, 5], $container->getDefinition('.debug.foo')->getDecoratedService());`
`42`	`42`	`}`
`43`	`43`
`44`	`44`	`public function testItRegistersDebugHttpClientToCollector()`