use promises at adapter getExecutedMigrationNames method

okv · Dec 27, 2019 · 55b2109 · 55b2109
1 parent 321d2d5
commit 55b2109
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 10 deletions.
diff --git a/lib/adapter.js b/lib/adapter.js
@@ -62,13 +62,11 @@ Adapter.prototype.disconnect = function() {
 };
 
 Adapter.prototype.getExecutedMigrationNames = function(callback) {
-	this.collection.find({}).toArray(function(err, docs) {
-		if (err) {
-			return callback(err);
-		}
-
-		callback(null, docs.map(function(doc) { return doc._id; }));
-	});
+	return Promise.resolve()
+		.then(() => this.collection.find({}).toArray())
+		.then((docs) => {
+			return docs.map((doc) => doc._id);
+		});
 };
 
 Adapter.prototype.markExecuted = function(name, callback) {

diff --git a/test/utils/createAdapter.js b/test/utils/createAdapter.js
@@ -19,9 +19,6 @@ module.exports = () => {
 
 	const adapter = new Adapter(defaultParams);
 
-	adapter.getExecutedMigrationNames = promisify(
-		adapter.getExecutedMigrationNames.bind(adapter)
-	);
 	adapter.helpers.dropIndexIfExists = promisify(
 		adapter.helpers.dropIndexIfExists
 	);