Merge pull request #7 from vierge-noire/cake3_next

Alias connections before running migrations

Author: pabloelcolombiano

src/Migrator.php

@@ -46,6 +46,10 @@ public static function migrate(array $config = [])
     {
         $migrator = new static();
 
+        // Make sure that the connections are aliased, in case
+        // the migrations invoke the table registry.
+        $migrator->getFixtureManager()->aliasConnections();
+
         $migrator
             ->prepareConfig($config)
             ->dropTablesForMissingMigrations()
@@ -129,4 +133,4 @@ protected function getFixtureManager(): FixtureManager
     {
         return $this->fixtureManager;
     }
-}
+}

tests/TestApp/config/Migrations/20200208100000_app_migration.php

@@ -17,7 +17,16 @@
 class AppMigration extends AbstractMigration
 {
     public function up()
-    {}
+    {
+        $this->table('articles')
+            ->addPrimaryKey(['id'])
+            ->addColumn('title', 'string', [
+                'limit' => 128,
+                'null' => false,
+            ])
+            ->addTimestamps('created', 'modified')
+            ->create();
+    }
 
     public function down()
     {}

tests/TestApp/src/Model/Table/ArticlesTable.php

@@ -0,0 +1,20 @@
+<?php
+declare(strict_types=1);
+
+/**
+ * Licensed under The MIT License
+ * For full copyright and license information, please see the LICENSE.txt
+ * Redistributions of files must retain the above copyright notice.
+ *
+ * @copyright     Copyright (c) 2020 Juan Pablo Ramirez and Nicolas Masson
+ * @link          https://webrider.de/
+ * @since         1.0.0
+ * @license       http://www.opensource.org/licenses/mit-license.php MIT License
+ */
+namespace TestApp\Model\Table;
+
+use Cake\ORM\Query;
+use Cake\ORM\Table;
+
+class ArticlesTable extends Table
+{}

tests/TestCase/MigratorTest.php

@@ -15,6 +15,7 @@
 
 
 use Cake\Datasource\ConnectionManager;
+use Cake\ORM\TableRegistry;
 use Cake\TestSuite\TestCase;
 use CakephpTestMigrator\Migrator;
 
@@ -64,4 +65,11 @@ public function testMigrate()
         $this->assertSame(['FooMigration'], $fooPluginMigrations);
         $this->assertSame(['BarMigration'], $barPluginMigrations);
     }
-}
+
+    public function testTableRegistryConnectionName()
+    {
+        $Articles = TableRegistry::getTableLocator()->get('Articles');
+        ConnectionManager::getConfigOrFail('default');
+        $this->assertSame('test', $Articles->getConnection()->configName());
+    }
+}

tests/bootstrap.php

@@ -1,6 +1,7 @@
 <?php
 
 use Cake\Core\Configure;
+use Cake\Datasource\ConnectionManager;
 
 if (!defined('DS')) {
     define('DS', DIRECTORY_SEPARATOR);
@@ -84,13 +85,17 @@ function loadPHPUnitAliases()
     ],
 ];
 
-\Cake\Datasource\ConnectionManager::setConfig('test', $dbConnection);
+ConnectionManager::setConfig('test', $dbConnection);
+
+$dbDefaultConnection = $dbConnection;
+$dbDefaultConnection['database'] = 'migrator';
+ConnectionManager::setConfig('default', $dbConnection);
 
 $dbConnection['migrations'] = ['plugin' => 'BarPlugin'];
-\Cake\Datasource\ConnectionManager::setConfig('test_2', $dbConnection);
+ConnectionManager::setConfig('test_2', $dbConnection);
 
 $dbConnection['migrations'] = true;
-\Cake\Datasource\ConnectionManager::setConfig('test_3', $dbConnection);
+ConnectionManager::setConfig('test_3', $dbConnection);
 
 \Cake\Core\Plugin::load('FooPlugin');
-\Cake\Core\Plugin::load('BarPlugin');
+\Cake\Core\Plugin::load('BarPlugin');