@webex/internal-plugin-metrics 3.0.0-beta.4 → 3.0.0-beta.400

package/test/unit/spec/prelogin-metrics-batcher.ts

@@ -0,0 +1,250 @@
+/*!
+ * Copyright (c) 2015-2020 Cisco Systems, Inc. See LICENSE file.
+ */
+
+import {assert} from '@webex/test-helper-chai';
+import {config, Utils} from '@webex/internal-plugin-metrics';
+import {CallDiagnosticUtils} from '@webex/internal-plugin-metrics';
+import MockWebex from '@webex/test-helper-mock-webex';
+import sinon from 'sinon';
+import FakeTimers from '@sinonjs/fake-timers';
+import {NewMetrics} from '@webex/internal-plugin-metrics';
+import {uniqueId} from 'lodash';
+
+const flushPromises = () => new Promise(setImmediate);
+
+describe('internal-plugin-metrics', () => {
+  describe('PreLoginMetricsBatcher', () => {
+    let webex;
+    let clock;
+    let now;
+
+    const preLoginId = 'my_prelogin_id';
+
+    beforeEach(() => {
+      now = new Date();
+      clock = FakeTimers.install({now});
+
+      //@ts-ignore
+      webex = new MockWebex({
+        children: {
+          newMetrics: NewMetrics,
+        },
+      });
+
+      webex.request = (options) =>
+        Promise.resolve({body: {items: []}, waitForServiceTimeout: 15, options});
+
+      sinon.spy(webex, 'request');
+      webex.emit('ready');
+      webex.config.metrics = config.metrics;
+    });
+
+    afterEach(() => {
+      sinon.restore();
+      clock.uninstall();
+    });
+
+    describe('#request()', () => {
+      it('when the request completes successfully, clears the queue', async () => {
+        const promise =
+          webex.internal.newMetrics.callDiagnosticMetrics.submitToCallDiagnosticsPreLogin(
+            //@ts-ignore
+            {event: {name: 'client.interstitial-window.launched'}},
+            preLoginId
+          );
+        await flushPromises();
+        clock.tick(config.metrics.batcherWait);
+
+        await promise;
+
+        const webexRequestArgs = webex.request.args[0][0];
+
+        const dateAfterBatcherWait = new Date(now.getTime() + config.metrics.batcherWait);
+        //@ts-ignore
+        assert.calledOnce(webex.request);
+
+        // matching because the request includes a symbol key: value pair and sinon cannot handle to compare it..
+        assert.match(webexRequestArgs, {
+          body: {
+            metrics: [
+              {
+                eventPayload: {
+                  event: {
+                    joinTimes: {
+                      meetingInfoReqResp: undefined,
+                      clickToInterstitial: undefined,
+                    },
+                    name: 'client.interstitial-window.launched',
+                  },
+                  origin: {
+                    buildType: 'test',
+                    networkType: 'unknown',
+                  },
+                  originTime: {
+                    sent: dateAfterBatcherWait.toISOString(),
+                  },
+                },
+                type: ['diagnostic-event'],
+              },
+            ],
+          },
+          headers: {
+            authorization: false,
+            'x-prelogin-userid': preLoginId,
+          },
+          method: 'POST',
+          resource: 'clientmetrics-prelogin',
+          service: 'metrics',
+          waitForServiceTimeout: 30,
+        });
+        assert.lengthOf(
+          webex.internal.newMetrics.callDiagnosticMetrics.preLoginMetricsBatcher.queue,
+          0
+        );
+      });
+
+      it('when the request fails, does not clear the queue', async () => {
+        webex.request = sinon.stub().rejects(new Error('my_error'));
+
+        webex.logger.error = sinon.stub();
+        webex.logger.log = sinon.stub();
+        sinon.stub(Utils, 'generateCommonErrorMetadata').returns('formattedError');
+
+        const promise =
+          webex.internal.newMetrics.callDiagnosticMetrics.submitToCallDiagnosticsPreLogin(
+            {
+              event: 'my.event',
+            },
+            preLoginId
+          );
+
+        await flushPromises();
+        clock.tick(config.metrics.batcherWait);
+
+        let error;
+
+        // catch the expected error and store it
+        try {
+          await promise;
+        } catch (err) {
+          error = err;
+        }
+
+        // This is horrific, but stubbing lodash is proving difficult
+        const expectedBatchId = parseInt(uniqueId()) - 1;
+
+        assert.equal(error.message, 'my_error');
+        assert.calledOnceWithExactly(
+          webex.logger.error,
+          'Pre Login Metrics -->',
+          `PreLoginMetricsBatcher: @submitHttpRequest#prelogin-ca-batch-${expectedBatchId}. Request failed:`,
+          `error: formattedError`
+        );
+        assert.lengthOf(
+          webex.internal.newMetrics.callDiagnosticMetrics.preLoginMetricsBatcher.queue,
+          0
+        );
+      });
+
+      it('fails if preLoinId is not set', async () => {
+        webex.internal.newMetrics.callDiagnosticMetrics.preLoginMetricsBatcher.preLoginId = undefined;
+
+        const promise =
+          webex.internal.newMetrics.callDiagnosticMetrics.submitToCallDiagnosticsPreLogin(
+            {
+              event: 'my.event',
+            },
+            undefined
+          );
+
+        await flushPromises();
+        clock.tick(config.metrics.batcherWait);
+
+        let error;
+
+        // catch the expected error and store it
+        try {
+          await promise;
+        } catch (err) {
+          error = err;
+        }
+
+        assert.equal(error.message, 'PreLoginId is not set.');
+      });
+    });
+
+    describe('prepareItem', () => {
+      it('calls prepareDiagnosticMetricItem correctly', async () => {
+         // avoid setting .sent timestamp
+        webex.internal.newMetrics.callDiagnosticMetrics.preLoginMetricsBatcher.prepareRequest = (q) =>
+          Promise.resolve(q);
+
+        const prepareItemSpy = sinon.spy(
+          webex.internal.newMetrics.callDiagnosticMetrics.preLoginMetricsBatcher,
+          'prepareItem'
+        );
+        const prepareDiagnosticMetricItemSpy = sinon.spy(
+          CallDiagnosticUtils,
+          'prepareDiagnosticMetricItem'
+        );
+
+        const promise =
+          webex.internal.newMetrics.callDiagnosticMetrics.submitToCallDiagnosticsPreLogin(
+            {
+              event: 'my.event',
+            },
+            preLoginId
+          );
+
+        await flushPromises();
+
+        clock.tick(config.metrics.batcherWait);
+
+        await promise;
+
+        const calls = prepareItemSpy.getCalls()[0];
+
+        // item also gets assigned a delay property but the key is a Symbol and haven't been able to test that..
+        assert.deepEqual(calls.args[0].eventPayload, {
+          event: 'my.event',
+          origin: {buildType: 'test', networkType: 'unknown'},
+        });
+
+        assert.deepEqual(calls.args[0].type, ['diagnostic-event']);
+
+        const prepareDiagnosticMetricItemCalls = prepareDiagnosticMetricItemSpy.getCalls();
+
+        // second argument (item) also gets assigned a delay property but the key is a Symbol and haven't been able to test that..
+        assert.deepEqual(prepareDiagnosticMetricItemCalls[0].args[0], webex);
+        assert.deepEqual(prepareDiagnosticMetricItemCalls[0].args[1].eventPayload, {
+          event: 'my.event',
+          origin: {
+            buildType: 'test',
+            networkType: 'unknown',
+          },
+        });
+        assert.deepEqual(prepareDiagnosticMetricItemCalls[0].args[1].type, ['diagnostic-event']);
+      });
+    });
+
+    describe('savePreLoginId', () => {
+      it('saves the preLoginId', () => {
+        const preLoginId = 'my_prelogin_id';
+
+        assert.isUndefined(
+          webex.internal.newMetrics.callDiagnosticMetrics.preLoginMetricsBatcher.preLoginId
+        );
+
+        webex.internal.newMetrics.callDiagnosticMetrics.preLoginMetricsBatcher.savePreLoginId(
+          preLoginId
+        );
+
+        assert.equal(
+          webex.internal.newMetrics.callDiagnosticMetrics.preLoginMetricsBatcher.preLoginId,
+          preLoginId
+        );
+      });
+    });
+  });
+});

package/test/unit/spec/utils.ts

@@ -0,0 +1,22 @@
+import {assert} from '@webex/test-helper-chai';
+import {Utils} from '@webex/internal-plugin-metrics';
+
+describe('internal-plugin-metrics', () => {
+  describe('generateCommonErrorMetadata', () => {
+    it('should return JSON stringified error object', () => {
+      const error = new Error('test error');
+      const result = Utils.generateCommonErrorMetadata(error);
+      assert.deepEqual(result, JSON.stringify({
+        message: 'test error',
+        name: 'Error',
+        stack: error.stack
+      }))
+    });
+
+    it('should return error if not instanceof Error', () => {
+      const error = 'test error';
+      const result = Utils.generateCommonErrorMetadata(error);
+     assert.deepEqual(result, 'test error')
+    });
+  });
+})

package/tsconfig.json

@@ -0,0 +1,6 @@
+{
+    "extends": "../../../tsconfig.json",
+    "include": [
+        "src"
+    ],
+}

package/dist/call-diagnostic-events-batcher.js

@@ -1,70 +0,0 @@
-"use strict";
-
-var _Object$defineProperty = require("@babel/runtime-corejs2/core-js/object/define-property");
-
-var _interopRequireDefault = require("@babel/runtime-corejs2/helpers/interopRequireDefault");
-
-_Object$defineProperty(exports, "__esModule", {
-  value: true
-});
-
-exports.default = void 0;
-
-var _assign = _interopRequireDefault(require("@babel/runtime-corejs2/core-js/object/assign"));
-
-var _promise = _interopRequireDefault(require("@babel/runtime-corejs2/core-js/promise"));
-
-var _batcher = _interopRequireDefault(require("./batcher"));
-
-/*!
- * Copyright (c) 2015-2020 Cisco Systems, Inc. See LICENSE file.
- */
-var CallDiagnosticEventsBatcher = _batcher.default.extend({
-  namespace: 'Metrics',
-
-  /**
-   * @param {string} webClientDomain
-   * @returns {string}
-  */
-  getBuildType: function getBuildType(webClientDomain) {
-    if (webClientDomain !== null && webClientDomain !== void 0 && webClientDomain.includes('teams.webex.com') || webClientDomain !== null && webClientDomain !== void 0 && webClientDomain.includes('localhost') || webClientDomain !== null && webClientDomain !== void 0 && webClientDomain.includes('127.0.0.1') || process.env.NODE_ENV !== 'production') {
-      return 'test';
-    }
-
-    return process.env.NODE_ENV === 'production' ? 'prod' : 'test';
-  },
-  prepareItem: function prepareItem(item) {
-    var _item$event, _item$event$eventData;
-
-    // networkType should be a enum value: `wifi`, `ethernet`, `cellular`, or `unknown`.
-    // Browsers cannot provide such information right now. However, it is a required field.
-    var origin = {
-      buildType: this.getBuildType((_item$event = item.event) === null || _item$event === void 0 ? void 0 : (_item$event$eventData = _item$event.eventData) === null || _item$event$eventData === void 0 ? void 0 : _item$event$eventData.webClientDomain),
-      networkType: 'unknown'
-    };
-    item.eventPayload.origin = (0, _assign.default)(origin, item.eventPayload.origin);
-    return _promise.default.resolve(item);
-  },
-  prepareRequest: function prepareRequest(queue) {
-    // Add sent timestamp
-    queue.forEach(function (item) {
-      item.eventPayload.originTime = item.eventPayload.originTime || {};
-      item.eventPayload.originTime.sent = new Date().toISOString();
-    });
-    return _promise.default.resolve(queue);
-  },
-  submitHttpRequest: function submitHttpRequest(payload) {
-    return this.webex.request({
-      method: 'POST',
-      service: 'metrics',
-      resource: 'clientmetrics',
-      body: {
-        metrics: payload
-      }
-    });
-  }
-});
-
-var _default = CallDiagnosticEventsBatcher;
-exports.default = _default;
-//# sourceMappingURL=call-diagnostic-events-batcher.js.map

package/dist/call-diagnostic-events-batcher.js.map

@@ -1 +0,0 @@
-{"version":3,"names":["CallDiagnosticEventsBatcher","Batcher","extend","namespace","getBuildType","webClientDomain","includes","process","env","NODE_ENV","prepareItem","item","origin","buildType","event","eventData","networkType","eventPayload","resolve","prepareRequest","queue","forEach","originTime","sent","Date","toISOString","submitHttpRequest","payload","webex","request","method","service","resource","body","metrics"],"sources":["call-diagnostic-events-batcher.js"],"sourcesContent":["/*!\n * Copyright (c) 2015-2020 Cisco Systems, Inc. See LICENSE file.\n */\n\nimport Batcher from './batcher';\n\nconst CallDiagnosticEventsBatcher = Batcher.extend({\n  namespace: 'Metrics',\n\n  /**\n   * @param {string} webClientDomain\n   * @returns {string}\n  */\n  getBuildType(webClientDomain) {\n    if (\n      webClientDomain?.includes('teams.webex.com') ||\n      webClientDomain?.includes('localhost') ||\n      webClientDomain?.includes('127.0.0.1') ||\n      process.env.NODE_ENV !== 'production'\n    ) {\n      return 'test';\n    }\n\n    return process.env.NODE_ENV === 'production' ? 'prod' : 'test';\n  },\n\n  prepareItem(item) {\n    // networkType should be a enum value: `wifi`, `ethernet`, `cellular`, or `unknown`.\n    // Browsers cannot provide such information right now. However, it is a required field.\n    const origin = {\n      buildType: this.getBuildType(item.event?.eventData?.webClientDomain),\n      networkType: 'unknown'\n    };\n\n    item.eventPayload.origin = Object.assign(origin, item.eventPayload.origin);\n\n    return Promise.resolve(item);\n  },\n\n  prepareRequest(queue) {\n    // Add sent timestamp\n    queue.forEach((item) => {\n      item.eventPayload.originTime = item.eventPayload.originTime || {};\n      item.eventPayload.originTime.sent = new Date().toISOString();\n    });\n\n    return Promise.resolve(queue);\n  },\n\n  submitHttpRequest(payload) {\n    return this.webex.request({\n      method: 'POST',\n      service: 'metrics',\n      resource: 'clientmetrics',\n      body: {\n        metrics: payload\n      }\n    });\n  }\n});\n\nexport default CallDiagnosticEventsBatcher;\n"],"mappings":";;;;;;;;;;;;;;;;AAIA;;AAJA;AACA;AACA;AAIA,IAAMA,2BAA2B,GAAGC,gBAAA,CAAQC,MAAR,CAAe;EACjDC,SAAS,EAAE,SADsC;;EAGjD;AACF;AACA;AACA;EACEC,YAPiD,wBAOpCC,eAPoC,EAOnB;IAC5B,IACEA,eAAe,SAAf,IAAAA,eAAe,WAAf,IAAAA,eAAe,CAAEC,QAAjB,CAA0B,iBAA1B,KACAD,eADA,aACAA,eADA,eACAA,eAAe,CAAEC,QAAjB,CAA0B,WAA1B,CADA,IAEAD,eAFA,aAEAA,eAFA,eAEAA,eAAe,CAAEC,QAAjB,CAA0B,WAA1B,CAFA,IAGAC,OAAO,CAACC,GAAR,CAAYC,QAAZ,KAAyB,YAJ3B,EAKE;MACA,OAAO,MAAP;IACD;;IAED,OAAOF,OAAO,CAACC,GAAR,CAAYC,QAAZ,KAAyB,YAAzB,GAAwC,MAAxC,GAAiD,MAAxD;EACD,CAlBgD;EAoBjDC,WApBiD,uBAoBrCC,IApBqC,EAoB/B;IAAA;;IAChB;IACA;IACA,IAAMC,MAAM,GAAG;MACbC,SAAS,EAAE,KAAKT,YAAL,gBAAkBO,IAAI,CAACG,KAAvB,yEAAkB,YAAYC,SAA9B,0DAAkB,sBAAuBV,eAAzC,CADE;MAEbW,WAAW,EAAE;IAFA,CAAf;IAKAL,IAAI,CAACM,YAAL,CAAkBL,MAAlB,GAA2B,qBAAcA,MAAd,EAAsBD,IAAI,CAACM,YAAL,CAAkBL,MAAxC,CAA3B;IAEA,OAAO,iBAAQM,OAAR,CAAgBP,IAAhB,CAAP;EACD,CA/BgD;EAiCjDQ,cAjCiD,0BAiClCC,KAjCkC,EAiC3B;IACpB;IACAA,KAAK,CAACC,OAAN,CAAc,UAACV,IAAD,EAAU;MACtBA,IAAI,CAACM,YAAL,CAAkBK,UAAlB,GAA+BX,IAAI,CAACM,YAAL,CAAkBK,UAAlB,IAAgC,EAA/D;MACAX,IAAI,CAACM,YAAL,CAAkBK,UAAlB,CAA6BC,IAA7B,GAAoC,IAAIC,IAAJ,GAAWC,WAAX,EAApC;IACD,CAHD;IAKA,OAAO,iBAAQP,OAAR,CAAgBE,KAAhB,CAAP;EACD,CAzCgD;EA2CjDM,iBA3CiD,6BA2C/BC,OA3C+B,EA2CtB;IACzB,OAAO,KAAKC,KAAL,CAAWC,OAAX,CAAmB;MACxBC,MAAM,EAAE,MADgB;MAExBC,OAAO,EAAE,SAFe;MAGxBC,QAAQ,EAAE,eAHc;MAIxBC,IAAI,EAAE;QACJC,OAAO,EAAEP;MADL;IAJkB,CAAnB,CAAP;EAQD;AApDgD,CAAf,CAApC;;eAuDe3B,2B"}

package/src/call-diagnostic-events-batcher.js

@@ -1,62 +0,0 @@
-/*!
- * Copyright (c) 2015-2020 Cisco Systems, Inc. See LICENSE file.
- */
-
-import Batcher from './batcher';
-
-const CallDiagnosticEventsBatcher = Batcher.extend({
-  namespace: 'Metrics',
-
-  /**
-   * @param {string} webClientDomain
-   * @returns {string}
-  */
-  getBuildType(webClientDomain) {
-    if (
-      webClientDomain?.includes('teams.webex.com') ||
-      webClientDomain?.includes('localhost') ||
-      webClientDomain?.includes('127.0.0.1') ||
-      process.env.NODE_ENV !== 'production'
-    ) {
-      return 'test';
-    }
-
-    return process.env.NODE_ENV === 'production' ? 'prod' : 'test';
-  },
-
-  prepareItem(item) {
-    // networkType should be a enum value: `wifi`, `ethernet`, `cellular`, or `unknown`.
-    // Browsers cannot provide such information right now. However, it is a required field.
-    const origin = {
-      buildType: this.getBuildType(item.event?.eventData?.webClientDomain),
-      networkType: 'unknown'
-    };
-
-    item.eventPayload.origin = Object.assign(origin, item.eventPayload.origin);
-
-    return Promise.resolve(item);
-  },
-
-  prepareRequest(queue) {
-    // Add sent timestamp
-    queue.forEach((item) => {
-      item.eventPayload.originTime = item.eventPayload.originTime || {};
-      item.eventPayload.originTime.sent = new Date().toISOString();
-    });
-
-    return Promise.resolve(queue);
-  },
-
-  submitHttpRequest(payload) {
-    return this.webex.request({
-      method: 'POST',
-      service: 'metrics',
-      resource: 'clientmetrics',
-      body: {
-        metrics: payload
-      }
-    });
-  }
-});
-
-export default CallDiagnosticEventsBatcher;

package/src/index.js

@@ -1,17 +0,0 @@
-/*!
- * Copyright (c) 2015-2020 Cisco Systems, Inc. See LICENSE file.
- */
-
-import '@webex/internal-plugin-device';
-
-import {registerInternalPlugin} from '@webex/webex-core';
-
-import Metrics from './metrics';
-import config from './config';
-
-registerInternalPlugin('metrics', Metrics, {
-  config
-});
-
-export {default} from './metrics';
-export {config};

package/test/unit/spec/call-diagnostic-events-batcher.js

@@ -1,180 +0,0 @@
-/*!
- * Copyright (c) 2015-2020 Cisco Systems, Inc. See LICENSE file.
- */
-
-import {assert} from '@webex/test-helper-chai';
-import FakeTimers from '@sinonjs/fake-timers';
-import Metrics, {config} from '@webex/internal-plugin-metrics';
-import MockWebex from '@webex/test-helper-mock-webex';
-import sinon from 'sinon';
-import {WebexHttpError} from '@webex/webex-core';
-
-function promiseTick(count) {
-  let promise = Promise.resolve();
-
-  while (count > 1) {
-    promise = promise.then(() => promiseTick(1));
-    count -= 1;
-  }
-
-  return promise;
-}
-
-describe('plugin-metrics', () => {
-  describe('callDiagnosticEventsBatcher', () => {
-    let webex;
-
-    beforeEach(() => {
-      webex = new MockWebex({
-        children: {
-          metrics: Metrics
-        }
-      });
-
-      webex.config.metrics = config.metrics;
-
-      webex.request = function (options) {
-        return Promise.resolve({
-          statusCode: 204,
-          body: undefined,
-          options
-        });
-      };
-      sinon.spy(webex, 'request');
-    });
-
-    let clock;
-
-    beforeEach(() => {
-      clock = FakeTimers.install();
-    });
-
-    afterEach(() => {
-      clock.uninstall();
-    });
-
-    describe('#request()', () => {
-      describe('when the request completes successfully', () => {
-        it('clears the queue', () => {
-          clock.uninstall();
-
-          return webex.internal.metrics.callDiagnosticEventsBatcher.request({
-            type: 'diagnostic-event',
-            eventPayload: {
-              originTime: {
-                triggered: 'mock triggered timestamp'
-              }
-            }
-          })
-            .then(() => {
-              assert.calledOnce(webex.request);
-              assert.lengthOf(webex.internal.metrics.callDiagnosticEventsBatcher.queue, 0);
-            });
-        });
-      });
-
-      describe('when the request fails due to network disconnect', () => {
-        it('reenqueues the payload', () => {
-          // sinon appears to have gap in its api where stub.onCall(n) doesn't
-          // accept a function, so the following is more verbose than one might
-          // desire
-          webex.request = function () {
-            // noop
-          };
-          let count = 0;
-
-          sinon.stub(webex, 'request').callsFake((options) => {
-            options.headers = {
-              trackingid: count
-            };
-
-            count += 1;
-            if (count < 9) {
-              return Promise.reject(new WebexHttpError.NetworkOrCORSError({
-                statusCode: 0,
-                options
-              }));
-            }
-
-            return Promise.resolve({
-              statusCode: 204,
-              body: undefined,
-              options
-            });
-          });
-
-          const promise = webex.internal.metrics.callDiagnosticEventsBatcher.request({
-            type: 'diagnostic-event',
-            eventPayload: {
-              originTime: {
-                triggered: 'mock triggered timestamp'
-              }
-            }
-          });
-
-          return promiseTick(50)
-            .then(() => assert.lengthOf(webex.internal.metrics.callDiagnosticEventsBatcher.queue, 1))
-            .then(() => clock.tick(config.metrics.batcherWait))
-            .then(() => assert.calledOnce(webex.request))
-
-            .then(() => promiseTick(50))
-            .then(() => clock.tick(1000))
-            .then(() => promiseTick(50))
-            .then(() => clock.tick(config.metrics.batcherWait))
-            .then(() => assert.calledTwice(webex.request))
-
-            .then(() => promiseTick(50))
-            .then(() => clock.tick(2000))
-            .then(() => promiseTick(50))
-            .then(() => clock.tick(config.metrics.batcherWait))
-            .then(() => assert.calledThrice(webex.request))
-
-            .then(() => promiseTick(50))
-            .then(() => clock.tick(4000))
-            .then(() => promiseTick(50))
-            .then(() => clock.tick(config.metrics.batcherWait))
-            .then(() => assert.callCount(webex.request, 4))
-
-            .then(() => promiseTick(50))
-            .then(() => clock.tick(8000))
-            .then(() => promiseTick(50))
-            .then(() => clock.tick(config.metrics.batcherWait))
-            .then(() => assert.callCount(webex.request, 5))
-
-            .then(() => promiseTick(50))
-            .then(() => clock.tick(16000))
-            .then(() => promiseTick(50))
-            .then(() => clock.tick(config.metrics.batcherWait))
-            .then(() => assert.callCount(webex.request, 6))
-
-            .then(() => promiseTick(50))
-            .then(() => clock.tick(32000))
-            .then(() => promiseTick(50))
-            .then(() => clock.tick(config.metrics.batcherWait))
-            .then(() => assert.callCount(webex.request, 7))
-
-            .then(() => promiseTick(50))
-            .then(() => clock.tick(32000))
-            .then(() => promiseTick(50))
-            .then(() => clock.tick(config.metrics.batcherWait))
-            .then(() => assert.callCount(webex.request, 8))
-
-            .then(() => promiseTick(50))
-            .then(() => clock.tick(32000))
-            .then(() => promiseTick(50))
-            .then(() => clock.tick(config.metrics.batcherWait))
-            .then(() => assert.callCount(webex.request, 9))
-
-            .then(() => promiseTick(50))
-            .then(() => assert.lengthOf(webex.internal.metrics.callDiagnosticEventsBatcher.queue, 0))
-            .then(() => promise)
-            .then(() => {
-              assert.lengthOf(webex.request.args[1][0].body.metrics, 1, 'Reenqueuing the metric once did not increase the number of metrics to be submitted');
-              assert.lengthOf(webex.request.args[2][0].body.metrics, 1, 'Reenqueuing the metric twice did not increase the number of metrics to be submitted');
-              assert.lengthOf(webex.internal.metrics.callDiagnosticEventsBatcher.queue, 0);
-            });
-        });
-      });
-    });
-  });
-});