minor #52689 [DependencyInjection] Fix tests on configuration prepend (alexandre-daubois)

nicolas-grekas · nicolas-grekas · commit cda729e5d4c1 · 2023-11-23T17:39:07.000+01:00
This PR was merged into the 7.1 branch. Discussion ---------- [DependencyInjection] Fix tests on configuration prepend | Q | A | ------------- | --- | Branch? | 7.1 | Bug fix? | yes | New feature? | no | Deprecations? | no | Issues | - | License | MIT Commits ------- a206da9 [DependencyInjection] Fix tests on configuration prepend
diff --git a/src/Symfony/Component/HttpKernel/Tests/DependencyInjection/MergeExtensionConfigurationPassTest.php b/src/Symfony/Component/HttpKernel/Tests/DependencyInjection/MergeExtensionConfigurationPassTest.php
@@ -48,7 +48,7 @@ public function testFooBundle()
         $configPass = new MergeExtensionConfigurationPass(['loaded', 'acme_foo']);
         $configPass->process($container);
 
-        $this->assertSame([[], ['bar' => 'baz']], $container->getExtensionConfig('loaded'), '->prependExtension() prepends an extension config');
+        $this->assertSame([['bar' => 'baz'], []], $container->getExtensionConfig('loaded'), '->prependExtension() prepends an extension config');
         $this->assertTrue($container->hasDefinition('acme_foo.foo'), '->loadExtension() registers a service');
         $this->assertTrue($container->hasDefinition('acme_foo.bar'), '->loadExtension() imports a service');
         $this->assertTrue($container->hasParameter('acme_foo.config'), '->loadExtension() sets a parameter');
diff --git a/src/Symfony/Component/HttpKernel/Tests/Fixtures/AcmeFooBundle/AcmeFooBundle.php b/src/Symfony/Component/HttpKernel/Tests/Fixtures/AcmeFooBundle/AcmeFooBundle.php
@@ -31,7 +31,7 @@ public function configure(DefinitionConfigurator $definition): void
 
     public function prependExtension(ContainerConfigurator $container, ContainerBuilder $builder): void
     {
-        $container->extension('loaded', ['bar' => 'baz'], true);
+        $builder->prependExtensionConfig('loaded', ['bar' => 'baz']);
     }
 
     public function loadExtension(array $config, ContainerConfigurator $container, ContainerBuilder $builder): void

Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ public function configure(DefinitionConfigurator $definition): void`
`31`	`31`
`32`	`32`	`public function prependExtension(ContainerConfigurator $container, ContainerBuilder $builder): void`
`33`	`33`	`{`
`34`		`- $container->extension('loaded', ['bar' => 'baz'], true);`
	`34`	`+ $builder->prependExtensionConfig('loaded', ['bar' => 'baz']);`
`35`	`35`	`}`
`36`	`36`
`37`	`37`	`public function loadExtension(array $config, ContainerConfigurator $container, ContainerBuilder $builder): void`