@webex/internal-plugin-mercury 3.0.0-beta.9 → 3.0.0-beta.91

package/src/socket/socket-base.js

@@ -14,7 +14,7 @@ import {
   ConnectionError,
   Forbidden,
   NotAuthorized,
-  UnknownResponse
+  UnknownResponse,
   // NotFound
 } from '../errors';
 
@@ -88,7 +88,9 @@ export default class Socket extends EventEmitter {
    * @returns {WebSocket}
    */
   static getWebSocketConstructor() {
-    throw new Error('Socket.getWebSocketConstructor() must be implemented in an environmentally appropriate way');
+    throw new Error(
+      'Socket.getWebSocketConstructor() must be implemented in an environmentally appropriate way'
+    );
   }
 
   /**
@@ -127,18 +129,19 @@ export default class Socket extends EventEmitter {
 
       options = defaults(options, {
         code: 1000,
-        reason: 'Done'
+        reason: 'Done',
       });
 
       const closeTimer = safeSetTimeout(() => {
         try {
           this.logger.info('socket: no close event received, forcing closure');
-          resolve(this.onclose({
-            code: 1000,
-            reason: 'Done (forced)'
-          }));
-        }
-        catch (error) {
+          resolve(
+            this.onclose({
+              code: 1000,
+              reason: 'Done (forced)',
+            })
+          );
+        } catch (error) {
           this.logger.warn('socket: force-close failed', error);
         }
       }, this.forceCloseDelay);
@@ -184,19 +187,15 @@ export default class Socket extends EventEmitter {
 
       options = options || {};
 
-      checkRequired([
-        'forceCloseDelay',
-        'pingInterval',
-        'pongTimeout',
-        'token',
-        'trackingId',
-        'logger'
-      ], options);
+      checkRequired(
+        ['forceCloseDelay', 'pingInterval', 'pongTimeout', 'token', 'trackingId', 'logger'],
+        options
+      );
 
       Object.keys(options).forEach((key) => {
         Reflect.defineProperty(this, key, {
           enumerable: false,
-          value: options[key]
+          value: options[key],
         });
       });
 
@@ -213,10 +212,10 @@ export default class Socket extends EventEmitter {
         this.logger.info('socket: closed before open', event.code, event.reason);
         switch (event.code) {
           case 1005:
-          // IE 11 doesn't seem to allow 4XXX codes, so if we get a 1005, assume
-          // it's a bad websocket url. That'll trigger a device refresh; if it
-          // turns out we had a bad token, the device refresh should 401 and
-          // trigger a token refresh.
+            // IE 11 doesn't seem to allow 4XXX codes, so if we get a 1005, assume
+            // it's a bad websocket url. That'll trigger a device refresh; if it
+            // turns out we had a bad token, the device refresh should 401 and
+            // trigger a token refresh.
             return reject(new UnknownResponse(event));
           case 4400:
             return reject(new BadRequest(event));
@@ -224,8 +223,8 @@ export default class Socket extends EventEmitter {
             return reject(new NotAuthorized(event));
           case 4403:
             return reject(new Forbidden(event));
-            // case 4404:
-            //   return reject(new NotFound(event));
+          // case 4404:
+          //   return reject(new NotFound(event));
           default:
             return reject(new ConnectionError(event));
         }
@@ -281,7 +280,9 @@ export default class Socket extends EventEmitter {
 
       this.logger.debug('socket: sequence number: ', sequenceNumber);
       if (this.expectedSequenceNumber && sequenceNumber !== this.expectedSequenceNumber) {
-        this.logger.debug(`socket: sequence number mismatch indicates lost mercury message. expected: ${this.expectedSequenceNumber}, actual: ${sequenceNumber}`);
+        this.logger.debug(
+          `socket: sequence number mismatch indicates lost mercury message. expected: ${this.expectedSequenceNumber}, actual: ${sequenceNumber}`
+        );
         this.emit('sequence-mismatch', sequenceNumber, this.expectedSequenceNumber);
       }
       this.expectedSequenceNumber = sequenceNumber + 1;
@@ -294,12 +295,10 @@ export default class Socket extends EventEmitter {
       this._acknowledge(processedEvent);
       if (data.type === 'pong') {
         this.emit('pong', processedEvent);
-      }
-      else {
+      } else {
         this.emit('message', processedEvent);
       }
-    }
-    catch (error) {
+    } catch (error) {
       // The above code should only be able to throw if we receive an unparsable
       // message from Mercury. At this time, the only action we have is to
       // ignore it and move on.
@@ -347,7 +346,7 @@ export default class Socket extends EventEmitter {
 
     return this.send({
       messageId: event.data.id,
-      type: 'ack'
+      type: 'ack',
     });
   }
 
@@ -363,14 +362,18 @@ export default class Socket extends EventEmitter {
         id: uuid.v4(),
         type: 'authorization',
         data: {
-          token: this.token
+          token: this.token,
         },
         trackingId: this.trackingId,
-        logLevelToken: this.logLevelToken
+        logLevelToken: this.logLevelToken,
       });
 
       const waitForBufferState = (event) => {
-        if (!event.data.type && (event.data.data.eventType === 'mercury.buffer_state' || event.data.data.eventType === 'mercury.registration_status')) {
+        if (
+          !event.data.type &&
+          (event.data.data.eventType === 'mercury.buffer_state' ||
+            event.data.data.eventType === 'mercury.registration_status')
+        ) {
           this.removeListener('message', waitForBufferState);
           this._ping();
           resolve();
@@ -423,11 +426,10 @@ export default class Socket extends EventEmitter {
           this.logger.debug('socket: expected', id, 'received', event.data.id);
           this.close({
             code: 1000,
-            reason: 'Pong mismatch'
+            reason: 'Pong mismatch',
           });
         }
-      }
-      catch (error) {
+      } catch (error) {
         // This try/catch block was added as a debugging step; to the best of my
         // knowledge, the above can never throw.
         /* istanbul ignore next */
@@ -440,13 +442,11 @@ export default class Socket extends EventEmitter {
         this.logger.info('socket: pong not receive in expected period, closing socket');
         this.close({
           code: 1000,
-          reason: 'Pong not received'
-        })
-          .catch((reason) => {
-            this.logger.warn('socket: failed to close socket after missed pong', reason);
-          });
-      }
-      catch (error) {
+          reason: 'Pong not received',
+        }).catch((reason) => {
+          this.logger.warn('socket: failed to close socket after missed pong', reason);
+        });
+      } catch (error) {
         // This try/catch block was added as a debugging step; to the best of my
         // knowledge, the above can never throw.
         /* istanbul ignore next */
@@ -458,8 +458,7 @@ export default class Socket extends EventEmitter {
       try {
         clearTimeout(this.pongTimer);
         this.pingTimer = safeSetTimeout(() => this._ping(), this.pingInterval);
-      }
-      catch (error) {
+      } catch (error) {
         // This try/catch block was added as a debugging step; to the best of my
         // knowledge, the above can never throw.
         /* istanbul ignore next */
@@ -476,7 +475,7 @@ export default class Socket extends EventEmitter {
 
     return this.send({
       id,
-      type: 'ping'
+      type: 'ping',
     });
   }
 }

package/src/socket/socket.shim.js

@@ -1,3 +1,5 @@
+/* eslint-disable no-restricted-globals */
+
 /*!
  * Copyright (c) 2015-2020 Cisco Systems, Inc. See LICENSE file.
  */
@@ -12,18 +14,14 @@ Socket.getWebSocketConstructor = function getWebSocketConstructor() {
 
   if (typeof WebSocket !== 'undefined') {
     ws = WebSocket;
-  }
-  else if (typeof MozWebSocket !== 'undefined') {
+  } else if (typeof MozWebSocket !== 'undefined') {
     // eslint-disable-next-line no-undef
     ws = MozWebSocket;
-  }
-  else if (typeof global !== 'undefined') {
+  } else if (typeof global !== 'undefined') {
     ws = global.WebSocket || global.MozWebSocket;
-  }
-  else if (typeof window !== 'undefined') {
+  } else if (typeof window !== 'undefined') {
     ws = window.WebSocket || window.MozWebSocket;
-  }
-  else if (typeof self !== 'undefined') {
+  } else if (typeof self !== 'undefined') {
     ws = self.WebSocket || self.MozWebSocket;
   }
 

package/test/integration/spec/mercury.js

@@ -16,34 +16,37 @@ describe('plugin-mercury', function () {
   describe('Mercury', () => {
     let webex;
 
-    beforeEach(() => testUsers.create({count: 1})
-      .then((users) => {
+    beforeEach(() =>
+      testUsers.create({count: 1}).then((users) => {
         webex = new WebexCore({
           credentials: {
-            supertoken: users[0].token
+            supertoken: users[0].token,
           },
           config: {
             credentials: {
-              refreshCallback
-            }
-          }
+              refreshCallback,
+            },
+          },
         });
-      }));
+      })
+    );
 
     afterEach(() => webex && webex.internal.mercury.disconnect());
 
     describe('#connect()', () => {
       it('connects to mercury', () => webex.internal.mercury.connect());
 
-      it('refreshes the access token when a 4401 is received', () => webex.internal.device.register()
-        .then(() => {
-          // eslint-disable-next-line camelcase
-          webex.credentials.supertoken.access_token = 'fake token';
+      it('refreshes the access token when a 4401 is received', () =>
+        webex.internal.device
+          .register()
+          .then(() => {
+            // eslint-disable-next-line camelcase
+            webex.credentials.supertoken.access_token = 'fake token';
 
-          return webex.internal.mercury.connect();
-        })
-        // eslint-disable-next-line camelcase
-        .then(() => assert.notEqual(webex.credentials.supertoken.access_token, 'fake token')));
+            return webex.internal.mercury.connect();
+          })
+          // eslint-disable-next-line camelcase
+          .then(() => assert.notEqual(webex.credentials.supertoken.access_token, 'fake token')));
 
       // This doesn't work as designed yet. The only way to get a 4404 is to try
       // to connect to someone else's valid registration; the intent was to get
@@ -70,26 +73,34 @@ describe('plugin-mercury', function () {
       });
 
       describe('when web-high-availability is enabled', () => {
-        flaky(it, process.env.SKIP_FLAKY_TESTS)('connects to mercury using service catalog url', () => {
-          let defaultWebSocketUrl;
-
-          // we need to ensure the feature is set for user before "registering"
-          // the device
-          return webex.internal.device.register()
-            .then(() => webex.internal.feature.setFeature('developer', 'web-high-availability', true))
-            .then(() => webex.internal.device.unregister())
-            // start the test flow the device list
-            .then(() => webex.internal.device.register())
-            .then(() => {
-              defaultWebSocketUrl = webex.internal.device.webSocketUrl;
-            })
-            .then(() => webex.internal.mercury.connect())
-            .then(() => webex.internal.device.getWebSocketUrl())
-            .then((wsUrl) => {
-              assert.notEqual(defaultWebSocketUrl, webex.internal.mercury.socket.url);
-              assert.include(webex.internal.mercury.socket.url, wsUrl);
-            });
-        });
+        flaky(it, process.env.SKIP_FLAKY_TESTS)(
+          'connects to mercury using service catalog url',
+          () => {
+            let defaultWebSocketUrl;
+
+            // we need to ensure the feature is set for user before "registering"
+            // the device
+            return (
+              webex.internal.device
+                .register()
+                .then(() =>
+                  webex.internal.feature.setFeature('developer', 'web-high-availability', true)
+                )
+                .then(() => webex.internal.device.unregister())
+                // start the test flow the device list
+                .then(() => webex.internal.device.register())
+                .then(() => {
+                  defaultWebSocketUrl = webex.internal.device.webSocketUrl;
+                })
+                .then(() => webex.internal.mercury.connect())
+                .then(() => webex.internal.device.getWebSocketUrl())
+                .then((wsUrl) => {
+                  assert.notEqual(defaultWebSocketUrl, webex.internal.mercury.socket.url);
+                  assert.include(webex.internal.mercury.socket.url, wsUrl);
+                })
+            );
+          }
+        );
       });
     });
 
@@ -98,10 +109,9 @@ describe('plugin-mercury', function () {
 
       webex.internal.mercury.on('event:mercury.buffer_state', spy);
 
-      return webex.internal.mercury.connect()
-        .then(() => {
-          assert.calledOnce(spy);
-        });
+      return webex.internal.mercury.connect().then(() => {
+        assert.calledOnce(spy);
+      });
     });
   });
 });

package/test/integration/spec/sharable-mercury.js

@@ -14,17 +14,19 @@ describe('plugin-mercury', function () {
   describe('Sharable Mercury', () => {
     let webex;
 
-    beforeEach(() => testUsers.create({count: 1})
-      .then((users) => {
+    beforeEach(() =>
+      testUsers.create({count: 1}).then((users) => {
         webex = new WebexCore({
           credentials: {
-            supertoken: users[0].token
-          }
+            supertoken: users[0].token,
+          },
         });
 
-        return webex.internal.device.register()
+        return webex.internal.device
+          .register()
           .then(() => webex.internal.feature.setFeature('developer', 'web-shared-mercury', true));
-      }));
+      })
+    );
 
     afterEach(() => webex && webex.internal.mercury.disconnect());
 
@@ -39,17 +41,19 @@ describe('plugin-mercury', function () {
       webex.internal.mercury.on('event:mercury.buffer_state', spy1);
       webex.internal.mercury.on('event:mercury.registration_status', spy2);
 
-      return webex.internal.mercury.connect()
-        .then(() => {
-          assert.notCalled(spy1);
-          assert.calledOnce(spy2);
-          const {data} = spy2.args[0][0];
+      return webex.internal.mercury.connect().then(() => {
+        assert.notCalled(spy1);
+        assert.calledOnce(spy2);
+        const {data} = spy2.args[0][0];
 
-          assert.property(data, 'bufferState');
-          assert.property(data, 'localClusterServiceUrls');
+        assert.property(data, 'bufferState');
+        assert.property(data, 'localClusterServiceUrls');
 
-          assert.deepEqual(webex.internal.mercury.localClusterServiceUrls, data.localClusterServiceUrls);
-        });
+        assert.deepEqual(
+          webex.internal.mercury.localClusterServiceUrls,
+          data.localClusterServiceUrls
+        );
+      });
     });
   });
 });

package/test/integration/spec/webex.js

@@ -14,25 +14,26 @@ describe('plugin-mercury', function () {
 
   let webex;
 
-  beforeEach('create users', () => testUsers.create({count: 1})
-    .then(async (users) => {
+  beforeEach('create users', () =>
+    testUsers.create({count: 1}).then(async (users) => {
       // Pause for 5 seconds for CI
       await new Promise((done) => setTimeout(done, 5000));
 
       webex = new WebexCore({
         credentials: {
-          supertoken: users[0].token
-        }
+          supertoken: users[0].token,
+        },
       });
       sinon.spy(webex.internal.mercury, 'disconnect');
       sinon.spy(webex.internal.device, 'unregister');
 
       return webex.internal.mercury.connect();
-    }));
+    })
+  );
 
   describe('onBeforeLogout()', () => {
-    it('disconnects the web socket', () => webex.logout({noRedirect: true})
-      .then(() => {
+    it('disconnects the web socket', () =>
+      webex.logout({noRedirect: true}).then(() => {
         assert.called(webex.internal.mercury.disconnect);
         assert.isFalse(webex.internal.mercury.connected);
         assert.called(webex.internal.device.unregister);