Skip to content

Commit

Permalink
test: use variable $table
Browse files Browse the repository at this point in the history
  • Loading branch information
kenjis committed Jan 30, 2024
1 parent 40a84c8 commit 1704a93
Showing 1 changed file with 10 additions and 8 deletions.
18 changes: 10 additions & 8 deletions tests/system/Database/Live/SQLite3/ForgeModifyColumnTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,9 @@ protected function setUp(): void

public function testModifyColumnRename(): void
{
$this->forge->dropTable('forge_test_three', true);
$table = 'forge_test_three';

$this->forge->dropTable($table, true);

$this->forge->addField([
'id' => [
Expand Down Expand Up @@ -71,11 +73,11 @@ public function testModifyColumnRename(): void
]);

$this->forge->addKey('id', true);
$this->forge->createTable('forge_test_three');
$this->forge->createTable($table);

$this->assertTrue($this->db->fieldExists('name', 'forge_test_three'));
$this->assertTrue($this->db->fieldExists('name', $table));

$this->forge->modifyColumn('forge_test_three', [
$this->forge->modifyColumn($table, [
'name' => [
'name' => 'altered',
'type' => 'VARCHAR',
Expand All @@ -86,15 +88,15 @@ public function testModifyColumnRename(): void

$this->db->resetDataCache();

$fieldData = $this->db->getFieldData('forge_test_three');
$fieldData = $this->db->getFieldData($table);
$fields = [];

foreach ($fieldData as $obj) {
$fields[$obj->name] = $obj;
}

$this->assertFalse($this->db->fieldExists('name', 'forge_test_three'));
$this->assertTrue($this->db->fieldExists('altered', 'forge_test_three'));
$this->assertFalse($this->db->fieldExists('name', $table));
$this->assertTrue($this->db->fieldExists('altered', $table));

$this->assertFalse($fields['int']->nullable);
$this->assertSame('0', $fields['int']->default);
Expand All @@ -108,6 +110,6 @@ public function testModifyColumnRename(): void
$this->assertTrue($fields['altered']->nullable);
$this->assertNull($fields['altered']->default);

$this->forge->dropTable('forge_test_three', true);
$this->forge->dropTable($table, true);
}
}

0 comments on commit 1704a93

Please sign in to comment.