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

package/test/unit/spec/services/interceptors/server-error.js

@@ -1,10 +1,7 @@
 import chai from 'chai';
 import chaiAsPromised from 'chai-as-promised';
 import sinon from 'sinon';
-import {
-  ServerErrorInterceptor,
-  WebexHttpError
-} from '@webex/webex-core';
+import {ServerErrorInterceptor, WebexHttpError} from '@webex/webex-core';
 
 const {assert} = chai;
 
@@ -40,9 +37,9 @@ describe('webex-core', () => {
             options: {
               url: 'http://not-a-url.com/',
               headers: {
-                trackingId: 'tid'
-              }
-            }
+                trackingId: 'tid',
+              },
+            },
           });
 
           interceptor.webex = {
@@ -50,41 +47,32 @@ describe('webex-core', () => {
               device: {
                 features: {
                   developer: {
-                    get: sinon.stub()
-                  }
-                }
+                    get: sinon.stub(),
+                  },
+                },
               },
               metrics: {
-                submitClientMetrics: sinon.spy()
+                submitClientMetrics: sinon.spy(),
               },
               services: {
-                markFailedUrl: sinon.stub()
-              }
-            }
+                markFailedUrl: sinon.stub(),
+              },
+            },
           };
 
           get = interceptor.webex.internal.device.features.developer.get;
-          markFailedUrl = interceptor
-            .webex
-            .internal
-            .services
-            .markFailedUrl;
-          submitClientMetrics = interceptor
-            .webex
-            .internal
-            .metrics
-            .submitClientMetrics;
+          markFailedUrl = interceptor.webex.internal.services.markFailedUrl;
+          submitClientMetrics = interceptor.webex.internal.metrics.submitClientMetrics;
 
           markFailedUrl.returns();
         });
 
-        it('should get the feature \'web-high-availability\'', (done) => {
-          interceptor.onResponseError(options, reason)
-            .catch(() => {
-              assert.calledWith(get, 'web-high-availability');
+        it("should get the feature 'web-high-availability'", (done) => {
+          interceptor.onResponseError(options, reason).catch(() => {
+            assert.calledWith(get, 'web-high-availability');
 
-              done();
-            });
+            done();
+          });
         });
 
         describe('when the web-ha feature is enabled', () => {
@@ -93,30 +81,26 @@ describe('webex-core', () => {
           });
 
           it('should submit appropriate client metrics', (done) => {
-            interceptor.onResponseError(options, reason)
-              .catch(() => {
-                assert.calledWith(submitClientMetrics,
-                  'web-ha',
-                  {
-                    fields: {success: false},
-                    tags: {
-                      action: 'failed',
-                      error: reason.message,
-                      url: options.uri
-                    }
-                  });
-
-                done();
+            interceptor.onResponseError(options, reason).catch(() => {
+              assert.calledWith(submitClientMetrics, 'web-ha', {
+                fields: {success: false},
+                tags: {
+                  action: 'failed',
+                  error: reason.message,
+                  url: options.uri,
+                },
               });
+
+              done();
+            });
           });
 
           it('should mark a url as failed', (done) => {
-            interceptor.onResponseError(options, reason)
-              .catch(() => {
-                assert.calledWith(markFailedUrl, options.uri);
+            interceptor.onResponseError(options, reason).catch(() => {
+              assert.calledWith(markFailedUrl, options.uri);
 
-                done();
-              });
+              done();
+            });
           });
 
           it('should mark a url as failed for a 503', (done) => {
@@ -126,26 +110,24 @@ describe('webex-core', () => {
               options: {
                 url: 'http://not-a-url.com/',
                 headers: {
-                  trackingId: 'tid'
-                }
-              }
+                  trackingId: 'tid',
+                },
+              },
             });
 
-            interceptor.onResponseError(options, reason)
-              .catch(() => {
-                assert.calledWith(markFailedUrl, options.uri);
+            interceptor.onResponseError(options, reason).catch(() => {
+              assert.calledWith(markFailedUrl, options.uri);
 
-                done();
-              });
+              done();
+            });
           });
 
           it('should return a rejected promise with a reason', (done) => {
-            interceptor.onResponseError(options, reason)
-              .catch((error) => {
-                assert.instanceOf(error, WebexHttpError.InternalServerError);
+            interceptor.onResponseError(options, reason).catch((error) => {
+              assert.instanceOf(error, WebexHttpError.InternalServerError);
 
-                done();
-              });
+              done();
+            });
           });
         });
 
@@ -155,30 +137,27 @@ describe('webex-core', () => {
           });
 
           it('should return a rejected promise with the reason', (done) => {
-            interceptor.onResponseError(options, reason)
-              .catch((error) => {
-                assert.instanceOf(error, WebexHttpError.InternalServerError);
+            interceptor.onResponseError(options, reason).catch((error) => {
+              assert.instanceOf(error, WebexHttpError.InternalServerError);
 
-                done();
-              });
+              done();
+            });
           });
 
           it('should not attempt to submit client metrics', (done) => {
-            interceptor.onResponseError(options, reason)
-              .catch(() => {
-                assert.notCalled(submitClientMetrics);
+            interceptor.onResponseError(options, reason).catch(() => {
+              assert.notCalled(submitClientMetrics);
 
-                done();
-              });
+              done();
+            });
           });
 
           it('should not attempt to mark a url as failed', (done) => {
-            interceptor.onResponseError(options, reason)
-              .catch(() => {
-                assert.notCalled(markFailedUrl);
+            interceptor.onResponseError(options, reason).catch(() => {
+              assert.notCalled(markFailedUrl);
 
-                done();
-              });
+              done();
+            });
           });
         });
       });
@@ -190,12 +169,11 @@ describe('webex-core', () => {
         });
 
         it('should return a rejected promise with the reason', (done) => {
-          interceptor.onResponseError(options, reason)
-            .catch((error) => {
-              assert.deepEqual(error, reason);
+          interceptor.onResponseError(options, reason).catch((error) => {
+            assert.deepEqual(error, reason);
 
-              done();
-            });
+            done();
+          });
         });
       });
 
@@ -207,19 +185,18 @@ describe('webex-core', () => {
             options: {
               url: 'http://not-a-url.com/',
               headers: {
-                trackingId: 'tid'
-              }
-            }
+                trackingId: 'tid',
+              },
+            },
           });
         });
 
         it('should return a rejected promise with the reason', (done) => {
-          interceptor.onResponseError(options, reason)
-            .catch((error) => {
-              assert.instanceOf(error, WebexHttpError.InternalServerError);
+          interceptor.onResponseError(options, reason).catch((error) => {
+            assert.instanceOf(error, WebexHttpError.InternalServerError);
 
-              done();
-            });
+            done();
+          });
         });
       });
     });

package/test/unit/spec/services/interceptors/service.js

@@ -25,7 +25,7 @@ describe('webex-core', () => {
         resource: '/example/resource/',
         service: 'example',
         serviceUrl: 'https://www.example-service.com/',
-        uri: 'https://www.example-uri.com/'
+        uri: 'https://www.example-uri.com/',
       };
 
       options = {};
@@ -35,13 +35,9 @@ describe('webex-core', () => {
       let uri;
 
       beforeEach('generate uri', () => {
-        uri = interceptor.generateUri(
-          fixture.serviceUrl,
-          fixture.resource
-        );
+        uri = interceptor.generateUri(fixture.serviceUrl, fixture.resource);
       });
-      it('should remove all trailing slashes', () =>
-        assert.equal(uri.split('//').length, 2));
+      it('should remove all trailing slashes', () => assert.equal(uri.split('//').length, 2));
 
       it('should combine the service url and the resource', () => {
         assert.isTrue(uri.includes(fixture.serviceUrl));
@@ -101,9 +97,9 @@ describe('webex-core', () => {
           interceptor.webex = {
             internal: {
               services: {
-                waitForService: sinon.stub()
-              }
-            }
+                waitForService: sinon.stub(),
+              },
+            },
           };
 
           waitForService = interceptor.webex.internal.services.waitForService;
@@ -114,32 +110,25 @@ describe('webex-core', () => {
         });
 
         it('should normalize the options', () =>
-          interceptor.onRequest(options)
-            .then(() => assert.called(interceptor.normalizeOptions)));
+          interceptor.onRequest(options).then(() => assert.called(interceptor.normalizeOptions)));
 
         it('should validate the options', () =>
-          interceptor.onRequest(options)
-            .then(() => assert.called(interceptor.validateOptions)));
+          interceptor.onRequest(options).then(() => assert.called(interceptor.validateOptions)));
 
         it('should attempt to collect the service url', () =>
-          interceptor.onRequest(options)
-            .then(() => assert.calledWith(
-              waitForService,
-              {name: options.service}
-            )));
+          interceptor
+            .onRequest(options)
+            .then(() => assert.calledWith(waitForService, {name: options.service})));
 
         describe('when the service url was collected successfully', () => {
-          beforeEach('generate additional mocks', () => {
-
-          });
+          beforeEach('generate additional mocks', () => {});
 
           it('should attempt to generate the full uri', () =>
-            interceptor.onRequest(options)
-              .then(() => assert.calledWith(
-                interceptor.generateUri,
-                fixture.serviceUrl,
-                fixture.resource
-              )));
+            interceptor
+              .onRequest(options)
+              .then(() =>
+                assert.calledWith(interceptor.generateUri, fixture.serviceUrl, fixture.resource)
+              ));
 
           it('should return a resolved promise', () => {
             const promise = interceptor.onRequest(options);

package/test/unit/spec/services/service-catalog.js

@@ -27,8 +27,13 @@ describe('webex-core', () => {
 
     describe('#serviceGroups', () => {
       it('has all the required keys', () => {
-        assert.hasAllKeys(catalog.serviceGroups,
-          ['discovery', 'override', 'preauth', 'signin', 'postauth']);
+        assert.hasAllKeys(catalog.serviceGroups, [
+          'discovery',
+          'override',
+          'preauth',
+          'signin',
+          'postauth',
+        ]);
       });
 
       it('contains values that are arrays', () => {
@@ -40,8 +45,13 @@ describe('webex-core', () => {
 
     describe('#status', () => {
       it('has all the required keys', () => {
-        assert.hasAllKeys(catalog.status,
-          ['discovery', 'override', 'preauth', 'postauth', 'signin']);
+        assert.hasAllKeys(catalog.status, [
+          'discovery',
+          'override',
+          'preauth',
+          'postauth',
+          'signin',
+        ]);
       });
 
       it('has valid key value types', () => {
@@ -91,11 +101,7 @@ describe('webex-core', () => {
       const domains = [];
 
       beforeEach('generate allowed domains', () => {
-        domains.push(
-          'example-a',
-          'example-b',
-          'example-c'
-        );
+        domains.push('example-a', 'example-b', 'example-c');
 
         catalog.setAllowedDomains(domains);
       });
@@ -105,9 +111,7 @@ describe('webex-core', () => {
       });
 
       it('finds an allowed domain that matches a specific url', () => {
-        const domain = catalog.findAllowedDomain(
-          'http://example-a.com/resource/id'
-        );
+        const domain = catalog.findAllowedDomain('http://example-a.com/resource/id');
 
         assert.include(domains, domain);
       });
@@ -117,11 +121,7 @@ describe('webex-core', () => {
       const domains = [];
 
       beforeEach('generate allowed domains', () => {
-        domains.push(
-          'example-a',
-          'example-b',
-          'example-c'
-        );
+        domains.push('example-a', 'example-b', 'example-c');
 
         catalog.setAllowedDomains(domains);
       });
@@ -160,11 +160,7 @@ describe('webex-core', () => {
       const domains = [];
 
       beforeEach('generate allowed domains', () => {
-        domains.push(
-          'example-a',
-          'example-b',
-          'example-c'
-        );
+        domains.push('example-a', 'example-b', 'example-c');
 
         catalog.setAllowedDomains(domains);
       });
@@ -174,11 +170,7 @@ describe('webex-core', () => {
       });
 
       it('sets the allowed domain entries to new values', () => {
-        const newValues = [
-          'example-d',
-          'example-e',
-          'example-f'
-        ];
+        const newValues = ['example-d', 'example-e', 'example-f'];
 
         catalog.setAllowedDomains(newValues);
 

package/test/unit/spec/services/service-host.js

@@ -15,7 +15,7 @@ describe('webex-core', () => {
         hostGroup: 'example-host-group.com',
         id: 'example-head:example-group:example-cluster:example-name',
         priority: 1,
-        uri: 'example-uri.com'
+        uri: 'example-uri.com',
       };
 
       defaultHostGroup = 'example-default.com';
@@ -132,7 +132,6 @@ describe('webex-core', () => {
         assert.isTrue(serviceHost.setStatus({failed: true}).failed);
       });
 
-
       it('should set the property failed to false', () => {
         assert.isFalse(serviceHost.setStatus({failed: false}).failed);
       });
@@ -141,33 +140,40 @@ describe('webex-core', () => {
         assert.isTrue(serviceHost.setStatus({replaced: true}).replaced);
       });
 
-
       it('should set the property replaced to false', () => {
         assert.isFalse(serviceHost.setStatus({replaced: false}).replaced);
       });
 
       it('should set the property replaced and failed to true', () => {
-        assert.isTrue(serviceHost.setStatus({
-          failed: true,
-          replaced: true
-        }).failed);
-
-        assert.isTrue(serviceHost.setStatus({
-          failed: true,
-          replaced: true
-        }).replaced);
+        assert.isTrue(
+          serviceHost.setStatus({
+            failed: true,
+            replaced: true,
+          }).failed
+        );
+
+        assert.isTrue(
+          serviceHost.setStatus({
+            failed: true,
+            replaced: true,
+          }).replaced
+        );
       });
 
       it('should set the property replaced and failed to false', () => {
-        assert.isFalse(serviceHost.setStatus({
-          failed: false,
-          replaced: false
-        }).failed);
-
-        assert.isFalse(serviceHost.setStatus({
-          failed: false,
-          replaced: false
-        }).replaced);
+        assert.isFalse(
+          serviceHost.setStatus({
+            failed: false,
+            replaced: false,
+          }).failed
+        );
+
+        assert.isFalse(
+          serviceHost.setStatus({
+            failed: false,
+            replaced: false,
+          }).replaced
+        );
       });
 
       describe('static methods', () => {
@@ -178,15 +184,12 @@ describe('webex-core', () => {
             polyFixture = {
               catalog: fixture.catalog,
               name: fixture.id.split(':')[3],
-              url: fixture.defaultUri
+              url: fixture.defaultUri,
             };
           });
 
           it('should generate a new ServiceHost', () => {
-            assert.instanceOf(
-              ServiceHost.polyGenerate(polyFixture),
-              ServiceHost
-            );
+            assert.instanceOf(ServiceHost.polyGenerate(polyFixture), ServiceHost);
           });
         });