sqs-consumer 5.7.0 → 5.8.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sqs-consumer",
-  "version": "5.7.0",
+  "version": "5.8.0",
   "description": "Build SQS-based Node applications without the boilerplate",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -11,11 +11,13 @@
     "prepublish": "npm run build",
     "pretest": "npm run build",
     "test": "mocha --recursive --full-trace --exit",
-    "lint": "eslint . --ext .ts",
-    "lint:fix": "eslint . --fix",
     "coverage": "nyc mocha && nyc report --reporter=html && nyc report --reporter=json-summary",
     "lcov": "nyc mocha && nyc report --reporter=lcov",
-    "posttest": "npm run lint"
+    "lint": "eslint . --ext .ts",
+    "lint:fix": "eslint . --fix",
+    "format": "prettier --loglevel warn --write \"**/*.{js,json,jsx,md,ts,tsx,html}\"",
+    "format:check": "prettier --check \"**/*.{js,json,jsx,md,ts,tsx,html}\"",
+    "posttest": "npm run lint && npm run format:check"
   },
   "repository": {
     "type": "git",
@@ -32,31 +34,33 @@
   ],
   "license": "Apache-2.0",
   "devDependencies": {
-    "@types/chai": "^4.2.11",
-    "@types/debug": "^4.1.5",
-    "@types/mocha": "^7.0.2",
-    "@types/node": "^14.0.13",
-    "@types/sinon": "^9.0.4",
-    "@types/typescript": "^2.0.0",
-    "chai": "^4.2.0",
-    "codeclimate-test-reporter": "^0.5.1",
-    "eslint": "^7.2.0",
-    "eslint-config-iplayer-ts": "^2.0.0",
-    "mocha": "^8.0.1",
+    "@types/chai": "^4.3.4",
+    "@types/debug": "^4.1.7",
+    "@types/mocha": "^10.0.1",
+    "@types/node": "^16.18.7",
+    "@types/sinon": "^10.0.13",
+    "chai": "^4.3.7",
+    "eslint": "^8.29.0",
+    "eslint-config-iplayer-ts": "^4.1.0",
+    "eslint-config-prettier": "^4.3.0",
+    "mocha": "^10.1.0",
     "nyc": "^15.1.0",
     "p-event": "^4.2.0",
-    "sinon": "^9.0.2",
-    "ts-node": "^8.10.2",
-    "tslint-config-airbnb": "^5.11.2",
-    "tslint-microsoft-contrib": "^6.2.0",
-    "typescript": "^3.9.5",
-    "aws-sdk": "^2.1114.0"
+    "prettier": "^2.8.1",
+    "sinon": "^15.0.0",
+    "ts-node": "^10.9.1",
+    "typescript": "^4.9.4"
   },
   "dependencies": {
-    "debug": "^4.3.1"
+    "aws-sdk": "^2.1271.0",
+    "debug": "^4.3.4"
   },
   "peerDependencies": {
-    "aws-sdk": "^2.1114.0"
+    "aws-sdk": "^2.1271.0"
+  },
+  "mocha": {
+    "spec": "test/**/**/*.test.ts",
+    "require": "ts-node/register"
   },
   "nyc": {
     "include": [
@@ -72,14 +76,28 @@
     "instrument": true
   },
   "eslintConfig": {
-    "extends": "iplayer-ts",
+    "extends": [
+      "iplayer-ts",
+      "prettier",
+      "prettier/react",
+      "prettier/@typescript-eslint"
+    ],
     "parserOptions": {
-      "ecmaVersion": 2017,
       "sourceType": "module"
+    },
+    "rules": {
+      "@typescript-eslint/naming-convention": [
+        "error",
+        {
+          "selector": "variable",
+          "format": [
+            "camelCase",
+            "UPPER_CASE",
+            "PascalCase"
+          ],
+          "leadingUnderscore": "allow"
+        }
+      ]
     }
-  },
-  "mocha": {
-    "spec": "test/**/**/*.ts",
-    "require": "ts-node/register"
   }
 }

package/src/consumer.ts

@@ -1,14 +1,16 @@
-import { AWSError } from 'aws-sdk';
-import * as SQS from 'aws-sdk/clients/sqs';
+import { AWSError, SQS } from 'aws-sdk';
 import { PromiseResult } from 'aws-sdk/lib/request';
-import * as Debug from 'debug';
+import Debug from 'debug';
 import { EventEmitter } from 'events';
 import { autoBind } from './bind';
 import { SQSError, TimeoutError } from './errors';
 
 const debug = Debug('sqs-consumer');
 
-type ReceieveMessageResponse = PromiseResult<SQS.Types.ReceiveMessageResult, AWSError>;
+type ReceieveMessageResponse = PromiseResult<
+  SQS.Types.ReceiveMessageResult,
+  AWSError
+>;
 type ReceiveMessageRequest = SQS.Types.ReceiveMessageRequest;
 export type SQSMessage = SQS.Types.Message;
 
@@ -37,7 +39,9 @@ function assertOptions(options: ConsumerOptions): void {
   requiredOptions.forEach((option) => {
     const possibilities = option.split('|');
     if (!possibilities.find((p) => options[p])) {
-      throw new Error(`Missing SQS consumer option [ ${possibilities.join(' or ')} ].`);
+      throw new Error(
+        `Missing SQS consumer option [ ${possibilities.join(' or ')} ].`
+      );
     }
   });
 
@@ -45,14 +49,21 @@ function assertOptions(options: ConsumerOptions): void {
     throw new Error('SQS batchSize option must be between 1 and 10.');
   }
 
-  if (options.heartbeatInterval && !(options.heartbeatInterval < options.visibilityTimeout)) {
+  if (
+    options.heartbeatInterval &&
+    !(options.heartbeatInterval < options.visibilityTimeout)
+  ) {
     throw new Error('heartbeatInterval must be less than visibilityTimeout.');
   }
 }
 
 function isConnectionError(err: Error): boolean {
   if (err instanceof SQSError) {
-    return (err.statusCode === 403 || err.code === 'CredentialsError' || err.code === 'UnknownEndpoint');
+    return (
+      err.statusCode === 403 ||
+      err.code === 'CredentialsError' ||
+      err.code === 'UnknownEndpoint'
+    );
   }
   return false;
 }
@@ -94,14 +105,14 @@ export interface ConsumerOptions {
 }
 
 interface Events {
-  'response_processed': [];
-  'empty': [];
-  'message_received': [SQSMessage];
-  'message_processed': [SQSMessage];
-  'error': [Error, void | SQSMessage | SQSMessage[]];
-  'timeout_error': [Error, SQSMessage];
-  'processing_error': [Error, SQSMessage];
-  'stopped': [];
+  response_processed: [];
+  empty: [];
+  message_received: [SQSMessage];
+  message_processed: [SQSMessage];
+  error: [Error, void | SQSMessage | SQSMessage[]];
+  timeout_error: [Error, SQSMessage];
+  processing_error: [Error, SQSMessage];
+  stopped: [];
 }
 
 export class Consumer extends EventEmitter {
@@ -134,16 +145,20 @@ export class Consumer extends EventEmitter {
     this.stopped = true;
     this.batchSize = options.batchSize || 1;
     this.visibilityTimeout = options.visibilityTimeout;
-    this.terminateVisibilityTimeout = options.terminateVisibilityTimeout || false;
+    this.terminateVisibilityTimeout =
+      options.terminateVisibilityTimeout || false;
     this.heartbeatInterval = options.heartbeatInterval;
     this.waitTimeSeconds = options.waitTimeSeconds ?? 20;
-    this.authenticationErrorTimeout = options.authenticationErrorTimeout ?? 10000;
+    this.authenticationErrorTimeout =
+      options.authenticationErrorTimeout ?? 10000;
     this.pollingWaitTimeMs = options.pollingWaitTimeMs ?? 0;
     this.shouldDeleteMessages = options.shouldDeleteMessages ?? true;
 
-    this.sqs = options.sqs || new SQS({
-      region: options.region || process.env.AWS_REGION || 'eu-west-1'
-    });
+    this.sqs =
+      options.sqs ||
+      new SQS({
+        region: options.region || process.env.AWS_REGION || 'eu-west-1'
+      });
 
     autoBind(this);
   }
@@ -152,11 +167,17 @@ export class Consumer extends EventEmitter {
     return super.emit(event, ...args);
   }
 
-  on<T extends keyof Events>(event: T, listener: (...args: Events[T]) => void): this {
+  on<T extends keyof Events>(
+    event: T,
+    listener: (...args: Events[T]) => void
+  ): this {
     return super.on(event, listener);
   }
 
-  once<T extends keyof Events>(event: T, listener: (...args: Events[T]) => void): this {
+  once<T extends keyof Events>(
+    event: T,
+    listener: (...args: Events[T]) => void
+  ): this {
     return super.once(event, listener);
   }
 
@@ -181,7 +202,9 @@ export class Consumer extends EventEmitter {
     this.stopped = true;
   }
 
-  private async handleSqsResponse(response: ReceieveMessageResponse): Promise<void> {
+  private async handleSqsResponse(
+    response: ReceieveMessageResponse
+  ): Promise<void> {
     debug('Received SQS response');
     debug(response);
 
@@ -224,11 +247,11 @@ export class Consumer extends EventEmitter {
     }
   }
 
-  private async receiveMessage(params: ReceiveMessageRequest): Promise<ReceieveMessageResponse> {
+  private async receiveMessage(
+    params: ReceiveMessageRequest
+  ): Promise<ReceieveMessageResponse> {
     try {
-      return await this.sqs
-        .receiveMessage(params)
-        .promise();
+      return await this.sqs.receiveMessage(params).promise();
     } catch (err) {
       throw toSQSError(err, `SQS receive message failed: ${err.message}`);
     }
@@ -236,7 +259,9 @@ export class Consumer extends EventEmitter {
 
   private async deleteMessage(message: SQSMessage): Promise<void> {
     if (!this.shouldDeleteMessages) {
-      debug('Skipping message delete since shouldDeleteMessages is set to false');
+      debug(
+        'Skipping message delete since shouldDeleteMessages is set to false'
+      );
       return;
     }
     debug('Deleting message %s', message.MessageId);
@@ -247,9 +272,7 @@ export class Consumer extends EventEmitter {
     };
 
     try {
-      await this.sqs
-        .deleteMessage(deleteParams)
-        .promise();
+      await this.sqs.deleteMessage(deleteParams).promise();
     } catch (err) {
       throw toSQSError(err, `SQS delete message failed: ${err.message}`);
     }
@@ -261,10 +284,7 @@ export class Consumer extends EventEmitter {
     try {
       if (this.handleMessageTimeout) {
         [timeout, pending] = createTimeout(this.handleMessageTimeout);
-        await Promise.race([
-          this.handleMessage(message),
-          pending
-        ]);
+        await Promise.race([this.handleMessage(message), pending]);
       } else {
         await this.handleMessage(message);
       }
@@ -280,7 +300,10 @@ export class Consumer extends EventEmitter {
     }
   }
 
-  private async changeVisibilityTimeout(message: SQSMessage, timeout: number): Promise<PromiseResult<any, AWSError>> {
+  private async changeVisibilityTimeout(
+    message: SQSMessage,
+    timeout: number
+  ): Promise<PromiseResult<any, AWSError>> {
     try {
       return await this.sqs
         .changeMessageVisibility({
@@ -290,7 +313,11 @@ export class Consumer extends EventEmitter {
         })
         .promise();
     } catch (err) {
-      this.emit('error', toSQSError(err, `Error changing visibility timeout: ${err.message}`), message);
+      this.emit(
+        'error',
+        toSQSError(err, `Error changing visibility timeout: ${err.message}`),
+        message
+      );
     }
   }
 
@@ -330,9 +357,11 @@ export class Consumer extends EventEmitter {
           currentPollingTimeout = this.authenticationErrorTimeout;
         }
         return;
-      }).then(() => {
+      })
+      .then(() => {
         setTimeout(this.poll, currentPollingTimeout);
-      }).catch((err) => {
+      })
+      .catch((err) => {
         this.emit('error', err);
       });
   }
@@ -346,7 +375,10 @@ export class Consumer extends EventEmitter {
     try {
       if (this.heartbeatInterval) {
         heartbeat = this.startHeartbeat(async () => {
-          return this.changeVisabilityTimeoutBatch(messages, this.visibilityTimeout);
+          return this.changeVisabilityTimeoutBatch(
+            messages,
+            this.visibilityTimeout
+          );
         });
       }
       await this.executeBatchHandler(messages);
@@ -367,10 +399,15 @@ export class Consumer extends EventEmitter {
 
   private async deleteMessageBatch(messages: SQSMessage[]): Promise<void> {
     if (!this.shouldDeleteMessages) {
-      debug('Skipping message delete since shouldDeleteMessages is set to false');
+      debug(
+        'Skipping message delete since shouldDeleteMessages is set to false'
+      );
       return;
     }
-    debug('Deleting messages %s', messages.map((msg) => msg.MessageId).join(' ,'));
+    debug(
+      'Deleting messages %s',
+      messages.map((msg) => msg.MessageId).join(' ,')
+    );
 
     const deleteParams = {
       QueueUrl: this.queueUrl,
@@ -381,9 +418,7 @@ export class Consumer extends EventEmitter {
     };
 
     try {
-      await this.sqs
-        .deleteMessageBatch(deleteParams)
-        .promise();
+      await this.sqs.deleteMessageBatch(deleteParams).promise();
     } catch (err) {
       throw toSQSError(err, `SQS delete message failed: ${err.message}`);
     }
@@ -398,7 +433,10 @@ export class Consumer extends EventEmitter {
     }
   }
 
-  private async changeVisabilityTimeoutBatch(messages: SQSMessage[], timeout: number): Promise<PromiseResult<any, AWSError>> {
+  private async changeVisabilityTimeoutBatch(
+    messages: SQSMessage[],
+    timeout: number
+  ): Promise<PromiseResult<any, AWSError>> {
     const params = {
       QueueUrl: this.queueUrl,
       Entries: messages.map((message) => ({
@@ -408,11 +446,13 @@ export class Consumer extends EventEmitter {
       }))
     };
     try {
-      return await this.sqs
-        .changeMessageVisibilityBatch(params)
-        .promise();
+      return await this.sqs.changeMessageVisibilityBatch(params).promise();
     } catch (err) {
-      this.emit('error', toSQSError(err, `Error changing visibility timeout: ${err.message}`), messages);
+      this.emit(
+        'error',
+        toSQSError(err, `Error changing visibility timeout: ${err.message}`),
+        messages
+      );
     }
   }
 

package/src/errors.ts

@@ -20,7 +20,4 @@ class TimeoutError extends Error {
   }
 }
 
-export {
-  SQSError,
-  TimeoutError
-};
+export { SQSError, TimeoutError };

package/test/{consumer.ts → consumer.test.ts}

@@ -1,8 +1,7 @@
 import { assert } from 'chai';
-import * as pEvent from 'p-event';
-
 import * as sinon from 'sinon';
-import { Consumer } from '../src/index';
+import * as pEvent from 'p-event';
+import { Consumer } from '../src/consumer';
 
 const sandbox = sinon.createSandbox();
 
@@ -10,15 +9,11 @@ const AUTHENTICATION_ERROR_TIMEOUT = 20;
 const POLLING_TIMEOUT = 100;
 
 function stubResolve(value?: any): any {
-  return sandbox
-    .stub()
-    .returns({ promise: sandbox.stub().resolves(value) });
+  return sandbox.stub().returns({ promise: sandbox.stub().resolves(value) });
 }
 
 function stubReject(value?: any): any {
-  return sandbox
-    .stub()
-    .returns({ promise: sandbox.stub().rejects(value) });
+  return sandbox.stub().returns({ promise: sandbox.stub().rejects(value) });
 }
 
 class MockSQSError extends Error {
@@ -35,7 +30,6 @@ class MockSQSError extends Error {
   }
 }
 
-// tslint:disable:no-unused-expression
 describe('Consumer', () => {
   let consumer;
   let clock;
@@ -43,11 +37,13 @@ describe('Consumer', () => {
   let handleMessageBatch;
   let sqs;
   const response = {
-    Messages: [{
-      ReceiptHandle: 'receipt-handle',
-      MessageId: '123',
-      Body: 'body'
-    }]
+    Messages: [
+      {
+        ReceiptHandle: 'receipt-handle',
+        MessageId: '123',
+        Body: 'body'
+      }
+    ]
   };
 
   beforeEach(() => {
@@ -71,6 +67,7 @@ describe('Consumer', () => {
   });
 
   afterEach(() => {
+    clock.restore();
     sandbox.restore();
   });
 
@@ -198,18 +195,25 @@ describe('Consumer', () => {
       consumer = new Consumer({
         queueUrl: 'some-queue-url',
         region: 'some-region',
-        handleMessage: () => new Promise((resolve) => setTimeout(resolve, 1000)),
+        handleMessage: () =>
+          new Promise((resolve) => setTimeout(resolve, 1000)),
         handleMessageTimeout,
         sqs,
         authenticationErrorTimeout: 20
       });
 
       consumer.start();
-      const [err]: any = await Promise.all([pEvent(consumer, 'timeout_error'), clock.tickAsync(handleMessageTimeout)]);
+      const [err]: any = await Promise.all([
+        pEvent(consumer, 'timeout_error'),
+        clock.tickAsync(handleMessageTimeout)
+      ]);
       consumer.stop();
 
       assert.ok(err);
-      assert.equal(err.message, `Message handler timed out after ${handleMessageTimeout}ms: Operation timed out.`);
+      assert.equal(
+        err.message,
+        `Message handler timed out after ${handleMessageTimeout}ms: Operation timed out.`
+      );
     });
 
     it('handles unexpected exceptions thrown by the handler function', async () => {
@@ -228,7 +232,10 @@ describe('Consumer', () => {
       consumer.stop();
 
       assert.ok(err);
-      assert.equal(err.message, 'Unexpected message handler failure: unexpected parsing error');
+      assert.equal(
+        err.message,
+        'Unexpected message handler failure: unexpected parsing error'
+      );
     });
 
     it('fires an error event when an error occurs deleting a message', async () => {
@@ -251,10 +258,18 @@ describe('Consumer', () => {
       handleMessage.rejects(processingErr);
 
       consumer.start();
-      const [err, message] = await pEvent(consumer, 'processing_error', { multiArgs: true });
+      const [err, message] = await pEvent<
+        string | symbol,
+        { [key: string]: string }[]
+      >(consumer, 'processing_error', {
+        multiArgs: true
+      });
       consumer.stop();
 
-      assert.equal(err.message, 'Unexpected message handler failure: Processing error');
+      assert.equal(
+        err instanceof Error ? err.message : '',
+        'Unexpected message handler failure: Processing error'
+      );
       assert.equal(message.MessageId, '123');
     });
 
@@ -266,7 +281,12 @@ describe('Consumer', () => {
       sqs.deleteMessage = stubReject(sqsError);
 
       consumer.start();
-      const [err, message] = await pEvent(consumer, 'error', { multiArgs: true });
+      const [err, message] = await pEvent<
+        string | symbol,
+        { [key: string]: string }[]
+      >(consumer, 'error', {
+        multiArgs: true
+      });
       consumer.stop();
 
       assert.equal(err.message, 'SQS delete message failed: Processing error');
@@ -310,7 +330,8 @@ describe('Consumer', () => {
     it('waits before repolling when a UnknownEndpoint error occurs', async () => {
       const unknownEndpointErr = {
         code: 'UnknownEndpoint',
-        message: 'Inaccessible host: `sqs.eu-west-1.amazonaws.com`. This service may not be available in the `eu-west-1` region.'
+        message:
+          'Inaccessible host: `sqs.eu-west-1.amazonaws.com`. This service may not be available in the `eu-west-1` region.'
       };
       sqs.receiveMessage = stubReject(unknownEndpointErr);
       const errorListener = sandbox.stub();
@@ -380,7 +401,7 @@ describe('Consumer', () => {
       });
     });
 
-    it('doesn\'t delete the message when a processing error is reported', async () => {
+    it("doesn't delete the message when a processing error is reported", async () => {
       handleMessage.rejects(new Error('Processing error'));
 
       consumer.start();
@@ -400,8 +421,10 @@ describe('Consumer', () => {
       sandbox.assert.calledTwice(handleMessage);
     });
 
-    it('doesn\'t consume more messages when called multiple times', () => {
-      sqs.receiveMessage = stubResolve(new Promise((res) => setTimeout(res, 100)));
+    it("doesn't consume more messages when called multiple times", () => {
+      sqs.receiveMessage = stubResolve(
+        new Promise((res) => setTimeout(res, 100))
+      );
       consumer.start();
       consumer.start();
       consumer.start();
@@ -461,7 +484,7 @@ describe('Consumer', () => {
       });
     });
 
-    it('consumes messages with message attribute \'ApproximateReceiveCount\'', async () => {
+    it("consumes messages with message attribute 'ApproximateReceiveCount'", async () => {
       const messageWithAttr = {
         ReceiptHandle: 'receipt-handle-1',
         MessageId: '1',
@@ -620,14 +643,14 @@ describe('Consumer', () => {
 
       sandbox.assert.callCount(handleMessageBatch, 1);
       sandbox.assert.callCount(handleMessage, 0);
-
     });
 
     it('uses the correct visibility timeout for long running handler functions', async () => {
       consumer = new Consumer({
         queueUrl: 'some-queue-url',
         region: 'some-region',
-        handleMessage: () => new Promise((resolve) => setTimeout(resolve, 75000)),
+        handleMessage: () =>
+          new Promise((resolve) => setTimeout(resolve, 75000)),
         sqs,
         visibilityTimeout: 40,
         heartbeatInterval: 30
@@ -635,7 +658,10 @@ describe('Consumer', () => {
       const clearIntervalSpy = sinon.spy(global, 'clearInterval');
 
       consumer.start();
-      await Promise.all([pEvent(consumer, 'response_processed'), clock.tickAsync(75000)]);
+      await Promise.all([
+        pEvent(consumer, 'response_processed'),
+        clock.tickAsync(75000)
+      ]);
       consumer.stop();
 
       sandbox.assert.calledWith(sqs.changeMessageVisibility, {
@@ -662,7 +688,8 @@ describe('Consumer', () => {
       consumer = new Consumer({
         queueUrl: 'some-queue-url',
         region: 'some-region',
-        handleMessageBatch: () => new Promise((resolve) => setTimeout(resolve, 75000)),
+        handleMessageBatch: () =>
+          new Promise((resolve) => setTimeout(resolve, 75000)),
         batchSize: 3,
         sqs,
         visibilityTimeout: 40,
@@ -671,7 +698,10 @@ describe('Consumer', () => {
       const clearIntervalSpy = sinon.spy(global, 'clearInterval');
 
       consumer.start();
-      await Promise.all([pEvent(consumer, 'response_processed'), clock.tickAsync(75000)]);
+      await Promise.all([
+        pEvent(consumer, 'response_processed'),
+        clock.tickAsync(75000)
+      ]);
       consumer.stop();
 
       sandbox.assert.calledWith(sqs.changeMessageVisibilityBatch, {
@@ -702,7 +732,8 @@ describe('Consumer', () => {
       consumer = new Consumer({
         queueUrl: 'some-queue-url',
         region: 'some-region',
-        handleMessage: () => new Promise((resolve) => setTimeout(resolve, 75000)),
+        handleMessage: () =>
+          new Promise((resolve) => setTimeout(resolve, 75000)),
         sqs,
         visibilityTimeout: 40,
         heartbeatInterval: 30
@@ -712,7 +743,10 @@ describe('Consumer', () => {
       sqs.changeMessageVisibility = stubReject(receiveErr);
 
       consumer.start();
-      const [err]: any[] = await Promise.all([pEvent(consumer, 'error'), clock.tickAsync(75000)]);
+      const [err]: any[] = await Promise.all([
+        pEvent(consumer, 'error'),
+        clock.tickAsync(75000)
+      ]);
       consumer.stop();
 
       assert.ok(err);
@@ -729,7 +763,8 @@ describe('Consumer', () => {
       consumer = new Consumer({
         queueUrl: 'some-queue-url',
         region: 'some-region',
-        handleMessageBatch: () => new Promise((resolve) => setTimeout(resolve, 75000)),
+        handleMessageBatch: () =>
+          new Promise((resolve) => setTimeout(resolve, 75000)),
         sqs,
         batchSize: 2,
         visibilityTimeout: 40,
@@ -740,7 +775,10 @@ describe('Consumer', () => {
       sqs.changeMessageVisibilityBatch = stubReject(receiveErr);
 
       consumer.start();
-      const [err]: any[] = await Promise.all([pEvent(consumer, 'error'), clock.tickAsync(75000)]);
+      const [err]: any[] = await Promise.all([
+        pEvent(consumer, 'error'),
+        clock.tickAsync(75000)
+      ]);
       consumer.stop();
 
       assert.ok(err);