Merge pull request #135 from lmorchard/use-extension-for-migration-li…

…st-all Only use files with the correct extension for migrations
knex · Dec 12, 2013 · e1be421 · e1be421
2 parents 7e4913b + f2dd5f8
commit e1be421
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/lib/migrate.js b/lib/migrate.js
@@ -129,7 +129,10 @@ Migrate.prototype = {
         return Promise.promisify(fs.readdir, fs)(this.config.directory);
       })
       .then(function(migrations) {
-        return migrations.sort();
+        var ext = this.config.extension;
+        return _.filter(migrations, function (value) {
+          return value.indexOf(ext, value.length - ext.length) !== -1;
+        }).sort();
       });
   },
 

diff --git a/test/integration/migrate/test/random_bad_file.txt b/test/integration/migrate/test/random_bad_file.txt
@@ -0,0 +1 @@
+This is definitely not a migration.