Added tests for all variants

doctrine · Jan 11, 2016 · f471a0f · f471a0f
1 parent d7bbd30
commit f471a0f
Showing 1 changed file with 23 additions and 5 deletions.
diff --git a/tests/Doctrine/Tests/DBAL/Functional/Driver/PDOPgSql/DriverTest.php b/tests/Doctrine/Tests/DBAL/Functional/Driver/PDOPgSql/DriverTest.php
@@ -5,6 +5,7 @@
 use Doctrine\DBAL\Connection;
 use Doctrine\DBAL\Driver\PDOPgSql\Driver;
 use Doctrine\Tests\DBAL\Functional\Driver\AbstractDriverTest;
+use Doctrine\Tests\TestUtil;
 
 class DriverTest extends AbstractDriverTest
 {
@@ -21,12 +22,14 @@ protected function setUp()
         }
     }
 
-    public function testDefaultDatabaseOption()
+    /**
+    @dataProvider getDatabaseParameter
+     */
+    public function testDatabaseParameters($databaseName, $defaultDatabaseName, $expectedDatabaseName)
     {
         $params = $this->_conn->getParams();
-        $dbName = $params['dbname'];
-        $params['default_dbname'] = $dbName;
-        unset($params['dbname']);
+        $params['dbname'] = $databaseName;
+        $params['default_dbname'] = $defaultDatabaseName;
 
         $connection = new Connection(
             $params,
@@ -36,11 +39,26 @@ public function testDefaultDatabaseOption()
         );
 
         $this->assertSame(
-            $dbName,
+            $expectedDatabaseName,
             $this->driver->getDatabase($connection)
         );
     }
 
+    public function getDatabaseParameter()
+    {
+        $params = TestUtil::getConnection()->getParams();
+        $realDatabaseName = $params['dbname'];
+        $dummyDatabaseName = $realDatabaseName . 'a';
+
+        return array(
+            // dbname, default_dbname, expected
+            array($realDatabaseName, null, $realDatabaseName),
+            array($realDatabaseName, $dummyDatabaseName, $realDatabaseName),
+            array(null, $realDatabaseName, $realDatabaseName),
+            array(null, null, $this->getDatabaseNameForConnectionWithoutDatabaseNameParameter()),
+        );
+    }
+
     /**
      * @group DBAL-1146
      */