Migration testing

Author: phpfui

Tests/Fixtures/Migration/Migration_1.php

@@ -0,0 +1,21 @@
+<?php
+
+namespace Tests\Fixtures\Migration;
+
+class Migration_1 extends \PHPFUI\ORM\Migration
+	{
+	public function description() : string
+		{
+		return 'Crash Test Dummy';
+		}
+
+	public function down() : bool
+		{
+		return true;
+		}
+
+	public function up() : bool
+		{
+		return true;
+		}
+	}

Tests/Fixtures/Migration/Migration_2.php

@@ -0,0 +1,21 @@
+<?php
+
+namespace Tests\Fixtures\Migration;
+
+class Migration_2 extends \PHPFUI\ORM\Migration
+	{
+	public function description() : string
+		{
+		return 'Dummy #2';
+		}
+
+	public function down() : bool
+		{
+		return true;
+		}
+
+	public function up() : bool
+		{
+		return true;
+		}
+	}

Tests/Fixtures/Migration/Migration_3.php

@@ -0,0 +1,21 @@
+<?php
+
+namespace Tests\Fixtures\Migration;
+
+class Migration_3 extends \PHPFUI\ORM\Migration
+	{
+	public function description() : string
+		{
+		return 'Another dummy';
+		}
+
+	public function down() : bool
+		{
+		return true;
+		}
+
+	public function up() : bool
+		{
+		return true;
+		}
+	}

Tests/Unit/MigrationTest.php

@@ -0,0 +1,26 @@
+<?php
+
+namespace Tests\Unit;
+
+class MigrationTest extends \PHPUnit\Framework\TestCase
+	{
+	public function testMigrations() : void
+		{
+		$transaction = new \PHPFUI\ORM\Transaction();
+		$tables = \PHPFUI\ORM::getTables();
+		$this->assertContains('migration', $tables);
+		$migrator = new \PHPFUI\ORM\Migrator();
+		$migrator->migrateUpOne();
+		$migrationTable = new \PHPFUI\ORM\Table\Migration();
+		$this->assertCount(1, $migrationTable);
+		$migrator->migrateUpOne();
+		$this->assertCount(2, $migrationTable);
+		$migrator->migrateDownOne();
+		$this->assertCount(1, $migrationTable);
+		$tables = \PHPFUI\ORM::getTables();
+		$migrator->migrateDownOne();
+		$this->assertCount(0, $migrationTable);
+		$migrator->migrate();
+		$this->assertCount(3, $migrationTable);
+		}
+	}

Tests/bootstrap.php

@@ -35,7 +35,7 @@ function autoload(string $className) : void
 \PHPFUI\ORM::$namespaceRoot = __DIR__ . '/..';
 \PHPFUI\ORM::$recordNamespace = 'Tests\\App\\Record';
 \PHPFUI\ORM::$tableNamespace = 'Tests\\App\\Table';
-\PHPFUI\ORM::$migrationNamespace = 'Tests\\App\\Migration';
+\PHPFUI\ORM::$migrationNamespace = 'Tests\\Fixtures\\Migration';
 \PHPFUI\ORM::$idSuffix = '_id';
 
 \PHPFUI\ORM::setTranslationCallback(\PHPFUI\Translation\Translator::trans(...));

northwind/northwind-default.sql

@@ -516,6 +516,9 @@ create table dateRecord (
   timestampDefaultCurrentNotNull timestamp not null default CURRENT_TIMESTAMP,
   PRIMARY KEY (dateRecordId));
 
+drop table if exists migration;
+create table migration (migrationId int NOT NULL primary key, ran TIMESTAMP DEFAULT CURRENT_TIMESTAMP);
+
 SET SQL_MODE=@OLD_SQL_MODE;
 SET FOREIGN_KEY_CHECKS=@OLD_FOREIGN_KEY_CHECKS;
 SET UNIQUE_CHECKS=@OLD_UNIQUE_CHECKS;

northwind/northwind-sqlite.sql

@@ -505,3 +505,7 @@ create table dateRecord (
   dateDefaultNotNull date not null default '2000-01-02',
   timestampDefaultCurrentNullable timestamp DEFAULT CURRENT_TIMESTAMP,
   timestampDefaultCurrentNotNull timestamp not null default CURRENT_TIMESTAMP);
+
+drop table if exists migration;
+create table migration (migrationId int NOT NULL primary key, ran TIMESTAMP DEFAULT CURRENT_TIMESTAMP);
+

northwind/northwind.db

@@ -57,23 +57,7 @@ public function executeAlters() : bool
 	 */
 	public function getAllTables(string $type = 'BASE TABLE') : array
 		{
-		$where = '';
-
-		if ($type)
-			{
-			$where = " where Table_Type = '{$type}'";
-			}
-
-		$tableArray = \PHPFUI\ORM::getRows('show full tables' . $where);
-
-		$tables = [];
-
-		foreach ($tableArray as $row)
-			{
-			$tables[] = \array_shift($row);
-			}
-
-		return $tables;
+		return \PHPFUI\ORM::getTables();
 		}
 
 	/** @return string[] */
@@ -500,18 +484,17 @@ private function alter(string $type, string $table, string $field, string $extra
 		$this->alters[$table][] = $sql;
 		}
 
-	private function getFieldInfo(string $table, string $field) : array
+	private function getFieldInfo(string $table, string $field) : ?PHPFUI\ORM\Schema\Field
 		{
-		$rows = \PHPFUI\ORM::getArrayCursor("SHOW COLUMNS FROM `{$table}`");
-
-		foreach ($rows as $row)
+		$fields = \PHPFUI\ORM::describeTable($table);
+		foreach ($fields as $field)
 			{
-			if (0 == \strcasecmp((string)$row['Field'], $field))
+			if ($field->name == $field)
 				{
-				return $row;
+				return $field;
 				}
 			}
 
-		return [];
+		return null;
 		}
 	}

src/PHPFUI/ORM/Migration.php

@@ -16,8 +16,8 @@ abstract class Migration extends \PHPFUI\ORM\Record
 	/** @var array<string, array<mixed>> */
 	protected static array $fields = [
 		// MYSQL_TYPE, PHP_TYPE, LENGTH, KEY, ALLOWS_NULL, DEFAULT
-		'migrationId' => ['int(11)', 'int', 11, true, false, ],
-		'ran' => ['timestamp', 'string', 20, false, false, ],
+		'migrationId' => ['int(11)', 'int', 11, true, false, null, ],
+		'ran' => ['timestamp', 'string', 20, false, false, null, ],
 	];
 
 	/** @var array<string, true> */