Switch around Promise resolution and completion events to ensure that, when using awaitCompleted, the execution order makes sense

master
Sven Slootweg 8 years ago
parent 01771c794b
commit 9514f5db8d

@ -80,8 +80,8 @@ module.exports = function createTaskQueue(options) {
Promise.try(function () { Promise.try(function () {
return handlers[type](task.data); return handlers[type](task.data);
}).then(function (result) { }).then(function (result) {
markSuccess(type, task);
task.resolve(result); task.resolve(result);
markSuccess(type, task);
}).catch(function (err) { }).catch(function (err) {
markFailed(type, task); markFailed(type, task);
task.reject(err); task.reject(err);

@ -80,8 +80,8 @@ module.exports = function createTaskQueue(options) {
Promise.try(() => { Promise.try(() => {
return handlers[type](task.data); return handlers[type](task.data);
}).then((result) => { }).then((result) => {
markSuccess(type, task);
task.resolve(result); task.resolve(result);
markSuccess(type, task);
}).catch((err) => { }).catch((err) => {
markFailed(type, task); markFailed(type, task);
task.reject(err); task.reject(err);

Loading…
Cancel
Save