@webex/calling 3.12.0-next.93 → 3.12.0-next.94

package/dist/CallingClient/registration/register.test.js

@@ -33,7 +33,7 @@ var _types4 = require("../line/types");
 var _LineError = require("../../Errors/catalog/LineError");
 var _types5 = require("../../Metrics/types");
 var _request = require("../utils/request");
-function _interopRequireWildcard(e, t) { if ("function" == typeof _WeakMap) var r = new _WeakMap(), n = new _WeakMap(); return (_interopRequireWildcard = function _interopRequireWildcard(e, t) { if (!t && e && e.__esModule) return e; var o, i, f = { __proto__: null, default: e }; if (null === e || "object" != _typeof(e) && "function" != typeof e) return f; if (o = t ? n : r) { if (o.has(e)) return o.get(e); o.set(e, f); } for (var _t in e) "default" !== _t && {}.hasOwnProperty.call(e, _t) && ((i = (o = _Object$defineProperty) && _Object$getOwnPropertyDescriptor(e, _t)) && (i.get || i.set) ? o(f, _t, i) : f[_t] = e[_t]); return f; })(e, t); }
+function _interopRequireWildcard(e, t) { if ("function" == typeof _WeakMap) var r = new _WeakMap(), n = new _WeakMap(); return (_interopRequireWildcard = function _interopRequireWildcard(e, t) { if (!t && e && e.__esModule) return e; var o, i, f = { __proto__: null, default: e }; if (null === e || "object" != _typeof(e) && "function" != typeof e) return f; if (o = t ? n : r) { if (o.has(e)) return o.get(e); o.set(e, f); } for (var _t2 in e) "default" !== _t2 && {}.hasOwnProperty.call(e, _t2) && ((i = (o = _Object$defineProperty) && _Object$getOwnPropertyDescriptor(e, _t2)) && (i.get || i.set) ? o(f, _t2, i) : f[_t2] = e[_t2]); return f; })(e, t); }
 function ownKeys(e, r) { var t = _Object$keys2(e); if (_Object$getOwnPropertySymbols) { var o = _Object$getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return _Object$getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
 function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { (0, _defineProperty2.default)(e, r, t[r]); }) : _Object$getOwnPropertyDescriptors ? _Object$defineProperties(e, _Object$getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { _Object$defineProperty(e, r, _Object$getOwnPropertyDescriptor(t, r)); }); } return e; } /* eslint-disable @typescript-eslint/no-shadow */
 jest.mock('../../mobius-socket', function () {
@@ -2172,22 +2172,308 @@ describe('Registration Tests', function () {
           return _ref52.apply(this, arguments);
         };
       }());
+      it('stays terminal when the network recovers after a superseded session', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/_regenerator.default.mark(function _callee46() {
+        var retry;
+        return _regenerator.default.wrap(function (_context46) {
+          while (1) switch (_context46.prev = _context46.next) {
+            case 0:
+              _context46.next = 1;
+              return beforeEachSetupForKeepalive();
+            case 1:
+              lineEmitter.mockClear();
+              _context46.next = 2;
+              return sendKeepaliveFailure(conflictError, 1);
+            case 2:
+              expect(reg.sessionSuperseded).toStrictEqual(true);
+              restoreSpy.mockClear();
+              restartSpy.mockClear();
+              postRegistrationSpy.mockClear();
+              failoverSpy.mockClear();
+              _context46.next = 3;
+              return reg.handleConnectionRestoration(true);
+            case 3:
+              retry = _context46.sent;
+              /* The restoration runs, but every registration attempt it makes is refused at the
+               * choke point, so nothing reaches Mobius and no failover is rescheduled. */
+              expect(retry).toStrictEqual(false);
+              expect(warnSpy).toHaveBeenCalledWith("Session superseded, skipping registration attempt - caller: ".concat(_constants.METHODS.HANDLE_CONNECTION_RESTORATION), {
+                file: _constants.REGISTRATION_FILE,
+                method: _constants.METHODS.HANDLE_CONNECTION_RESTORATION
+              });
+              expect(postRegistrationSpy).not.toHaveBeenCalled();
+              expect(restartSpy).not.toHaveBeenCalled();
+              expect(failoverSpy).not.toHaveBeenCalled();
+              expect(reg.getStatus()).toEqual(_types.RegistrationStatus.INACTIVE);
+            case 4:
+            case "end":
+              return _context46.stop();
+          }
+        }, _callee46);
+      })));
+      it('abandons a restoration that was already queued when the session was superseded', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/_regenerator.default.mark(function _callee47() {
+        var release, restoration, _t;
+        return _regenerator.default.wrap(function (_context47) {
+          while (1) switch (_context47.prev = _context47.next) {
+            case 0:
+              _context47.next = 1;
+              return beforeEachSetupForKeepalive();
+            case 1:
+              lineEmitter.mockClear();
+              postRegistrationSpy.mockClear();
+              restartSpy.mockClear();
+              failoverSpy.mockClear();
+
+              /* Something else holds the mutex, so the restoration queues behind it having
+               * begun while the session was still valid. */
+              _context47.next = 2;
+              return reg.mutex.acquire();
+            case 2:
+              release = _context47.sent;
+              restoration = reg.handleConnectionRestoration(true);
+              _context47.next = 3;
+              return flushPromises();
+            case 3:
+              /* The 409 cleanup latches without the mutex, so the flip lands mid-wait. */
+              reg.sessionSuperseded = true;
+              release();
+              _t = expect;
+              _context47.next = 4;
+              return restoration;
+            case 4:
+              _t(_context47.sent).toStrictEqual(false);
+              /* The latch flipped after this restoration had already started, so the choke
+               * point is what stops it: no device is registered for the dead session. */
+              expect(warnSpy).toHaveBeenCalledWith("Session superseded, skipping registration attempt - caller: ".concat(_constants.METHODS.HANDLE_CONNECTION_RESTORATION), {
+                file: _constants.REGISTRATION_FILE,
+                method: _constants.METHODS.HANDLE_CONNECTION_RESTORATION
+              });
+              expect(postRegistrationSpy).not.toHaveBeenCalled();
+              expect(restartSpy).not.toHaveBeenCalled();
+              expect(failoverSpy).not.toHaveBeenCalled();
+            case 5:
+            case "end":
+              return _context47.stop();
+          }
+        }, _callee47);
+      })));
+      it('stays terminal when all calls are cleared after a superseded session', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/_regenerator.default.mark(function _callee48() {
+        return _regenerator.default.wrap(function (_context48) {
+          while (1) switch (_context48.prev = _context48.next) {
+            case 0:
+              _context48.next = 1;
+              return beforeEachSetupForKeepalive();
+            case 1:
+              lineEmitter.mockClear();
+              _context48.next = 2;
+              return sendKeepaliveFailure(conflictError, 1);
+            case 2:
+              postRegistrationSpy.mockClear();
+              restartSpy.mockClear();
+              failoverSpy.mockClear();
+              _context48.next = 3;
+              return reg.reconnectOnFailure(_constants.CALLS_CLEARED_HANDLER_UTIL);
+            case 3:
+              expect(warnSpy).toHaveBeenCalledWith("Session superseded, skipping registration attempt - caller: ".concat(_constants.CALLS_CLEARED_HANDLER_UTIL), {
+                file: _constants.REGISTRATION_FILE,
+                method: _constants.CALLS_CLEARED_HANDLER_UTIL
+              });
+              expect(postRegistrationSpy).not.toHaveBeenCalled();
+              expect(restartSpy).not.toHaveBeenCalled();
+              expect(failoverSpy).not.toHaveBeenCalled();
+              expect(reg.getStatus()).toEqual(_types.RegistrationStatus.INACTIVE);
+            case 4:
+            case "end":
+              return _context48.stop();
+          }
+        }, _callee48);
+      })));
+      it('lets an explicit registration by the consumer clear the superseded state', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/_regenerator.default.mark(function _callee49() {
+        return _regenerator.default.wrap(function (_context49) {
+          while (1) switch (_context49.prev = _context49.next) {
+            case 0:
+              _context49.next = 1;
+              return beforeEachSetupForKeepalive();
+            case 1:
+              lineEmitter.mockClear();
+              _context49.next = 2;
+              return sendKeepaliveFailure(conflictError, 1);
+            case 2:
+              expect(reg.sessionSuperseded).toStrictEqual(true);
+              postRegistrationSpy.mockResolvedValueOnce(successPayload);
+              _context49.next = 3;
+              return reg.triggerRegistration();
+            case 3:
+              expect(reg.sessionSuperseded).toStrictEqual(false);
+              expect(reg.getStatus()).toBe(_types.RegistrationStatus.ACTIVE);
+            case 4:
+            case "end":
+              return _context49.stop();
+          }
+        }, _callee49);
+      })));
+      it('latches the superseded state before waiting on the shared mutex', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/_regenerator.default.mark(function _callee50() {
+        var worker, release, inFlight, abort;
+        return _regenerator.default.wrap(function (_context50) {
+          while (1) switch (_context50.prev = _context50.next) {
+            case 0:
+              _context50.next = 1;
+              return beforeEachSetupForKeepalive();
+            case 1:
+              worker = reg.webWorker;
+              lineEmitter.mockClear();
+              postRegistrationSpy.mockClear();
+
+              /* A recovery already holds the mutex, so the 409 cleanup queues behind it. */
+              _context50.next = 2;
+              return reg.mutex.acquire();
+            case 2:
+              release = _context50.sent;
+              inFlight = worker.onmessage({
+                data: {
+                  type: _types.WorkerMessageType.KEEPALIVE_FAILURE,
+                  err: conflictError,
+                  keepAliveRetryCount: 1
+                }
+              });
+              _context50.next = 3;
+              return flushPromises();
+            case 3:
+              /* The latch and the worker teardown are visible before cleanup has run. */
+              expect(reg.sessionSuperseded).toStrictEqual(true);
+              expect(reg.webWorker).toBeUndefined();
+              expect(lineEmitter).not.toHaveBeenCalledWith(_types4.LINE_EVENTS.UNREGISTERED, undefined, expect.anything());
+
+              /* A registration path starting now aborts instead of racing the queued cleanup. */
+              _context50.next = 4;
+              return reg.attemptRegistrationWithServers(_constants.FAILOVER_UTIL);
+            case 4:
+              abort = _context50.sent;
+              expect(abort).toStrictEqual(true);
+              expect(postRegistrationSpy).not.toHaveBeenCalled();
+              expect(warnSpy).toHaveBeenCalledWith("Session superseded, skipping registration attempt - caller: ".concat(_constants.FAILOVER_UTIL), {
+                file: _constants.REGISTRATION_FILE,
+                method: _constants.FAILOVER_UTIL
+              });
+              release();
+              _context50.next = 5;
+              return inFlight;
+            case 5:
+              _context50.next = 6;
+              return flushPromises();
+            case 6:
+              expect(reg.getStatus()).toEqual(_types.RegistrationStatus.INACTIVE);
+            case 7:
+            case "end":
+              return _context50.stop();
+          }
+        }, _callee50);
+      })));
+      it('discards a 409 delivered by a keepalive worker that has been replaced', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/_regenerator.default.mark(function _callee51() {
+        var staleWorker, staleWorkerPostSpy, handle409Spy, failKeepalive, inFlight, currentWorker;
+        return _regenerator.default.wrap(function (_context51) {
+          while (1) switch (_context51.prev = _context51.next) {
+            case 0:
+              _context51.next = 1;
+              return beforeEachSetupForKeepalive();
+            case 1:
+              staleWorker = reg.webWorker;
+              staleWorkerPostSpy = jest.spyOn(staleWorker, 'postMessage');
+              handle409Spy = jest.spyOn(reg, 'handle409KeepaliveFailure');
+              jest.spyOn(reg, 'postKeepAlive').mockReturnValue(new _promise.default(function (_, reject) {
+                failKeepalive = reject;
+              }));
+              lineEmitter.mockClear();
+              inFlight = staleWorker.onmessage({
+                data: {
+                  type: _types.WorkerMessageType.SEND_KEEPALIVE
+                }
+              });
+              /* A re-registration (failback, connection restoration) swaps the worker while the
+               * keepalive of the previous registration is still in flight. */
+              currentWorker = {
+                postMessage: jest.fn(),
+                terminate: jest.fn(),
+                onmessage: null
+              };
+              reg.webWorker = currentWorker;
+              failKeepalive(conflictError);
+              _context51.next = 2;
+              return inFlight;
+            case 2:
+              _context51.next = 3;
+              return flushPromises();
+            case 3:
+              expect(staleWorkerPostSpy).not.toHaveBeenCalledWith(expect.objectContaining({
+                type: _types.WorkerMessageType.KEEPALIVE_RESULT
+              }));
+              expect(currentWorker.postMessage).not.toHaveBeenCalled();
+              expect(handle409Spy).not.toHaveBeenCalled();
+              expect(lineEmitter).not.toHaveBeenCalledWith(_types4.LINE_EVENTS.UNREGISTERED, undefined, expect.anything());
+            case 4:
+            case "end":
+              return _context51.stop();
+          }
+        }, _callee51);
+      })));
     });
   });
+  describe('409 Conflict outside the keepalive flow', function () {
+    /* The registration flow passes no sessionSupersededCb, so a 409 there is logged and
+     * ignored: no consumer event, no registration-error metric, and non-final so the
+     * server loop and failover proceed as they would for any non-fatal status. */
+    it('ignores a 409 during registration without notifying the consumer', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/_regenerator.default.mark(function _callee52() {
+      var conflictPayload;
+      return _regenerator.default.wrap(function (_context52) {
+        while (1) switch (_context52.prev = _context52.next) {
+          case 0:
+            jest.useFakeTimers();
+            conflictPayload = {
+              statusCode: 409,
+              headers: {
+                trackingid: 'tid-reg-409'
+              }
+            };
+            webex.request.mockRejectedValue(conflictPayload);
+            lineEmitter.mockClear();
+            metricSpy.mockClear();
+            _context52.next = 1;
+            return reg.triggerRegistration();
+          case 1:
+            expect(warnSpy).toHaveBeenCalledWith('409 Conflict: ignored, caller does not handle a superseded session', {
+              file: _constants.REGISTRATION_FILE,
+              method: _constants.REGISTRATION_UTIL
+            });
+
+            /* No consumer notification and no registration-error metric on this path. */
+            expect(lineEmitter).not.toHaveBeenCalledWith(_types4.LINE_EVENTS.UNREGISTERED);
+            expect(lineEmitter).not.toHaveBeenCalledWith(_types4.LINE_EVENTS.ERROR, undefined, expect.anything());
+            expect(metricSpy).not.toHaveBeenCalledWith(_types5.METRIC_EVENT.REGISTRATION_ERROR, _types5.REG_ACTION.REGISTER, _types5.METRIC_TYPE.BEHAVIORAL, expect.anything(), expect.anything(), expect.anything(), expect.anything(), expect.anything());
+
+            /* Non-final: every server was tried and failover was scheduled. */
+            expect(reg.getStatus()).toEqual(_types.RegistrationStatus.INACTIVE);
+            expect(reg.sessionSuperseded).toStrictEqual(false);
+            expect(failoverSpy).toHaveBeenCalled();
+          case 2:
+          case "end":
+            return _context52.stop();
+        }
+      }, _callee52);
+    })));
+  });
   describe('Primary server status checks', function () {
-    it('success: primary server status to be up', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/_regenerator.default.mark(function _callee46() {
+    it('success: primary server status to be up', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/_regenerator.default.mark(function _callee53() {
       var pingSuccessPayload, status;
-      return _regenerator.default.wrap(function (_context46) {
-        while (1) switch (_context46.prev = _context46.next) {
+      return _regenerator.default.wrap(function (_context53) {
+        while (1) switch (_context53.prev = _context53.next) {
           case 0:
             pingSuccessPayload = {
               statusCode: 200
             };
             webex.request.mockResolvedValue(pingSuccessPayload);
-            _context46.next = 1;
+            _context53.next = 1;
             return reg.isPrimaryActive();
           case 1:
-            status = _context46.sent;
+            status = _context53.sent;
             expect(webex.request).toBeCalledWith(_objectSpread({
               method: 'GET',
               uri: "https://mobius-dfw.webex.com/api/v1/ping"
@@ -2195,23 +2481,23 @@ describe('Registration Tests', function () {
             expect(status).toEqual(true);
           case 2:
           case "end":
-            return _context46.stop();
+            return _context53.stop();
         }
-      }, _callee46);
+      }, _callee53);
     })));
-    it('failed: primary server status to be down', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/_regenerator.default.mark(function _callee47() {
+    it('failed: primary server status to be down', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/_regenerator.default.mark(function _callee54() {
       var pingFailurePayload, status;
-      return _regenerator.default.wrap(function (_context47) {
-        while (1) switch (_context47.prev = _context47.next) {
+      return _regenerator.default.wrap(function (_context54) {
+        while (1) switch (_context54.prev = _context54.next) {
           case 0:
             pingFailurePayload = {
               statusCode: 500
             };
             webex.request.mockResolvedValue(pingFailurePayload);
-            _context47.next = 1;
+            _context54.next = 1;
             return reg.isPrimaryActive();
           case 1:
-            status = _context47.sent;
+            status = _context54.sent;
             expect(webex.request).toBeCalledWith(_objectSpread({
               method: 'GET',
               uri: "https://mobius-dfw.webex.com/api/v1/ping"
@@ -2219,9 +2505,9 @@ describe('Registration Tests', function () {
             expect(status).toEqual(false);
           case 2:
           case "end":
-            return _context47.stop();
+            return _context54.stop();
         }
-      }, _callee47);
+      }, _callee54);
     })));
   });
   describe('handleRegistrationDownEvent tests', function () {
@@ -2241,20 +2527,20 @@ describe('Registration Tests', function () {
         }
       }
     };
-    beforeEach(/*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/_regenerator.default.mark(function _callee48() {
-      return _regenerator.default.wrap(function (_context48) {
-        while (1) switch (_context48.prev = _context48.next) {
+    beforeEach(/*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/_regenerator.default.mark(function _callee55() {
+      return _regenerator.default.wrap(function (_context55) {
+        while (1) switch (_context55.prev = _context55.next) {
           case 0:
             postRegistrationSpy.mockResolvedValueOnce(successPayload);
-            _context48.next = 1;
+            _context55.next = 1;
             return reg.triggerRegistration();
           case 1:
             expect(reg.getStatus()).toBe(_types.RegistrationStatus.ACTIVE);
           case 2:
           case "end":
-            return _context48.stop();
+            return _context55.stop();
         }
-      }, _callee48);
+      }, _callee55);
     })));
     afterEach(function () {
       var calls = (0, _values.default)(reg.callManager.getActiveCalls());
@@ -2263,10 +2549,10 @@ describe('Registration Tests', function () {
       });
       reg.callManager.callCollection = {};
     });
-    it('runs cleanup immediately when no active calls are present (socket disabled)', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/_regenerator.default.mark(function _callee49() {
+    it('runs cleanup immediately when no active calls are present (socket disabled)', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/_regenerator.default.mark(function _callee56() {
       var clearKeepaliveSpy, setStatusSpy, disconnectSocketSpy;
-      return _regenerator.default.wrap(function (_context49) {
-        while (1) switch (_context49.prev = _context49.next) {
+      return _regenerator.default.wrap(function (_context56) {
+        while (1) switch (_context56.prev = _context56.next) {
           case 0:
             clearKeepaliveSpy = jest.spyOn(reg, 'clearKeepaliveTimer');
             setStatusSpy = jest.spyOn(reg, 'setStatus');
@@ -2275,7 +2561,7 @@ describe('Registration Tests', function () {
             }), 'disconnectFromMobiusSocket');
             lineEmitter.mockClear();
             expect((0, _keys.default)(reg.callManager.getActiveCalls()).length).toBe(0);
-            _context49.next = 1;
+            _context56.next = 1;
             return reg.handleRegistrationDownEvent(registrationDownEvent);
           case 1:
             expect(clearKeepaliveSpy).toHaveBeenCalled();
@@ -2291,14 +2577,14 @@ describe('Registration Tests', function () {
             expect(lineEmitter).toHaveBeenCalledWith(_types4.LINE_EVENTS.UNREGISTERED, undefined, undefined);
           case 2:
           case "end":
-            return _context49.stop();
+            return _context56.stop();
         }
-      }, _callee49);
+      }, _callee56);
     })));
-    it('ends the active call and still runs cleanup when an active call is present', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/_regenerator.default.mark(function _callee50() {
+    it('ends the active call and still runs cleanup when an active call is present', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/_regenerator.default.mark(function _callee57() {
       var clearKeepaliveSpy, setStatusSpy, activeCall, endSpy;
-      return _regenerator.default.wrap(function (_context50) {
-        while (1) switch (_context50.prev = _context50.next) {
+      return _regenerator.default.wrap(function (_context57) {
+        while (1) switch (_context57.prev = _context57.next) {
           case 0:
             clearKeepaliveSpy = jest.spyOn(reg, 'clearKeepaliveTimer');
             setStatusSpy = jest.spyOn(reg, 'setStatus');
@@ -2306,7 +2592,7 @@ describe('Registration Tests', function () {
             endSpy = jest.spyOn(activeCall, 'end');
             expect((0, _keys.default)(reg.callManager.getActiveCalls()).length).toBe(1);
             lineEmitter.mockClear();
-            _context50.next = 1;
+            _context57.next = 1;
             return reg.handleRegistrationDownEvent(registrationDownEvent);
           case 1:
             expect(endSpy).toHaveBeenCalledTimes(1);
@@ -2316,18 +2602,18 @@ describe('Registration Tests', function () {
             expect(lineEmitter).toHaveBeenCalledWith(_types4.LINE_EVENTS.UNREGISTERED, undefined, undefined);
           case 2:
           case "end":
-            return _context50.stop();
+            return _context57.stop();
         }
-      }, _callee50);
+      }, _callee57);
     })));
-    it('runs cleanup without calling end when no active call is present on re-invocation', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/_regenerator.default.mark(function _callee51() {
+    it('runs cleanup without calling end when no active call is present on re-invocation', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/_regenerator.default.mark(function _callee58() {
       var clearKeepaliveSpy, setStatusSpy;
-      return _regenerator.default.wrap(function (_context51) {
-        while (1) switch (_context51.prev = _context51.next) {
+      return _regenerator.default.wrap(function (_context58) {
+        while (1) switch (_context58.prev = _context58.next) {
           case 0:
             clearKeepaliveSpy = jest.spyOn(reg, 'clearKeepaliveTimer');
             setStatusSpy = jest.spyOn(reg, 'setStatus');
-            _context51.next = 1;
+            _context58.next = 1;
             return reg.handleRegistrationDownEvent(registrationDownEvent);
           case 1:
             expect(reg.getStatus()).toBe(_types.RegistrationStatus.INACTIVE);
@@ -2335,7 +2621,7 @@ describe('Registration Tests', function () {
             setStatusSpy.mockClear();
             lineEmitter.mockClear();
             expect((0, _keys.default)(reg.callManager.getActiveCalls()).length).toBe(0);
-            _context51.next = 2;
+            _context58.next = 2;
             return reg.handleRegistrationDownEvent(registrationDownEvent);
           case 2:
             expect(clearKeepaliveSpy).toHaveBeenCalled();
@@ -2344,14 +2630,14 @@ describe('Registration Tests', function () {
             expect(lineEmitter).toHaveBeenCalledWith(_types4.LINE_EVENTS.UNREGISTERED, undefined, undefined);
           case 3:
           case "end":
-            return _context51.stop();
+            return _context58.stop();
         }
-      }, _callee51);
+      }, _callee58);
     })));
-    it('disconnects the Mobius WebSocket when socket is enabled', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/_regenerator.default.mark(function _callee52() {
+    it('disconnects the Mobius WebSocket when socket is enabled', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/_regenerator.default.mark(function _callee59() {
       var apiRequest, disconnectSocketSpy;
-      return _regenerator.default.wrap(function (_context52) {
-        while (1) switch (_context52.prev = _context52.next) {
+      return _regenerator.default.wrap(function (_context59) {
+        while (1) switch (_context59.prev = _context59.next) {
           case 0:
             apiRequest = _request.APIRequest.getInstance({
               webex: webex
@@ -2360,7 +2646,7 @@ describe('Registration Tests', function () {
             disconnectSocketSpy = jest.spyOn(apiRequest, 'disconnectFromMobiusSocket').mockResolvedValue();
             lineEmitter.mockClear();
             expect((0, _keys.default)(reg.callManager.getActiveCalls()).length).toBe(0);
-            _context52.next = 1;
+            _context59.next = 1;
             return reg.handleRegistrationDownEvent(registrationDownEvent);
           case 1:
             expect(disconnectSocketSpy).toHaveBeenCalledWith({
@@ -2371,14 +2657,14 @@ describe('Registration Tests', function () {
             expect(lineEmitter).toHaveBeenCalledWith(_types4.LINE_EVENTS.UNREGISTERED, undefined, undefined);
           case 2:
           case "end":
-            return _context52.stop();
+            return _context59.stop();
         }
-      }, _callee52);
+      }, _callee59);
     })));
-    it('still emits UNREGISTERED when socket disconnect fails', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/_regenerator.default.mark(function _callee53() {
+    it('still emits UNREGISTERED when socket disconnect fails', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/_regenerator.default.mark(function _callee60() {
       var apiRequest, disconnectSocketSpy;
-      return _regenerator.default.wrap(function (_context53) {
-        while (1) switch (_context53.prev = _context53.next) {
+      return _regenerator.default.wrap(function (_context60) {
+        while (1) switch (_context60.prev = _context60.next) {
           case 0:
             apiRequest = _request.APIRequest.getInstance({
               webex: webex
@@ -2386,7 +2672,7 @@ describe('Registration Tests', function () {
             jest.spyOn(apiRequest, 'isSocketEnabled').mockReturnValue(true);
             disconnectSocketSpy = jest.spyOn(apiRequest, 'disconnectFromMobiusSocket').mockRejectedValue(new Error('socket teardown failed'));
             lineEmitter.mockClear();
-            _context53.next = 1;
+            _context60.next = 1;
             return reg.handleRegistrationDownEvent(registrationDownEvent);
           case 1:
             expect(disconnectSocketSpy).toHaveBeenCalled();
@@ -2398,9 +2684,9 @@ describe('Registration Tests', function () {
             expect(lineEmitter).toHaveBeenCalledWith(_types4.LINE_EVENTS.UNREGISTERED, undefined, undefined);
           case 2:
           case "end":
-            return _context53.stop();
+            return _context60.stop();
         }
-      }, _callee53);
+      }, _callee60);
     })));
   });
   describe('attemptRegistrationWithServers transport alignment', function () {
@@ -2415,61 +2701,61 @@ describe('Registration Tests', function () {
       setSocketEnabledSpy = jest.spyOn(apiRequest, 'setSocketEnabled').mockImplementation(function () {});
       webex.request.mockResolvedValue(successPayload);
     });
-    it('enables the WebSocket transport when the server group uses the wss:// scheme', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/_regenerator.default.mark(function _callee54() {
-      return _regenerator.default.wrap(function (_context54) {
-        while (1) switch (_context54.prev = _context54.next) {
+    it('enables the WebSocket transport when the server group uses the wss:// scheme', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/_regenerator.default.mark(function _callee61() {
+      return _regenerator.default.wrap(function (_context61) {
+        while (1) switch (_context61.prev = _context61.next) {
           case 0:
-            _context54.next = 1;
+            _context61.next = 1;
             return reg.attemptRegistrationWithServers(_constants.REGISTRATION_UTIL, ['wss://mobius.example.com/api/v1/calling/web/']);
           case 1:
             expect(setSocketEnabledSpy).toHaveBeenCalledWith(true);
           case 2:
           case "end":
-            return _context54.stop();
+            return _context61.stop();
         }
-      }, _callee54);
+      }, _callee61);
     })));
-    it('falls back to the HTTP transport when the server group uses the https:// scheme', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/_regenerator.default.mark(function _callee55() {
-      return _regenerator.default.wrap(function (_context55) {
-        while (1) switch (_context55.prev = _context55.next) {
+    it('falls back to the HTTP transport when the server group uses the https:// scheme', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/_regenerator.default.mark(function _callee62() {
+      return _regenerator.default.wrap(function (_context62) {
+        while (1) switch (_context62.prev = _context62.next) {
           case 0:
-            _context55.next = 1;
+            _context62.next = 1;
             return reg.attemptRegistrationWithServers(_constants.REGISTRATION_UTIL, ['https://mobius.example.com/api/v1/calling/web/']);
           case 1:
             expect(setSocketEnabledSpy).toHaveBeenCalledWith(false);
           case 2:
           case "end":
-            return _context55.stop();
+            return _context62.stop();
         }
-      }, _callee55);
+      }, _callee62);
     })));
-    it('does not change the transport when the server group is empty', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/_regenerator.default.mark(function _callee56() {
-      return _regenerator.default.wrap(function (_context56) {
-        while (1) switch (_context56.prev = _context56.next) {
+    it('does not change the transport when the server group is empty', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/_regenerator.default.mark(function _callee63() {
+      return _regenerator.default.wrap(function (_context63) {
+        while (1) switch (_context63.prev = _context63.next) {
           case 0:
-            _context56.next = 1;
+            _context63.next = 1;
             return reg.attemptRegistrationWithServers(_constants.REGISTRATION_UTIL, []);
           case 1:
             expect(setSocketEnabledSpy).not.toHaveBeenCalled();
           case 2:
           case "end":
-            return _context56.stop();
+            return _context63.stop();
         }
-      }, _callee56);
+      }, _callee63);
     })));
-    it('derives the transport from the first server in the group', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/_regenerator.default.mark(function _callee57() {
-      return _regenerator.default.wrap(function (_context57) {
-        while (1) switch (_context57.prev = _context57.next) {
+    it('derives the transport from the first server in the group', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/_regenerator.default.mark(function _callee64() {
+      return _regenerator.default.wrap(function (_context64) {
+        while (1) switch (_context64.prev = _context64.next) {
           case 0:
-            _context57.next = 1;
+            _context64.next = 1;
             return reg.attemptRegistrationWithServers(_constants.REGISTRATION_UTIL, ['wss://primary.example.com/api/v1/calling/web/', 'https://backup.example.com/api/v1/calling/web/']);
           case 1:
             expect(setSocketEnabledSpy).toHaveBeenCalledWith(true);
           case 2:
           case "end":
-            return _context57.stop();
+            return _context64.stop();
         }
-      }, _callee57);
+      }, _callee64);
     })));
   });
 });