Skip to content

Commit

Permalink
Add migration version and name to pending actions error (#2289)
Browse files Browse the repository at this point in the history
  • Loading branch information
ajibarra committed Jun 10, 2024
1 parent ef38e41 commit 5830da2
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion src/Phinx/Migration/AbstractMigration.php
Original file line number Diff line number Diff line change
Expand Up @@ -358,7 +358,7 @@ public function postFlightCheck(): void
{
foreach ($this->tables as $table) {
if ($table->hasPendingActions()) {
throw new RuntimeException('Migration has pending actions after execution!');
throw new RuntimeException(sprintf('Migration %s_%s has pending actions after execution!', $this->getVersion(), $this->getName()));
}
}
}
Expand Down
4 changes: 2 additions & 2 deletions tests/Phinx/Migration/AbstractMigrationTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -265,7 +265,7 @@ public function testTableMethod()
public function testPostFlightCheckFail()
{
// stub migration
$migrationStub = $this->getMockForAbstractClass('\Phinx\Migration\AbstractMigration', ['mockenv', 20230102030405]);
$migrationStub = $this->getMockForAbstractClass('\Phinx\Migration\AbstractMigration', ['mockenv', 20230102030405], 'PostFlightCheck');

$adapterStub = $this->getMockBuilder('\Phinx\Db\Adapter\MysqlAdapter')
->setConstructorArgs([[]])
Expand All @@ -280,7 +280,7 @@ public function testPostFlightCheckFail()
$table->addColumn('column1', 'integer', ['null' => true]);

$this->expectException(RuntimeException::class);
$this->expectExceptionMessage('Migration has pending actions after execution!');
$this->expectExceptionMessage('Migration 20230102030405_PostFlightCheck has pending actions after execution!');

$migrationStub->postFlightCheck();
}
Expand Down

0 comments on commit 5830da2

Please sign in to comment.