Use native MigrationFailedException instead of PDOMigrationFailed

Author: ZhukV

src/Exception/MigrationFailedException.php

@@ -0,0 +1,18 @@
+<?php
+
+/*
+ * This file is part of the FiveLab Migrator package
+ *
+ * (c) FiveLab
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code
+ */
+
+declare(strict_types = 1);
+
+namespace FiveLab\Component\Migrator\Exception;
+
+class MigrationFailedException extends \Exception
+{
+}

src/Exception/PdoMigrationFailedException.php

@@ -13,7 +13,7 @@
 
 namespace FiveLab\Component\Migrator\Migration;
 
-use FiveLab\Component\Migrator\Exception\PdoMigrationFailedException;
+use FiveLab\Component\Migrator\Exception\MigrationFailedException;
 
 abstract readonly class AbstractPdoMigration extends AbstractMigration
 {
@@ -65,7 +65,7 @@ final protected function addSql(string $sql, array $parameters = []): void
      *
      * @param array{"0": string, 1: array<string|int, string|int|float>} $entry
      *
-     * @throws PdoMigrationFailedException
+     * @throws MigrationFailedException
      */
     private function executeEntry(array $entry): void
     {
@@ -74,7 +74,14 @@ private function executeEntry(array $entry): void
         try {
             $stmt->execute($entry[1]);
         } catch (\Throwable $error) {
-            throw new PdoMigrationFailedException($entry[0], $entry[1], $error);
+            $message = \sprintf(
+                'Migration failed - "%s", parameters: %s with message: %s.',
+                $entry[0],
+                \json_encode($entry[1], \JSON_THROW_ON_ERROR),
+                \rtrim($error->getMessage(), '.')
+            );
+
+            throw new MigrationFailedException($message, 0, $error);
         }
     }
 }

src/Migration/AbstractPdoMigration.php

@@ -13,7 +13,7 @@
 
 namespace FiveLab\Component\Migrator\Tests\Functional\Migrator;
 
-use FiveLab\Component\Migrator\Exception\PdoMigrationFailedException;
+use FiveLab\Component\Migrator\Exception\MigrationFailedException;
 use FiveLab\Component\Migrator\Locator\FilesystemMigrationsLocator;
 use FiveLab\Component\Migrator\MigrateDirection;
 use FiveLab\Component\Migrator\Migrator;
@@ -132,7 +132,7 @@ public function shouldSuccessWrapPdoException(): void
     {
         $migrator = $this->createMigrator(__DIR__.'/../../Migrations/DataSet03', 'Database');
 
-        $this->expectException(PdoMigrationFailedException::class);
+        $this->expectException(MigrationFailedException::class);
         $this->expectExceptionMessage('Migration failed - "SELECT * FROM not_existing_table WHERE label = :foo", parameters: {"foo":"bar"} with message: SQLSTATE[42S02]: Base table or view not found:');
 
         $migrator->migrate(MigrateDirection::Up, null);