Merge pull request #127 from daniel-de-wit/main

Support connect_via_database

Author: freekmurze

src/DbDumperFactory.php

@@ -32,9 +32,11 @@ public static function createForConnection(string $connectionName): DbDumper
             $fallback
         );
 
+        $dbName = $dbConfig['connect_via_database'] ?? $dbConfig['database'];
+
         $dbDumper = static::forDriver($dbConfig['driver'])
             ->setHost($dbHost ?? '')
-            ->setDbName($dbConfig['database'])
+            ->setDbName($dbName)
             ->setUserName($dbConfig['username'] ?? '')
             ->setPassword($dbConfig['password'] ?? '');
 

tests/DbDumperFactoryTest.php

@@ -71,6 +71,25 @@ public function it_will_use_the_read_db_when_one_is_defined()
         $this->assertEquals('localhost-read', $dumper->getHost());
     }
 
+    /** @test */
+    public function it_will_use_connect_via_database_when_one_is_defined()
+    {
+        $dbConfig = [
+            'driver' => 'pgsql',
+            'connect_via_database' => 'connection_pool',
+            'username' => 'root',
+            'password' => 'myPassword',
+            'database' => 'myDb',
+            'dump' => ['add_extra_option' => '--extra-option=value'],
+        ];
+
+        $this->app['config']->set('database.connections.pgsql', $dbConfig);
+
+        $dumper = DbDumperFactory::createForConnection('pgsql');
+
+        $this->assertEquals('connection_pool', $dumper->getDbName());
+    }
+
     /** @test */
     public function it_will_throw_an_exception_when_creating_an_unknown_type_of_dumper()
     {