@webex/webex-core 2.60.1-next.9 → 2.60.2

package/test/unit/spec/credentials/credentials.js

@@ -11,7 +11,6 @@ import {inBrowser} from '@webex/common';
 import FakeTimers from '@sinonjs/fake-timers';
 import {skipInBrowser} from '@webex/test-helper-mocha';
 import Logger from '@webex/plugin-logger';
-import Metrics, {config} from '@webex/internal-plugin-metrics';
 
 /* eslint camelcase: [0] */
 
@@ -60,35 +59,6 @@ describe('webex-core', () => {
       });
     });
 
-    describe('#isUnverifiedGuest', () => {
-      let credentials;
-      let webex;
-      beforeEach(() => {
-        //generate the webex instance
-        webex = new MockWebex();
-        credentials = new Credentials(undefined, {parent: webex});
-      });
-
-      it('should have #isUnverifiedGuest', () => {
-        assert.exists(credentials.isUnverifiedGuest);
-      });
-
-      it('should get the user status and return as a boolean', () => {
-        credentials.set('supertoken', 'AT');
-        assert.isFalse(credentials.isUnverifiedGuest);
-      });
-
-      it('should get guest user ', () => {
-        credentials.set('supertoken', 'eyJhbGciOiJSUzI1NiJ9.eyJ1c2VyX3R5cGUiOiJndWVzdCJ9');
-        assert.isTrue(credentials.isUnverifiedGuest);
-      });
-
-      it('should get login user ', () => {
-        credentials.set('supertoken', 'dGhpc2lzbm90YXJlYWx1c2VydG9rZW4=');
-        assert.isFalse(credentials.isUnverifiedGuest);
-      });
-    });
-
     describe('#canAuthorize', () => {
       it('indicates if the current state has enough information to populate an auth header, even if a token refresh or token downscope is required', () => {
         const webex = new MockWebex();
@@ -447,11 +417,7 @@ describe('webex-core', () => {
         });
 
       it('schedules a refreshTimer', () => {
-        const webex = new MockWebex({
-          children: {
-            metrics: Metrics,
-          },
-        });
+        const webex = new MockWebex();
         const supertoken = makeToken(webex, {
           access_token: 'ST',
           refresh_token: 'RT',
@@ -464,7 +430,6 @@ describe('webex-core', () => {
         });
 
         sinon.stub(supertoken, 'refresh').returns(Promise.resolve(supertoken2));
-        sinon.stub(webex.internal.metrics, 'submitClientMetrics').callsFake(() => {});
         const credentials = new Credentials(supertoken, {parent: webex});
 
         webex.trigger('change:config');
@@ -499,19 +464,7 @@ describe('webex-core', () => {
     });
 
     describe('#getUserToken()', () => {
-      it('resolves with the supertoken if the supertoken matches the requested scopes', () => {
-        const webex = new MockWebex();
-        const credentials = new Credentials(undefined, {parent: webex});
-
-        webex.trigger('change:config');
-        const st = makeToken(webex, {access_token: 'ST', scope: 'scope1'});
-
-        credentials.set({
-          supertoken: st,
-        });
-
-        return credentials.getUserToken('scope1').then((result) => assert.deepEqual(result, st));
-      });
+      // it('resolves with the supertoken if the supertoken matches the requested scopes');
 
       it('resolves with the token identified by the specified scopes', () => {
         const webex = new MockWebex();
@@ -539,25 +492,6 @@ describe('webex-core', () => {
         ]);
       });
 
-      it('uses the supertoken.scope instead of the config.scope for downscope', () => {
-        const webex = new MockWebex();
-        const credentials = new Credentials(undefined, {parent: webex});
-
-        webex.trigger('change:config');
-        const st = makeToken(webex, {access_token: 'ST', scope: 'scope1 spark:kms'});
-
-        credentials.set({
-          supertoken: st,
-          scope: 'invalidScope scope1',
-        });
-
-        sinon.stub(credentials, 'downscope').returns(Promise.resolve());
-
-        return credentials.getUserToken().then(() => {
-          assert.calledWith(credentials.downscope, 'scope1');
-        });
-      });
-
       describe('when no matching token is found', () => {
         it('downscopes the supertoken', () => {
           const webex = new MockWebex();
@@ -595,13 +529,13 @@ describe('webex-core', () => {
         it('resolves with a token containing all but the kms scopes', () => {
           const webex = new MockWebex();
 
+          webex.config.credentials.scope = 'scope1 spark:kms';
           const credentials = new Credentials(undefined, {parent: webex});
 
           webex.trigger('change:config');
 
           credentials.supertoken = makeToken(webex, {
             access_token: 'ST',
-            scope: 'scope1 spark:kms',
           });
 
           // const t2 = makeToken(webex, {
@@ -628,11 +562,9 @@ describe('webex-core', () => {
           const webex = new MockWebex({
             children: {
               logger: Logger,
-              metrics: Metrics,
             },
           });
 
-          webex.config.metrics = config.metrics;
           webex.config.credentials.scope = 'scope1 spark:kms';
           const credentials = new Credentials(undefined, {parent: webex});
 
@@ -642,11 +574,9 @@ describe('webex-core', () => {
             access_token: 'ST',
           });
 
-          const failReason = 'downscope failed';
-          sinon.stub(credentials.supertoken, 'downscope').returns(Promise.reject(failReason));
-
-          sinon.stub(credentials.logger, 'warn').callsFake(() => {});
-          sinon.stub(webex.internal.metrics, 'submitClientMetrics').callsFake(() => {});
+          sinon
+            .stub(credentials.supertoken, 'downscope')
+            .returns(Promise.reject(new Error('downscope failed')));
 
           const t1 = makeToken(webex, {
             access_token: 'AT1',
@@ -657,27 +587,14 @@ describe('webex-core', () => {
             userTokens: [t1],
           });
 
-          return credentials.getUserToken('scope2').then((t) => {
-            assert.equal(t.access_token, credentials.supertoken.access_token);
-            assert.calledWith(
-              credentials.logger.warn,
-              'credentials: failed to downscope supertoken to "scope2"'
-            );
-            assert.calledWith(
-              webex.internal.metrics.submitClientMetrics,
-              'JS_SDK_CREDENTIALS_DOWNSCOPE_FAILED',
-              {fields: {failReason, requestedScope: 'scope2'}}
-            );
-          });
+          return credentials
+            .getUserToken('scope2')
+            .then((t) => assert.equal(t.access_token, credentials.supertoken.access_token));
         });
       });
 
       it('is blocked while a token refresh is inflight', () => {
-        const webex = new MockWebex({
-          children: {
-            metrics: Metrics,
-          },
-        });
+        const webex = new MockWebex();
 
         webex.config.credentials.scope = 'scope1 spark:kms';
         const credentials = new Credentials(undefined, {parent: webex});
@@ -703,7 +620,6 @@ describe('webex-core', () => {
         const at2 = makeToken(webex, {access_token: 'ST2ATD'});
 
         sinon.stub(supertoken2, 'downscope').returns(Promise.resolve(at2));
-        sinon.stub(webex.internal.metrics, 'submitClientMetrics').callsFake(() => {});
 
         return Promise.all([
           credentials.refresh(),
@@ -835,24 +751,18 @@ describe('webex-core', () => {
 
     describe('#refresh()', () => {
       it('refreshes and downscopes the supertoken, and revokes previous tokens', () => {
-        const webex = new MockWebex({
-          children: {
-            metrics: Metrics,
-          },
-        });
+        const webex = new MockWebex();
         const credentials = new Credentials(undefined, {parent: webex});
 
         webex.trigger('change:config');
         const st = makeToken(webex, {
           access_token: 'ST',
           refresh_token: 'RT',
-          scope: 'scope1 scope2',
         });
 
         const st2 = makeToken(webex, {
           access_token: 'ST2',
           refresh_token: 'RT2',
-          scope: 'scope1 scope2',
         });
 
         const t1 = makeToken(webex, {
@@ -869,7 +779,6 @@ describe('webex-core', () => {
         sinon.stub(st, 'refresh').returns(Promise.resolve(st2));
         sinon.stub(t1, 'revoke').returns(Promise.resolve());
         sinon.spy(credentials, 'scheduleRefresh');
-        sinon.stub(webex.internal.metrics, 'submitClientMetrics').callsFake(() => {});
 
         credentials.set({
           supertoken: st,
@@ -889,11 +798,7 @@ describe('webex-core', () => {
       });
 
       it('refreshes and downscopes the supertoken even if revocation of previous token fails', () => {
-        const webex = new MockWebex({
-          children: {
-            metrics: Metrics,
-          },
-        });
+        const webex = new MockWebex();
         const credentials = new Credentials(undefined, {parent: webex});
 
         webex.trigger('change:config');
@@ -905,7 +810,6 @@ describe('webex-core', () => {
         const st2 = makeToken(webex, {
           access_token: 'ST2',
           refresh_token: 'RT2',
-          scope: 'scope1 scope2',
         });
 
         const t1 = makeToken(webex, {
@@ -922,7 +826,6 @@ describe('webex-core', () => {
         sinon.stub(st, 'refresh').returns(Promise.resolve(st2));
         sinon.stub(t1, 'revoke').returns(Promise.reject());
         sinon.spy(credentials, 'scheduleRefresh');
-        sinon.stub(webex.internal.metrics, 'submitClientMetrics').callsFake(() => {});
 
         credentials.set({
           supertoken: st,
@@ -945,7 +848,6 @@ describe('webex-core', () => {
         const webex = new MockWebex({
           children: {
             logger: Logger,
-            metrics: Metrics,
           },
         });
         const credentials = new Credentials(undefined, {parent: webex});
@@ -954,11 +856,9 @@ describe('webex-core', () => {
         const st = makeToken(webex, {
           access_token: 'ST',
           refresh_token: 'RT',
-          scope: '',
         });
 
         sinon.stub(st, 'refresh').returns(Promise.resolve(makeToken(webex, {access_token: 'ST2'})));
-        sinon.stub(webex.internal.metrics, 'submitClientMetrics').callsFake(() => {});
 
         const t1 = makeToken(webex, {
           access_token: 'AT1',
@@ -974,7 +874,7 @@ describe('webex-core', () => {
       });
 
       it('allows #getUserToken() to be revoked, but #getUserToken() promises will not resolve until the suport token has been refreshed', () => {
-        const webex = new MockWebex({children: {metrics: Metrics}});
+        const webex = new MockWebex();
         const credentials = new Credentials(undefined, {parent: webex});
 
         webex.trigger('change:config');
@@ -1000,7 +900,6 @@ describe('webex-core', () => {
 
         sinon.stub(st1, 'refresh').returns(Promise.resolve(st2));
         sinon.stub(st2, 'downscope').returns(Promise.resolve(t2));
-        sinon.stub(webex.internal.metrics, 'submitClientMetrics').callsFake(() => {});
 
         credentials.set({
           supertoken: st1,
@@ -1057,61 +956,6 @@ describe('webex-core', () => {
             assert.calledWith(triggerSpy, sinon.match('client:InvalidRequestError'));
           });
       });
-
-      it('exclude invalid scopes from user token, log and call metrics when fetched supertoken scope mismatch with the configured scope', () => {
-        const webex = new MockWebex({
-          children: {
-            logger: Logger,
-            metrics: Metrics,
-          },
-        });
-        const credentials = new Credentials(undefined, {parent: webex});
-
-        webex.trigger('change:config');
-        const st = makeToken(webex, {
-          access_token: 'ST',
-          refresh_token: 'RT',
-        });
-
-        const st2 = makeToken(webex, {
-          access_token: 'ST2',
-          refresh_token: 'RT2',
-          scope: 'scope1',
-        });
-
-        const userToken = makeToken(webex, {
-          access_token: 'AT1',
-          scope: 'scope1 invalidScope1',
-        });
-
-        credentials.set({
-          supertoken: st,
-          userTokens: [userToken],
-        });
-        const invalidScopes = 'invalidScope1 invalidScope2';
-        credentials.config.scope = `scope1 ${invalidScopes}`;
-
-        sinon.stub(st2, 'downscope').returns(Promise.resolve());
-        sinon.stub(st, 'refresh').returns(Promise.resolve(st2));
-        sinon.spy(credentials, 'downscope');
-        sinon.spy(credentials, 'scheduleRefresh');
-
-        sinon.stub(credentials.logger, 'warn').callsFake(() => {});
-        sinon.stub(webex.internal.metrics, 'submitClientMetrics').callsFake(() => {});
-
-        return credentials.refresh().then(() => {
-          assert.calledWith(
-            credentials.logger.warn,
-            `credentials: "${invalidScopes}" scope(s) are invalid because not listed in the supertoken, they will be excluded from user token requests.`
-          );
-          assert.calledWith(
-            webex.internal.metrics.submitClientMetrics,
-            'JS_SDK_CREDENTIALS_TOKEN_REFRESH_SCOPE_MISMATCH',
-            {fields: {invalidScopes}}
-          );
-          assert.calledWith(credentials.downscope, 'scope1');
-        });
-      });
     });
 
     describe('#scheduleRefresh()', () => {

package/test/unit/spec/credentials/token.js

@@ -166,21 +166,11 @@ describe('webex-core', () => {
           );
         });
 
-        it('rejects downscope when the new scope is not super set of the available scopes', () => {
-          const token = makeToken();
-          token.config.scope = 'scopeY scopeZ';
-
-          return assert.isRejected(
-            token.downscope('scopeX'),
-            /new scope \(scopeX\) is not subset of the available scopes \(scopeY scopeZ\)/
-          );
-        });
-
         it('alphabetizes the requested scope', () => {
           const token = makeToken();
 
           webex.request.returns(Promise.resolve({body: {access_token: 'AT2'}}));
-          token.config.scope = 'a b c';
+
           return token
             .downscope('b a')
             .then(() => assert.equal(webex.request.args[0][0].form.scope, 'a b'));

package/test/unit/spec/interceptors/auth.js

@@ -12,7 +12,6 @@ import Logger from '@webex/plugin-logger';
 import MockWebex from '@webex/test-helper-mock-webex';
 import {AuthInterceptor, config, Credentials, WebexHttpError, Token} from '@webex/webex-core';
 import {cloneDeep, merge} from 'lodash';
-import Metrics from '@webex/internal-plugin-metrics';
 
 const {assert} = chai;
 
@@ -29,7 +28,6 @@ describe('webex-core', () => {
           children: {
             credentials: Credentials,
             logger: Logger,
-            metrics: Metrics,
           },
           config: merge(cloneDeep(config), {credentials: {client_secret: 'fake'}}),
         });
@@ -43,7 +41,6 @@ describe('webex-core', () => {
         );
 
         interceptor = Reflect.apply(AuthInterceptor.create, webex, []);
-        sinon.stub(webex.internal.metrics, 'submitClientMetrics').callsFake(() => {});
       });
 
       describe('#onRequest()', () => {

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

@@ -27,7 +27,7 @@ describe('webex-core', () => {
           assert.property(options.headers, 'spark-user-agent');
           assert.equal(
             options.headers['spark-user-agent'],
-            `webex-js-sdk/development (${typeof window === 'undefined' ? 'node' : 'web'})`
+            `webex-js-sdk/${pkg.version} (${typeof window === 'undefined' ? 'node' : 'web'})`
           );
         });
 
@@ -47,7 +47,7 @@ describe('webex-core', () => {
           assert.property(options.headers, 'spark-user-agent');
           assert.equal(
             options.headers['spark-user-agent'],
-            `webex-js-sdk/development (${typeof window === 'undefined' ? 'node' : 'web'})`
+            `webex-js-sdk/${pkg.version} (${typeof window === 'undefined' ? 'node' : 'web'})`
           );
         });
 
@@ -74,7 +74,7 @@ describe('webex-core', () => {
           assert.property(options.headers, 'spark-user-agent');
           assert.equal(
             options.headers['spark-user-agent'],
-            `webex-js-sdk/development (${
+            `webex-js-sdk/${pkg.version} (${
               typeof window === 'undefined' ? 'node' : 'web'
             }) sample/1.0.0`
           );
@@ -104,7 +104,7 @@ describe('webex-core', () => {
           assert.property(options.headers, 'spark-user-agent');
           assert.equal(
             options.headers['spark-user-agent'],
-            `webex-js-sdk/development (${
+            `webex-js-sdk/${pkg.version} (${
               typeof window === 'undefined' ? 'node' : 'web'
             }) sample/1.0.0 custom-label/1.0.0`
           );
@@ -128,7 +128,7 @@ describe('webex-core', () => {
             assert.property(options.headers, 'spark-user-agent');
             assert.equal(
               options.headers['spark-user-agent'],
-              `webex/development (${typeof window === 'undefined' ? 'node' : 'web'})`
+              `webex/${pkg.version} (${typeof window === 'undefined' ? 'node' : 'web'})`
             );
           });
 
@@ -149,7 +149,7 @@ describe('webex-core', () => {
             assert.property(options.headers, 'spark-user-agent');
             assert.equal(
               options.headers['spark-user-agent'],
-              `webex/development (${typeof window === 'undefined' ? 'node' : 'web'})`
+              `webex/${pkg.version} (${typeof window === 'undefined' ? 'node' : 'web'})`
             );
           });
         });

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

@@ -5,7 +5,6 @@ import chai from 'chai';
 import chaiAsPromised from 'chai-as-promised';
 import sinon from 'sinon';
 import {ServiceInterceptor} from '@webex/webex-core';
-import CONFIG from '../../../../../src/config';
 
 const {assert} = chai;
 
@@ -27,7 +26,6 @@ describe('webex-core', () => {
         service: 'example',
         serviceUrl: 'https://www.example-service.com/',
         uri: 'https://www.example-uri.com/',
-        waitForServiceTimeout: 11,
       };
 
       options = {};
@@ -112,7 +110,6 @@ describe('webex-core', () => {
 
           options.service = fixture.service;
           options.resource = fixture.resource;
-          options.timeout = fixture.waitForServiceTimeout;
         });
 
         it('should normalize the options', () =>
@@ -122,12 +119,9 @@ describe('webex-core', () => {
           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,
-              timeout: options.waitForServiceTimeout,
-            })
-          ));
+          interceptor
+            .onRequest(options)
+            .then(() => assert.calledWith(waitForService, {name: options.service})));
 
         describe('when the service url was collected successfully', () => {
           it('should attempt to generate the full uri', () =>

package/test/unit/spec/webex-core.js

@@ -54,18 +54,6 @@ describe('Webex', () => {
     });
   });
 
-  describe('#request', () => {
-    it('exists', () => {
-      assert.property(webex, 'request');
-    });
-  });
-
-  describe('#prepareFetchOptions', () => {
-    it('exists', () => {
-      assert.property(webex, 'prepareFetchOptions');
-    });
-  });
-
   describe('#initialize()', () => {
     it('initializes without arguments', () => {
       let webex;

package/dist/lib/constants.js

@@ -1,13 +0,0 @@
-"use strict";
-
-var _Object$defineProperty = require("@babel/runtime-corejs2/core-js/object/define-property");
-_Object$defineProperty(exports, "__esModule", {
-  value: true
-});
-exports.METRICS = void 0;
-// Metric to do with WDM registration
-var METRICS = exports.METRICS = {
-  JS_SDK_CREDENTIALS_DOWNSCOPE_FAILED: 'JS_SDK_CREDENTIALS_DOWNSCOPE_FAILED',
-  JS_SDK_CREDENTIALS_TOKEN_REFRESH_SCOPE_MISMATCH: 'JS_SDK_CREDENTIALS_TOKEN_REFRESH_SCOPE_MISMATCH'
-};
-//# sourceMappingURL=constants.js.map

package/dist/lib/constants.js.map

@@ -1 +0,0 @@
-{"version":3,"names":["METRICS","exports","JS_SDK_CREDENTIALS_DOWNSCOPE_FAILED","JS_SDK_CREDENTIALS_TOKEN_REFRESH_SCOPE_MISMATCH"],"sources":["constants.js"],"sourcesContent":["// Metric to do with WDM registration\nexport const METRICS = {\n  JS_SDK_CREDENTIALS_DOWNSCOPE_FAILED: 'JS_SDK_CREDENTIALS_DOWNSCOPE_FAILED',\n  JS_SDK_CREDENTIALS_TOKEN_REFRESH_SCOPE_MISMATCH:\n    'JS_SDK_CREDENTIALS_TOKEN_REFRESH_SCOPE_MISMATCH',\n};\n"],"mappings":";;;;;;;AAAA;AACO,IAAMA,OAAO,GAAAC,OAAA,CAAAD,OAAA,GAAG;EACrBE,mCAAmC,EAAE,qCAAqC;EAC1EC,+CAA+C,EAC7C;AACJ,CAAC"}

package/src/lib/constants.js

@@ -1,6 +0,0 @@
-// Metric to do with WDM registration
-export const METRICS = {
-  JS_SDK_CREDENTIALS_DOWNSCOPE_FAILED: 'JS_SDK_CREDENTIALS_DOWNSCOPE_FAILED',
-  JS_SDK_CREDENTIALS_TOKEN_REFRESH_SCOPE_MISMATCH:
-    'JS_SDK_CREDENTIALS_TOKEN_REFRESH_SCOPE_MISMATCH',
-};

package/test/unit/spec/credentials/scope.js

@@ -1,80 +0,0 @@
-import {assert} from '@webex/test-helper-chai';
-import {
-  sortScope,
-  filterScope,
-  diffScopes,
-  isGuestScope,
-} from '@webex/webex-core/src/lib/credentials/scope';
-
-describe('webex-core', () => {
-  describe('scope utils', () => {
-    describe('sortScope', () => {
-      [
-        {scope: undefined, expected: ''},
-        {scope: '', expected: ''},
-        {scope: 'a', expected: 'a'},
-        {scope: 'b c a', expected: 'a b c'},
-      ].forEach(({scope, expected}) =>
-        it(`should sort "${scope}" alphabetically`, () => {
-          assert.equal(sortScope(scope), expected);
-        })
-      );
-    });
-
-    describe('filterScope', () => {
-      [
-        {toFilter: 'a', scope: undefined, expected: ''},
-        {toFilter: 'a', scope: '', expected: ''},
-        {toFilter: 'a', scope: 'a', expected: ''},
-        {toFilter: 'a', scope: 'a b c', expected: 'b c'},
-        {toFilter: 'c', scope: 'a c b', expected: 'a b'},
-      ].forEach(({toFilter, scope, expected}) =>
-        it(`should filter out ${toFilter} scope from ${scope} scope and sort the result`, () => {
-          assert.equal(filterScope(toFilter, scope), expected);
-        })
-      );
-
-      [
-        {toFilter: [], scope: 'a', expected: 'a'},
-        {toFilter: ['a', 'b'], scope: undefined, expected: ''},
-        {toFilter: ['a', 'b'], scope: '', expected: ''},
-        {toFilter: ['a', 'b'], scope: 'a', expected: ''},
-        {toFilter: ['a', 'b'], scope: 'a b c', expected: 'c'},
-        {toFilter: ['a', 'd'], scope: 'a c a b', expected: 'b c'},
-      ].forEach(({toFilter, scope, expected}) =>
-        it(`should filter out ${toFilter} from ${scope} and sort the result`, () => {
-          assert.equal(filterScope(toFilter, scope), expected);
-        })
-      );
-    });
-
-    describe('diffScopes', () => {
-      [
-        {scope1: undefined, scope2: undefined, expected: ''},
-        {scope1: undefined, scope2: '', expected: ''},
-        {scope1: '', scope2: undefined, expected: ''},
-        {scope1: '', scope2: '', expected: ''},
-        {scope1: 'a', scope2: 'a', expected: ''},
-        {scope1: 'a b c', scope2: 'a b c', expected: ''},
-        {scope1: undefined, scope2: 'a b c', expected: ''},
-        {scope1: 'a b c', scope2: 'a b c d', expected: ''},
-      ].forEach(({scope1, scope2, expected}) =>
-        it(`should return an empty string, when all items in ${scope1} scope are contained in the ${scope2} scope`, () => {
-          assert.deepEqual(diffScopes(scope1, scope2), expected);
-        })
-      );
-
-      [
-        {scope1: 'a', scope2: undefined, expected: 'a'},
-        {scope1: 'a', scope2: 'b', expected: 'a'},
-        {scope1: 'a b c', scope2: 'a b', expected: 'c'},
-        {scope1: 'a b c d', scope2: 'a b c', expected: 'd'},
-        {scope1: 'a b c', scope2: undefined, expected: 'a b c'},
-      ].forEach(({scope1, scope2, expected}) =>
-        it(`should return a string containing all items in the ${scope1} scope that are not in the ${scope2} scope`, () => {
-          assert.deepEqual(diffScopes(scope1, scope2), expected);
-        })
-      );
-    });
-  });
-});