Use finally() after attempting to run worker jobs

This prevents now() from being passed an error. Previously, if the
UPDATE to increment the failure count was rejected, now() would be
passed an error, which isn't something it knows how to handle.

lib/worker/worker.js

@@ -11,7 +11,7 @@ const { min } = Math;
 const { inspect } = require('util');
 const { head } = require('ramda');
 const { sql } = require('slonik');
-const { timebound, resolve, runSequentially } = require('../util/promise');
+const { timebound, runSequentially } = require('../util/promise');
 const defaultJobMap = require('./jobs').jobs;
 
 // TODO: domain catch/restart? << investigated and seems unlikely.
@@ -58,10 +58,9 @@ const workerQueue = (container, jobMap = defaultJobMap) => {
       .then(() => { process.stdout.write(`[${(new Date()).toISOString()}] finish processing event ${logname}\n`); })
       .catch((err) => {
         report(err);
-        return run(sql`update audits set claimed=null, failures=${event.failures + 1}, "lastFailure"=clock_timestamp() where id=${event.id}`)
-          .then(() => resolve());
+        return run(sql`update audits set claimed=null, failures=${event.failures + 1}, "lastFailure"=clock_timestamp() where id=${event.id}`);
       })
-      .then(done, done);
+      .finally(done);
 
     return true;
   };