Marked the ResolveParameterPlaceHoldersPassTest as legacy

saro0h · saro0h · commit 74c589e6b893 · 2015-03-16T16:27:33.000+01:00
diff --git a/src/Symfony/Component/DependencyInjection/Tests/Compiler/LegacyResolveParameterPlaceHoldersPassTest.php b/src/Symfony/Component/DependencyInjection/Tests/Compiler/LegacyResolveParameterPlaceHoldersPassTest.php
@@ -0,0 +1,51 @@
+<?php
+
+/*
+ * This file is part of the Symfony package.
+ *
+ * (c) Fabien Potencier <fabien@symfony.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Symfony\Component\DependencyInjection\Tests\Compiler;
+
+use Symfony\Component\DependencyInjection\Compiler\ResolveParameterPlaceHoldersPass;
+use Symfony\Component\DependencyInjection\ContainerBuilder;
+
+/**
+ * @group legacy
+ */
+class LegacyResolveParameterPlaceHoldersPassTest extends \PHPUnit_Framework_TestCase
+{
+    private $compilerPass;
+    private $container;
+    private $fooDefinition;
+
+    protected function setUp()
+    {
+        $this->iniSet('error_reporting', -1 & ~E_USER_DEPRECATED);
+
+        $this->compilerPass = new ResolveParameterPlaceHoldersPass();
+        $this->container = $this->createContainerBuilder();
+        $this->compilerPass->process($this->container);
+        $this->fooDefinition = $this->container->getDefinition('foo');
+    }
+
+    private function createContainerBuilder()
+    {
+        $containerBuilder = new ContainerBuilder();
+        $containerBuilder->setParameter('foo.class', 'Foo');
+        $containerBuilder->setParameter('foo.factory.class', 'FooFactory');
+        $fooDefinition = $containerBuilder->register('foo', '%foo.class%');
+        $fooDefinition->setFactoryClass('%foo.factory.class%');
+
+        return $containerBuilder;
+    }
+
+    public function testFactoryClassParametersShouldBeResolved()
+    {
+        $this->assertSame('FooFactory', $this->fooDefinition->getFactoryClass());
+    }
+}
diff --git a/src/Symfony/Component/DependencyInjection/Tests/Compiler/ResolveParameterPlaceHoldersPassTest.php b/src/Symfony/Component/DependencyInjection/Tests/Compiler/ResolveParameterPlaceHoldersPassTest.php
@@ -33,9 +33,9 @@ public function testClassParametersShouldBeResolved()
         $this->assertSame('Foo', $this->fooDefinition->getClass());
     }
 
-    public function testFactoryClassParametersShouldBeResolved()
+    public function testFactoryParametersShouldBeResolved()
     {
-        $this->assertSame('FooFactory', $this->fooDefinition->getFactoryClass());
+        $this->assertSame(array('FooFactory', 'bar'), $this->fooDefinition->getFactory());
     }
 
     public function testArgumentParametersShouldBeResolved()
@@ -78,7 +78,7 @@ private function createContainerBuilder()
         $containerBuilder->setParameter('alias.id', 'bar');
 
         $fooDefinition = $containerBuilder->register('foo', '%foo.class%');
-        $fooDefinition->setFactoryClass('%foo.factory.class%');
+        $fooDefinition->setFactory(array('%foo.factory.class%', 'bar'));
         $fooDefinition->setArguments(array('%foo.arg1%', '%foo.arg2%'));
         $fooDefinition->addMethodCall('%foo.method%', array('%foo.arg1%', '%foo.arg2%'));
         $fooDefinition->setProperty('%foo.property.name%', '%foo.property.value%');

Original file line number	Diff line number	Diff line change
`@@ -33,9 +33,9 @@ public function testClassParametersShouldBeResolved()`
`33`	`33`	`$this->assertSame('Foo', $this->fooDefinition->getClass());`
`34`	`34`	`}`
`35`	`35`
`36`		`- public function testFactoryClassParametersShouldBeResolved()`
	`36`	`+ public function testFactoryParametersShouldBeResolved()`
`37`	`37`	`{`
`38`		`- $this->assertSame('FooFactory', $this->fooDefinition->getFactoryClass());`
	`38`	`+ $this->assertSame(array('FooFactory', 'bar'), $this->fooDefinition->getFactory());`
`39`	`39`	`}`
`40`	`40`
`41`	`41`	`public function testArgumentParametersShouldBeResolved()`
`@@ -78,7 +78,7 @@ private function createContainerBuilder()`
`78`	`78`	`$containerBuilder->setParameter('alias.id', 'bar');`
`79`	`79`
`80`	`80`	`$fooDefinition = $containerBuilder->register('foo', '%foo.class%');`
`81`		`- $fooDefinition->setFactoryClass('%foo.factory.class%');`
	`81`	`+ $fooDefinition->setFactory(array('%foo.factory.class%', 'bar'));`
`82`	`82`	`$fooDefinition->setArguments(array('%foo.arg1%', '%foo.arg2%'));`
`83`	`83`	`$fooDefinition->addMethodCall('%foo.method%', array('%foo.arg1%', '%foo.arg2%'));`
`84`	`84`	`$fooDefinition->setProperty('%foo.property.name%', '%foo.property.value%');`