Merge pull request #24 from Insolita/fix_bug_with_empty_migrations

bugfix #3

Author: cebe

src/lib/MigrationBuilder.php

@@ -95,8 +95,12 @@ public function buildFresh():MigrationModel
         $this->migration = new MigrationModel($this->model, true);
         $this->uniqueColumns = $this->model->getUniqueColumnsList();
         $this->newColumns = $this->model->attributesToColumnSchema();
+        if (empty($this->newColumns)) {
+            return $this->migration;
+        }
         $builder = $this->recordBuilder;
         $tableName = $this->model->getTableAlias();
+
         $this->migration->addUpCode($builder->createTable($tableName, $this->newColumns, $this->uniqueColumns))
                         ->addDownCode($builder->dropTable($tableName));
         if ($this->isPostgres) {

src/lib/MigrationsGenerator.php

@@ -49,7 +49,7 @@ public function generate(array $models):array
                 $this->migrations[$model->tableAlias] = $migration;
             }
         }
-        return $this->sortMigrationsByDeps();
+        return !empty($this->migrations) ? $this->sortMigrationsByDeps() : [];
     }
 
     /**

tests/unit/MigrationsGeneratorTest.php

@@ -16,6 +16,15 @@
 class MigrationsGeneratorTest extends TestCase
 {
 
+    public function testNoMigrations()
+    {
+        $this->prepareTempDir();
+        $this->mockApplication($this->mockDbSchemaAsEmpty());
+        $model = new DbModel(['name' => 'dummy', 'tableName' => 'dummy', 'attributes' => []]);
+        $generator = new MigrationsGenerator();
+        $migrations = $generator->generate([$model]);
+        self::assertEmpty($migrations);
+    }
     /**
      * @dataProvider simpleDbModelsProvider
      * @param array|DbModel[]        $dbModels