Confinguration: check if migration class exists added

lib/Doctrine/DBAL/Migrations/Configuration/Configuration.php

@@ -22,6 +22,7 @@
 use Doctrine\DBAL\Connection;
 use Doctrine\DBAL\Migrations\Finder\MigrationDeepFinderInterface;
 use Doctrine\DBAL\Migrations\MigrationException;
+use Doctrine\DBAL\Migrations\MigrationClassNotFoundException;
 use Doctrine\DBAL\Migrations\OutputWriter;
 use Doctrine\DBAL\Migrations\Version;
 use Doctrine\DBAL\Migrations\Finder\MigrationFinderInterface;
@@ -371,6 +372,8 @@ public function registerMigrationsFromDirectory($path)
      */
     public function registerMigration($version, $class)
     {
+        $this->ensureMigrationClassExists($class);
+
         $version = (string) $version;
         $class = (string) $class;
         if (isset($this->migrations[$version])) {
@@ -779,4 +782,20 @@ private function shouldExecuteMigration($direction, Version $version, $to, $migr
             return $version->getVersion() <= $to;
         }
     }
+
+    /**
+     * @param string $class
+     */
+    private function ensureMigrationClassExists($class)
+    {
+        if ( ! class_exists($class)) {
+            throw new MigrationClassNotFoundException(
+                sprintf(
+                    'Migration class "%s" was not found. Is it placed in "%s" namespace?',
+                    $class,
+                    $this->getMigrationsNamespace()
+                )
+            );
+        }
+    }
 }

lib/Doctrine/DBAL/Migrations/MigrationClassNotFoundException.php

@@ -0,0 +1,26 @@
+<?php
+/*
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+ * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+ * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+ * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+ * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+ * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+ * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ *
+ * This software consists of voluntary contributions made by many individuals
+ * and is licensed under the LGPL. For more information, see
+ * <http://www.doctrine-project.org>.
+ */
+
+namespace Doctrine\DBAL\Migrations;
+
+use Exception;
+
+class MigrationClassNotFoundException extends Exception
+{
+}

tests/Doctrine/DBAL/Migrations/Tests/Configuration/ConfigurationTest.php

@@ -37,6 +37,20 @@ public function testOutputWriterCanBeSet()
         $this->assertSame($outputWriter, $configuration->getOutputWriter());
     }
 
+    public function testRegisterMigrationsClassExistCheck()
+    {
+        $migrationsDir = __DIR__ . '/ConfigurationTestSource/Migrations';
+
+        $configuration = new Configuration($this->getConnectionMock());
+        $configuration->setMigrationsNamespace('Migrations');
+        $configuration->setMigrationsDirectory($migrationsDir);
+
+        $this->setExpectedException(
+            'Doctrine\DBAL\Migrations\MigrationClassNotFoundException',
+            'Migration class "Migrations\Version123" was not found. Is it placed in "Migrations" namespace?');
+        $configuration->registerMigrationsFromDirectory($migrationsDir);
+    }
+
     /**
      * @return \PHPUnit_Framework_MockObject_MockObject|Connection
      */

tests/Doctrine/DBAL/Migrations/Tests/Configuration/ConfigurationTestSource/Migrations/Version123.php

@@ -0,0 +1,8 @@
+<?php
+
+namespace Migrations\IncorrectNamespace;
+
+class Version123
+{
+
+}