@webex/webex-core 3.0.0-beta.2 → 3.0.0-beta.21

package/test/unit/spec/interceptors/webex-user-agent.js

@@ -12,99 +12,145 @@ describe('webex-core', () => {
     describe('WebexUserAgentInterceptor', () => {
       describe('#onRequest', () => {
         it('adds a basic spark-user-agent header', () => {
-          const interceptor = Reflect.apply(WebexUserAgentInterceptor.create, {
-            version: pkg.version
-          }, []);
+          const interceptor = Reflect.apply(
+            WebexUserAgentInterceptor.create,
+            {
+              version: pkg.version,
+            },
+            []
+          );
           const options = {headers: {}};
 
           interceptor.onRequest(options);
 
           assert.property(options, 'headers');
           assert.property(options.headers, 'spark-user-agent');
-          assert.equal(options.headers['spark-user-agent'], `webex-js-sdk/${pkg.version} (${typeof window === 'undefined' ? 'node' : 'web'})`);
+          assert.equal(
+            options.headers['spark-user-agent'],
+            `webex-js-sdk/${pkg.version} (${typeof window === 'undefined' ? 'node' : 'web'})`
+          );
         });
 
         it('adds a complex spark-user-agent header', () => {
-          const interceptor = Reflect.apply(WebexUserAgentInterceptor.create, {
-            version: pkg.version
-          }, []);
+          const interceptor = Reflect.apply(
+            WebexUserAgentInterceptor.create,
+            {
+              version: pkg.version,
+            },
+            []
+          );
           const options = {headers: {}};
 
           interceptor.onRequest(options);
 
           assert.property(options, 'headers');
           assert.property(options.headers, 'spark-user-agent');
-          assert.equal(options.headers['spark-user-agent'], `webex-js-sdk/${pkg.version} (${typeof window === 'undefined' ? 'node' : 'web'})`);
+          assert.equal(
+            options.headers['spark-user-agent'],
+            `webex-js-sdk/${pkg.version} (${typeof window === 'undefined' ? 'node' : 'web'})`
+          );
         });
 
         it('adds a spark-user-agent header using appName/appVersion from config', () => {
-          const interceptor = Reflect.apply(WebexUserAgentInterceptor.create, {
-            version: pkg.version,
-            config: {
-              appName: 'sample',
-              appVersion: '1.0.0'
-            }
-          }, []);
+          const interceptor = Reflect.apply(
+            WebexUserAgentInterceptor.create,
+            {
+              version: pkg.version,
+              config: {
+                appName: 'sample',
+                appVersion: '1.0.0',
+              },
+            },
+            []
+          );
 
           const options = {
-            headers: {}
+            headers: {},
           };
 
           interceptor.onRequest(options);
 
           assert.property(options, 'headers');
           assert.property(options.headers, 'spark-user-agent');
-          assert.equal(options.headers['spark-user-agent'], `webex-js-sdk/${pkg.version} (${typeof window === 'undefined' ? 'node' : 'web'}) sample/1.0.0`);
+          assert.equal(
+            options.headers['spark-user-agent'],
+            `webex-js-sdk/${pkg.version} (${
+              typeof window === 'undefined' ? 'node' : 'web'
+            }) sample/1.0.0`
+          );
         });
 
         it('adds a spark-user-agent header using appName/appVersion + appPlatform from config', () => {
-          const interceptor = Reflect.apply(WebexUserAgentInterceptor.create, {
-            version: pkg.version,
-            config: {
-              appName: 'sample',
-              appVersion: '1.0.0',
-              appPlatform: 'custom-label/1.0.0'
-            }
-          }, []);
+          const interceptor = Reflect.apply(
+            WebexUserAgentInterceptor.create,
+            {
+              version: pkg.version,
+              config: {
+                appName: 'sample',
+                appVersion: '1.0.0',
+                appPlatform: 'custom-label/1.0.0',
+              },
+            },
+            []
+          );
 
           const options = {
-            headers: {}
+            headers: {},
           };
 
           interceptor.onRequest(options);
 
           assert.property(options, 'headers');
           assert.property(options.headers, 'spark-user-agent');
-          assert.equal(options.headers['spark-user-agent'], `webex-js-sdk/${pkg.version} (${typeof window === 'undefined' ? 'node' : 'web'}) sample/1.0.0 custom-label/1.0.0`);
+          assert.equal(
+            options.headers['spark-user-agent'],
+            `webex-js-sdk/${pkg.version} (${
+              typeof window === 'undefined' ? 'node' : 'web'
+            }) sample/1.0.0 custom-label/1.0.0`
+          );
         });
 
         describe('when consumed by the webex package', () => {
           it('adds a basic spark-user-agent header using "webex" instead of "webex-js-sdk"', () => {
-            const interceptor = Reflect.apply(WebexUserAgentInterceptor.create, {
-              webex: true,
-              version: pkg.version
-            }, []);
+            const interceptor = Reflect.apply(
+              WebexUserAgentInterceptor.create,
+              {
+                webex: true,
+                version: pkg.version,
+              },
+              []
+            );
             const options = {headers: {}};
 
             interceptor.onRequest(options);
 
             assert.property(options, 'headers');
             assert.property(options.headers, 'spark-user-agent');
-            assert.equal(options.headers['spark-user-agent'], `webex/${pkg.version} (${typeof window === 'undefined' ? 'node' : 'web'})`);
+            assert.equal(
+              options.headers['spark-user-agent'],
+              `webex/${pkg.version} (${typeof window === 'undefined' ? 'node' : 'web'})`
+            );
           });
 
           it('adds a complex spark-user-agent header using "webex" instead of "webex-js-sdk"', () => {
-            const interceptor = Reflect.apply(WebexUserAgentInterceptor.create, {
-              webex: true,
-              version: pkg.version
-            }, []);
+            const interceptor = Reflect.apply(
+              WebexUserAgentInterceptor.create,
+              {
+                webex: true,
+                version: pkg.version,
+              },
+              []
+            );
             const options = {headers: {}};
 
             interceptor.onRequest(options);
 
             assert.property(options, 'headers');
             assert.property(options.headers, 'spark-user-agent');
-            assert.equal(options.headers['spark-user-agent'], `webex/${pkg.version} (${typeof window === 'undefined' ? 'node' : 'web'})`);
+            assert.equal(
+              options.headers['spark-user-agent'],
+              `webex/${pkg.version} (${typeof window === 'undefined' ? 'node' : 'web'})`
+            );
           });
         });
       });

package/test/unit/spec/lib/batcher.js

@@ -28,7 +28,7 @@ describe('webex-core', () => {
         return webex.request({
           service: 'mock',
           resource: '/batch',
-          body: payload
+          body: payload,
         });
       },
       fingerprintRequest(req) {
@@ -36,7 +36,7 @@ describe('webex-core', () => {
       },
       fingerprintResponse(res) {
         return Promise.resolve(res);
-      }
+      },
     });
 
     const OutOfBandBatcher = MockBatcher.extend({
@@ -49,7 +49,7 @@ describe('webex-core', () => {
       },
       fingerprintResponse(res) {
         return Promise.resolve(res.id);
-      }
+      },
     });
 
     const BATCHER_MAX_CALLS = 10;
@@ -60,14 +60,14 @@ describe('webex-core', () => {
       webex = new MockWebex({
         children: {
           batcher: MockBatcher,
-          outOfBandBatcher: OutOfBandBatcher
-        }
+          outOfBandBatcher: OutOfBandBatcher,
+        },
       });
 
       webex.config.mock = {
         batcherMaxCalls: BATCHER_MAX_CALLS,
         batcherWait: BATCHER_WAIT,
-        batcherMaxWait: BATCHER_MAX_WAIT
+        batcherMaxWait: BATCHER_MAX_WAIT,
       };
     });
 
@@ -96,7 +96,6 @@ describe('webex-core', () => {
         promises.push(webex.internal.batcher.request(2));
         assert.notCalled(webex.request);
 
-
         return promiseTick(50)
           .then(() => {
             clock.tick(1);
@@ -130,7 +129,9 @@ describe('webex-core', () => {
       it('propagates error from inside the call chain', () => {
         // This is way easier to prove if we don't need to control the clock
         clock.uninstall();
-        sinon.stub(webex.internal.batcher, 'fingerprintResponse').throws(new Error('simulated failure'));
+        sinon
+          .stub(webex.internal.batcher, 'fingerprintResponse')
+          .throws(new Error('simulated failure'));
         webex.request.returns(Promise.resolve({body: [{id: 1}]}));
 
         return assert.isRejected(webex.internal.batcher.request({id: 1}), /simulated failure/);
@@ -150,10 +151,7 @@ describe('webex-core', () => {
             .then(() => {
               assert.calledOnce(webex.request);
 
-              return Promise.all([
-                assert.isRejected(p1),
-                assert.isRejected(p2)
-              ]);
+              return Promise.all([assert.isRejected(p1), assert.isRejected(p2)]);
             });
         });
       });
@@ -208,12 +206,17 @@ describe('webex-core', () => {
             webex.request.returns(Promise.resolve({body: result}));
             webex.request.onCall(1).returns(Promise.resolve({body: []}));
 
-            return result.reduce((promise, i) => promise.then(() => {
-              promises.push(webex.internal.batcher.request(i));
-              clock.tick(1);
-
-              return promiseTick(50);
-            }), Promise.resolve())
+            return result
+              .reduce(
+                (promise, i) =>
+                  promise.then(() => {
+                    promises.push(webex.internal.batcher.request(i));
+                    clock.tick(1);
+
+                    return promiseTick(50);
+                  }),
+                Promise.resolve()
+              )
               .then(() => {
                 assert.calledOnce(webex.request);
 
@@ -253,13 +256,12 @@ describe('webex-core', () => {
               promises.push(webex.internal.batcher.request(i));
             }
 
-            return Promise.all(promises)
-              .then((results) => {
-                assert.calledTwice(webex.request);
-                assert.lengthOf(webex.request.args[0][0].body, BATCHER_MAX_CALLS);
-                assert.lengthOf(webex.request.args[1][0].body, BATCHER_MAX_CALLS);
-                assert.lengthOf(results, BATCHER_MAX_CALLS * 2);
-              });
+            return Promise.all(promises).then((results) => {
+              assert.calledTwice(webex.request);
+              assert.lengthOf(webex.request.args[0][0].body, BATCHER_MAX_CALLS);
+              assert.lengthOf(webex.request.args[1][0].body, BATCHER_MAX_CALLS);
+              assert.lengthOf(results, BATCHER_MAX_CALLS * 2);
+            });
           });
         });
       });
@@ -268,7 +270,6 @@ describe('webex-core', () => {
         it('returns the same result', () => {
           webex.request.returns(Promise.resolve({body: [1]}));
 
-
           const p1 = webex.internal.batcher.request(1);
           const p2 = webex.internal.batcher.request(1);
 
@@ -285,16 +286,15 @@ describe('webex-core', () => {
               assert.deepEqual(webex.request.args[0][0], {
                 service: 'mock',
                 resource: '/batch',
-                body: [1]
+                body: [1],
               });
 
-              return promise
-                .then((result) => assert.deepEqual(result, [1, 1]));
+              return promise.then((result) => assert.deepEqual(result, [1, 1]));
             });
         });
       });
 
-      describe('when it\'s overridden to handle out-of-band responses', () => {
+      describe("when it's overridden to handle out-of-band responses", () => {
         it('resolves as expected', () => {
           sinon.spy(webex.internal.outOfBandBatcher, 'fingerprintResponse');
           const promise = webex.internal.outOfBandBatcher.request({id: 1});
@@ -312,7 +312,7 @@ describe('webex-core', () => {
             .then((res) => {
               assert.deepEqual(res, {
                 id: 1,
-                data: 2
+                data: 2,
               });
             });
         });
@@ -320,7 +320,11 @@ describe('webex-core', () => {
     });
 
     describe('#handleHttpError()', () => {
-      it('handles a non WebexHttpError object passed', () => assert.isRejected(webex.internal.batcher.handleHttpError('simulated failure'), /simulated failure/));
+      it('handles a non WebexHttpError object passed', () =>
+        assert.isRejected(
+          webex.internal.batcher.handleHttpError('simulated failure'),
+          /simulated failure/
+        ));
     });
   });
 });

package/test/unit/spec/lib/page.js

@@ -15,11 +15,11 @@ describe('webex-core', () => {
         sinon.stub(Page, 'parseLinkHeaders');
         const response = {
           body: {
-            items: [1, 2, 3]
+            items: [1, 2, 3],
           },
           headers: {
-            link: 'FakeHeaderLinks'
-          }
+            link: 'FakeHeaderLinks',
+          },
         };
 
         page = new Page(response, 'FakeWebex');
@@ -47,22 +47,24 @@ describe('webex-core', () => {
 
       before(() => {
         webex = {
-          request: sinon.stub().returns(Promise.resolve({
-            body: {
-              items: [4, 5, 6]
-            },
-            headers: {
-              link: '<https://www.cisco.com>; rel=previous'
-            }
-          }))
+          request: sinon.stub().returns(
+            Promise.resolve({
+              body: {
+                items: [4, 5, 6],
+              },
+              headers: {
+                link: '<https://www.cisco.com>; rel=previous',
+              },
+            })
+          ),
         };
         const response = {
           body: {
-            items: [1, 2, 3]
+            items: [1, 2, 3],
           },
           headers: {
-            link: '<https://www.cisco.com>; rel=next'
-          }
+            link: '<https://www.cisco.com>; rel=next',
+          },
         };
 
         page = new Page(response, webex);
@@ -72,8 +74,8 @@ describe('webex-core', () => {
         assert(page.hasNext());
       });
 
-      it('retrieves previous link', () => page.next()
-        .then((nextPage) => {
+      it('retrieves previous link', () =>
+        page.next().then((nextPage) => {
           assert.deepEqual(nextPage.items, [4, 5, 6]);
           assert.calledWith(webex.request, {uri: 'https://www.cisco.com'});
         }));
@@ -84,22 +86,24 @@ describe('webex-core', () => {
 
       before(() => {
         webex = {
-          request: sinon.stub().returns(Promise.resolve({
-            body: {
-              items: [4, 5, 6]
-            },
-            headers: {
-              link: '<https://www.cisco.com>; rel=previous'
-            }
-          }))
+          request: sinon.stub().returns(
+            Promise.resolve({
+              body: {
+                items: [4, 5, 6],
+              },
+              headers: {
+                link: '<https://www.cisco.com>; rel=previous',
+              },
+            })
+          ),
         };
         const response = {
           body: {
-            items: [1, 2, 3]
+            items: [1, 2, 3],
           },
           headers: {
-            link: '<https://www.cisco.com>; rel=previous'
-          }
+            link: '<https://www.cisco.com>; rel=previous',
+          },
         };
 
         page = new Page(response, webex);
@@ -109,8 +113,8 @@ describe('webex-core', () => {
         assert(page.hasPrevious());
       });
 
-      it('retrieves previous link', () => page.previous()
-        .then((prevPage) => {
+      it('retrieves previous link', () =>
+        page.previous().then((prevPage) => {
           assert.deepEqual(prevPage.items, [4, 5, 6]);
           assert.calledWith(webex.request, {uri: 'https://www.cisco.com'});
         }));
@@ -120,7 +124,7 @@ describe('webex-core', () => {
       const singleLinkHeader = '<https://www.cisco.com>; rel=cisco';
       const multipleLinkHeader = [
         '<https://www.ciscospark.com>; rel=webex',
-        '<https://www.cisco.com>; rel=cisco'
+        '<https://www.cisco.com>; rel=cisco',
       ];
 
       it('returns empty object if there are not any link headers', () => {
@@ -129,14 +133,14 @@ describe('webex-core', () => {
 
       it('returns object containing one link if only one link header passed as a string', () => {
         assert.deepEqual(Page.parseLinkHeaders(singleLinkHeader), {
-          cisco: 'https://www.cisco.com'
+          cisco: 'https://www.cisco.com',
         });
       });
 
       it('returns object containing multiple links when multiple headers passed as an array', () => {
         assert.deepEqual(Page.parseLinkHeaders(multipleLinkHeader), {
           webex: 'https://www.ciscospark.com',
-          cisco: 'https://www.cisco.com'
+          cisco: 'https://www.cisco.com',
         });
       });
     });

package/test/unit/spec/lib/webex-plugin.js

@@ -16,7 +16,7 @@ describe('webex-core', () => {
 
     /* eslint require-jsdoc: [0] */
     const MockPlugin = WebexPlugin.extend({
-      namespace: 'mock'
+      namespace: 'mock',
     });
 
     describe('#config', () => {