From f9b4daf1e44cd95af6f1ac02d2ed4970c950464b Mon Sep 17 00:00:00 2001 From: William DURAND Date: Sun, 30 Jan 2011 22:36:59 +0100 Subject: [PATCH] Updated unit tests --- .../DependencyInjection/PropelExtensionTest.php | 17 ++++++++--------- Tests/TestCase.php | 2 +- 2 files changed, 9 insertions(+), 10 deletions(-) diff --git a/Tests/DependencyInjection/PropelExtensionTest.php b/Tests/DependencyInjection/PropelExtensionTest.php index 60e7c30..5ed5fc5 100644 --- a/Tests/DependencyInjection/PropelExtensionTest.php +++ b/Tests/DependencyInjection/PropelExtensionTest.php @@ -23,16 +23,16 @@ class PropelExtensionTest extends TestCase $loader = new PropelExtension(); try { - $loader->configLoad(array(), $container); + $loader->configLoad(array(array()), $container); $this->fail(); } catch (\Exception $e) { $this->assertInstanceOf('InvalidArgumentException', $e, '->configLoad() throws an \InvalidArgumentException if the Propel path is not set.'); } - $loader->configLoad(array('path' => '/propel'), $container); + $loader->configLoad(array(array('path' => '/propel')), $container); $this->assertEquals('/propel', $container->getParameter('propel.path'), '->configLoad() sets the Propel path'); - $loader->configLoad(array(), $container); + $loader->configLoad(array(array()), $container); $this->assertEquals('/propel', $container->getParameter('propel.path'), '->configLoad() sets the Propel path'); } @@ -41,26 +41,25 @@ class PropelExtensionTest extends TestCase $container = new ContainerBuilder(); $loader = new PropelExtension(); - $loader->dbalLoad(array(), $container); + $loader->dbalLoad(array(array()), $container); $this->assertEquals('Propel', $container->getParameter('propel.class'), '->dbalLoad() loads the propel.xml file if not already loaded'); // propel.dbal.default_connection $this->assertEquals('default', $container->getParameter('propel.dbal.default_connection'), '->dbalLoad() overrides existing configuration options'); - $loader->dbalLoad(array('default_connection' => 'foo'), $container); - $this->assertEquals('foo', $container->getParameter('propel.dbal.default_connection'), '->dbalLoad() overrides existing configuration options'); - $loader->dbalLoad(array(), $container); + $loader->dbalLoad(array(array('default_connection' => 'foo')), $container); $this->assertEquals('foo', $container->getParameter('propel.dbal.default_connection'), '->dbalLoad() overrides existing configuration options'); $container = new ContainerBuilder(); $loader = new PropelExtension(); - $loader->dbalLoad(array('password' => 'foo'), $container); + + $loader->dbalLoad(array(array('password' => 'foo')), $container); $arguments = $container->getDefinition('propel.configuration')->getArguments(); $config = $arguments[0]; $this->assertEquals('foo', $config['datasources']['default']['connection']['password']); $this->assertEquals('root', $config['datasources']['default']['connection']['user']); - $loader->dbalLoad(array('user' => 'foo'), $container); + $loader->dbalLoad(array(array('user' => 'foo')), $container); $this->assertEquals('foo', $config['datasources']['default']['connection']['password']); $this->assertEquals('root', $config['datasources']['default']['connection']['user']); } diff --git a/Tests/TestCase.php b/Tests/TestCase.php index 2e4719b..3ca374b 100644 --- a/Tests/TestCase.php +++ b/Tests/TestCase.php @@ -15,7 +15,7 @@ class TestCase extends \PHPUnit_Framework_TestCase { public function setUp() { - if (!file_exists($file = __DIR__.'/../../../../../vendor/propel/runtime/lib/Propel.php')) { + if (!file_exists($file = __DIR__.'/../../../../../../../vendor/propel/runtime/lib/Propel.php')) { $this->markTestSkipped('Propel is not available.'); }