Merge pull request #16420 from owncloud/update-useconfigtableprefix
Add unit test for migrator with different prefix
This commit is contained in:
commit
5f4e0863f5
1 changed files with 28 additions and 1 deletions
|
@ -26,11 +26,17 @@ class Migrator extends \Test\TestCase {
|
|||
*/
|
||||
private $manager;
|
||||
|
||||
/**
|
||||
* @var IConfig
|
||||
**/
|
||||
private $config;
|
||||
|
||||
private $tableName;
|
||||
|
||||
protected function setUp() {
|
||||
parent::setUp();
|
||||
|
||||
$this->config = \OC::$server->getConfig();
|
||||
$this->connection = \OC_DB::getConnection();
|
||||
if ($this->connection->getDatabasePlatform() instanceof OraclePlatform) {
|
||||
$this->markTestSkipped('DB migration tests are not supported on OCI');
|
||||
|
@ -39,7 +45,7 @@ class Migrator extends \Test\TestCase {
|
|||
$this->markTestSkipped('DB migration tests are not supported on MSSQL');
|
||||
}
|
||||
$this->manager = new \OC\DB\MDB2SchemaManager($this->connection);
|
||||
$this->tableName = strtolower($this->getUniqueID('oc_test_'));
|
||||
$this->tableName = strtolower($this->getUniqueID($this->config->getSystemValue('dbtableprefix', 'oc_') . 'test_'));
|
||||
}
|
||||
|
||||
protected function tearDown() {
|
||||
|
@ -109,6 +115,27 @@ class Migrator extends \Test\TestCase {
|
|||
$this->assertTrue(true);
|
||||
}
|
||||
|
||||
public function testUpgradeDifferentPrefix() {
|
||||
$oldTablePrefix = $this->config->getSystemValue('dbtableprefix', 'oc_');
|
||||
|
||||
$this->config->setSystemValue('dbtableprefix', 'ownc_');
|
||||
$this->tableName = strtolower($this->getUniqueID($this->config->getSystemValue('dbtableprefix') . 'test_'));
|
||||
|
||||
list($startSchema, $endSchema) = $this->getDuplicateKeySchemas();
|
||||
$migrator = $this->manager->getMigrator();
|
||||
$migrator->migrate($startSchema);
|
||||
|
||||
$this->connection->insert($this->tableName, array('id' => 1, 'name' => 'foo'));
|
||||
$this->connection->insert($this->tableName, array('id' => 2, 'name' => 'bar'));
|
||||
$this->connection->insert($this->tableName, array('id' => 3, 'name' => 'qwerty'));
|
||||
|
||||
$migrator->checkMigrate($endSchema);
|
||||
$migrator->migrate($endSchema);
|
||||
$this->assertTrue(true);
|
||||
|
||||
$this->config->setSystemValue('dbtableprefix', $oldTablePrefix);
|
||||
}
|
||||
|
||||
public function testInsertAfterUpgrade() {
|
||||
list($startSchema, $endSchema) = $this->getDuplicateKeySchemas();
|
||||
$migrator = $this->manager->getMigrator();
|
||||
|
|
Loading…
Reference in a new issue