chore: return array of files correctly on check

Signed-off-by: Tobias Gurtzick <[email protected]>

Author: wzrdtales

lib/walker.js

@@ -107,35 +107,40 @@ Walker.prototype = {
   up: function (options, callback) {
     const partialName = options.destination;
     const count = options.count;
-    return Promise.all([
+    const files = Promise.all([
       File.loadFromFileystem(this.directory, this.prefix, this.internals),
       File.loadFromDatabase(
         this.directory,
         this.prefix,
         this._driver,
         this.internals
       )
-    ])
-      .spread((allFiles, completedFiles) => {
-        const toRun = dbmUtil.filterUp(
-          allFiles,
-          completedFiles,
-          partialName,
-          count
-        );
+    ]).spread((allFiles, completedFiles) => {
+      const toRun = dbmUtil.filterUp(
+        allFiles,
+        completedFiles,
+        partialName,
+        count
+      );
+
+      if (toRun.length === 0) {
+        log.info(this.title + 'Nothing to run');
+      }
 
-        if (toRun.length === 0) {
-          log.info(this.title + 'Nothing to run');
-        }
+      if (this.internals.check) {
+        const toRunNames = toRun.map(migration => migration.name);
+        log.info(this.title + 'run:', toRunNames);
+        return toRunNames;
+      }
 
-        if (this.internals.check) {
-          const toRunNames = toRun.map(migration => migration.name);
-          log.info(this.title + 'run:', toRunNames);
-          return [];
-        }
+      return toRun;
+    });
 
-        return toRun;
-      })
+    if (this.internals.check) {
+      return files.nodeify(callback);
+    }
+
+    return files
       .each(file => {
         log.verbose(this.title + 'preparing to run up:', file.name);
         const _meta = file.get()._meta || {};
@@ -153,27 +158,32 @@ Walker.prototype = {
     const partialName = options.destination;
     const count = options.count;
 
-    return File.loadFromDatabase(
+    const files = File.loadFromDatabase(
       this.directory,
       this.prefix,
       this._driver,
       this.internals
-    )
-      .then(completedFiles => {
-        const toRun = dbmUtil.filterDown(completedFiles, partialName, count);
+    ).then(completedFiles => {
+      const toRun = dbmUtil.filterDown(completedFiles, partialName, count);
 
-        if (toRun.length === 0) {
-          log.info(this.title + 'Nothing to run');
-        }
+      if (toRun.length === 0) {
+        log.info(this.title + 'Nothing to run');
+      }
 
-        if (this.internals.check) {
-          const toRunNames = toRun.map(migration => migration.name);
-          log.info(this.title + 'To run:', toRunNames);
-          return [];
-        }
+      if (this.internals.check) {
+        const toRunNames = toRun.map(migration => migration.name);
+        log.info(this.title + 'run:', toRunNames);
+        return toRunNames;
+      }
 
-        return toRun;
-      })
+      return toRun;
+    });
+
+    if (this.internals.check) {
+      return files.nodeify(callback);
+    }
+
+    return files
       .each(file => {
         log.verbose(this.title + 'preparing to run down:', file.name);
         const _meta = file.get()._meta || {};