✨ Allow resetting ack when calling ping()

At the moment, when calling:

```js
queue.ping({resetTries: true})
```

The tries are reset as if the job hasn't been picked up, but the `ack`
is left as-is.

This isn't necessarily problematic in the operation of the queue, but it
does mean that the pinged job will still show up as [in-flight][1].

This change adds an optional `resetAck` flag, which will also unset the
`ack`, and means that the job can be marked as not in-flight, as if it
has never been picked up.

[1]: https://github.com/reedsy/mongodb-queue/blob/6133fc9367f4fce719e36d8866841d531e956b6b/mongodb-queue.ts#L262

Author: alecgibson

mongodb-queue.ts

@@ -39,6 +39,7 @@ export type GetOptions = {
 export type PingOptions = {
   visibility?: number;
   resetTries?: boolean;
+  resetAck?: boolean;
 };
 
 export type BaseMessage<T = any> = {
@@ -205,6 +206,10 @@ export class MongoDBQueue<T = any> {
       };
     }
 
+    if (opts.resetAck) {
+      update.$unset = {ack: 1};
+    }
+
     const msg = await this.col.findOneAndUpdate(query, update, options);
     if (!msg.value) {
       throw new Error('Queue.ping(): Unidentified ack  : ' + ack);

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reedsy/mongodb-queue",
-  "version": "8.1.0",
+  "version": "8.2.0",
   "description": "Message queues which uses MongoDB.",
   "main": "mongodb-queue.js",
   "scripts": {

test/ping.js

@@ -109,6 +109,33 @@ setup().then(({client, db}) => {
     t.end();
   });
 
+  test('ping: reset ack', async function(t) {
+    const queue = new MongoDBQueue(db, 'ping', {visibility: 3});
+    let msg;
+    let id;
+
+    id = await queue.add('Hello, World!');
+    t.ok(id, 'There is an id returned when adding a message.');
+    msg = await queue.get();
+    const ack = msg.ack;
+    // message should reset in three seconds
+    t.ok(msg.id, 'Got a msg.id (sanity check)');
+    await timeout(2000);
+    id = await queue.ping(msg.ack, {resetAck: true});
+    t.ok(id, 'Received an id when acking this message');
+    // wait until the msg has returned to the queue
+    await timeout(6000);
+    msg = await queue.get();
+    t.notEqual(ack, msg.ack, 'Ack was reset');
+    await queue.ack(msg.ack);
+    msg = await queue.get();
+    // no more messages
+    t.ok(!msg, 'No msg received');
+
+    t.pass('Finished test ok');
+    t.end();
+  });
+
   test('client.close()', function(t) {
     t.pass('client.close()');
     client.close();