@webex/webex-core 3.12.0-next.9 → 3.12.0-webex-services-ready.1

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

@@ -34,6 +34,53 @@ describe('webex-core', () => {
     });
 
     describe('#initialize', () => {
+      it('listens for "loaded" event instead of "ready" to avoid deadlock', () => {
+        services.listenToOnce = sinon.stub();
+        services.initialize();
+
+        // Second listenToOnce call should be for 'loaded' event
+        assert.equal(services.listenToOnce.getCall(1).args[1], 'loaded');
+      });
+
+      it('services.ready starts as false', () => {
+        assert.isFalse(services.ready);
+      });
+
+      it('sets ready to true and triggers services:initialized when initialization succeeds with credentials', async () => {
+        services.listenToOnce = sinon.stub();
+        services.initServiceCatalogs = sinon.stub().returns(Promise.resolve());
+        services.trigger = sinon.stub();
+        services.webex.credentials = {
+          supertoken: {
+            access_token: 'token',
+          },
+        };
+
+        services.initialize();
+
+        // call the onLoaded callback
+        services.listenToOnce.getCall(1).args[2]();
+        await waitForAsync();
+
+        assert.isTrue(services.ready);
+        sinon.assert.calledWith(services.trigger, 'services:initialized');
+      });
+
+      it('sets ready to true and triggers services:initialized when initialization succeeds without credentials', async () => {
+        services.listenToOnce = sinon.stub();
+        services.collectPreauthCatalog = sinon.stub().returns(Promise.resolve());
+        services.trigger = sinon.stub();
+
+        services.initialize();
+
+        // call the onLoaded callback
+        services.listenToOnce.getCall(1).args[2]();
+        await waitForAsync();
+
+        assert.isTrue(services.ready);
+        sinon.assert.calledWith(services.trigger, 'services:initialized');
+      });
+
       it('initFailed is false when initialization succeeds and credentials are available', async () => {
         services.listenToOnce = sinon.stub();
         services.initServiceCatalogs = sinon.stub().returns(Promise.resolve());
@@ -45,7 +92,7 @@ describe('webex-core', () => {
 
         services.initialize();
 
-        // call the onReady callback
+        // call the onLoaded callback
         services.listenToOnce.getCall(1).args[2]();
         await waitForAsync();
 
@@ -58,7 +105,7 @@ describe('webex-core', () => {
 
         services.initialize();
 
-        // call the onReady callback
+        // call the onLoaded callback
         services.listenToOnce.getCall(1).args[2]();
         await waitForAsync();
 
@@ -67,9 +114,9 @@ describe('webex-core', () => {
 
       it.each([
         {error: new Error('failed'), expectedMessage: 'failed'},
-        {error: undefined, expectedMessage: undefined}
+        {error: undefined, expectedMessage: undefined},
       ])(
-        'sets initFailed to true when collectPreauthCatalog errors',
+        'sets initFailed to true when collectPreauthCatalog errors but still sets ready to true',
         async ({error, expectedMessage}) => {
           services.collectPreauthCatalog = sinon.stub().callsFake(() => {
             return Promise.reject(error);
@@ -77,15 +124,18 @@ describe('webex-core', () => {
 
           services.listenToOnce = sinon.stub();
           services.logger.error = sinon.stub();
+          services.trigger = sinon.stub();
 
           services.initialize();
 
-          // call the onReady callback
+          // call the onLoaded callback
           services.listenToOnce.getCall(1).args[2]();
 
           await waitForAsync();
 
           assert.isTrue(services.initFailed);
+          assert.isTrue(services.ready);
+          sinon.assert.calledWith(services.trigger, 'services:initialized');
           sinon.assert.calledWith(
             services.logger.error,
             `services: failed to init initial services when no credentials available, ${expectedMessage}`
@@ -95,32 +145,132 @@ describe('webex-core', () => {
 
       it.each([
         {error: new Error('failed'), expectedMessage: 'failed'},
-        {error: undefined, expectedMessage: undefined}
-      ])('sets initFailed to true when initServiceCatalogs errors', async ({error, expectedMessage}) => {
-        services.initServiceCatalogs = sinon.stub().callsFake(() => {
-          return Promise.reject(error);
-        });
-        services.webex.credentials = {
-          supertoken: {
-            access_token: 'token'
-          }
+        {error: undefined, expectedMessage: undefined},
+      ])(
+        'sets initFailed to true when initServiceCatalogs errors but still sets ready to true',
+        async ({error, expectedMessage}) => {
+          services.initServiceCatalogs = sinon.stub().callsFake(() => {
+            return Promise.reject(error);
+          });
+          services.webex.credentials = {
+            supertoken: {
+              access_token: 'token',
+            },
+          };
+
+          services.listenToOnce = sinon.stub();
+          services.logger.error = sinon.stub();
+          services.trigger = sinon.stub();
+
+          services.initialize();
+
+          // call the onLoaded callback
+          services.listenToOnce.getCall(1).args[2]();
+
+          await waitForAsync();
+
+          assert.isTrue(services.initFailed);
+          assert.isTrue(services.ready);
+          sinon.assert.calledWith(services.trigger, 'services:initialized');
+          sinon.assert.calledWith(
+            services.logger.error,
+            `services: failed to init initial services when credentials available, ${expectedMessage}`
+          );
         }
+      );
 
-        services.listenToOnce = sinon.stub();
-        services.logger.error = sinon.stub();
+      describe('when change:canAuthorize fires', () => {
+        it('calls initServiceCatalogs when canAuthorize becomes true and postauth catalog is not ready', async () => {
+          services.listenToOnce = sinon.stub();
+          services._loadCatalogFromCache = sinon.stub().returns(Promise.resolve(false));
+          services.collectPreauthCatalog = sinon.stub().returns(Promise.resolve());
+          services.initServiceCatalogs = sinon.stub().returns(Promise.resolve());
+          services.trigger = sinon.stub();
+          // Ensure no credentials so we go to the preauth path
+          services.webex.credentials = {};
 
-        services.initialize();
+          services.initialize();
 
-        // call the onReady callback
-        services.listenToOnce.getCall(1).args[2]();
+          // Get the catalog after initialize creates it
+          const testCatalog = services._getCatalog();
+          testCatalog.status.postauth.ready = false;
 
-        await waitForAsync();
+          // call the onLoaded callback (preauth path)
+          services.listenToOnce.getCall(1).args[2]();
+          await waitForAsync();
 
-        assert.isTrue(services.initFailed);
-        sinon.assert.calledWith(
-          services.logger.error,
-          `services: failed to init initial services when credentials available, ${expectedMessage}`
-        );
+          // Now set canAuthorize to true to simulate auth completing
+          services.webex.canAuthorize = true;
+
+          // call the change:canAuthorize callback (should be call 2)
+          services.listenToOnce.getCall(2).args[2]();
+          await waitForAsync();
+
+          sinon.assert.calledOnce(services.initServiceCatalogs);
+          assert.isTrue(testCatalog.isReady);
+        });
+
+        it('does not call initServiceCatalogs when postauth catalog is already ready', async () => {
+          services.listenToOnce = sinon.stub();
+          services._loadCatalogFromCache = sinon.stub().returns(Promise.resolve(false));
+          services.collectPreauthCatalog = sinon.stub().returns(Promise.resolve());
+          services.initServiceCatalogs = sinon.stub().returns(Promise.resolve());
+          services.trigger = sinon.stub();
+          // Ensure no credentials so we go to the preauth path
+          services.webex.credentials = {};
+
+          services.initialize();
+
+          // Get the catalog after initialize creates it
+          const testCatalog = services._getCatalog();
+
+          // call the onLoaded callback (preauth path)
+          services.listenToOnce.getCall(1).args[2]();
+          await waitForAsync();
+
+          // Set canAuthorize to true and postauth as ready BEFORE calling the callback
+          services.webex.canAuthorize = true;
+          testCatalog.status.postauth.ready = true;
+
+          // call the change:canAuthorize callback (should be call 2)
+          services.listenToOnce.getCall(2).args[2]();
+          await waitForAsync();
+
+          sinon.assert.notCalled(services.initServiceCatalogs);
+        });
+
+        it('logs an error when initServiceCatalogs fails after auth', async () => {
+          services.listenToOnce = sinon.stub();
+          services._loadCatalogFromCache = sinon.stub().returns(Promise.resolve(false));
+          services.collectPreauthCatalog = sinon.stub().returns(Promise.resolve());
+          services.initServiceCatalogs = sinon.stub().rejects(new Error('auth failed'));
+          services.logger.error = sinon.stub();
+          services.trigger = sinon.stub();
+          // Ensure no credentials so we go to the preauth path
+          services.webex.credentials = {};
+
+          services.initialize();
+
+          // Get the catalog after initialize creates it
+          const testCatalog = services._getCatalog();
+          testCatalog.status.postauth.ready = false;
+
+          // call the onLoaded callback (preauth path)
+          services.listenToOnce.getCall(1).args[2]();
+          await waitForAsync();
+
+          // Now set canAuthorize to true to simulate auth completing
+          services.webex.canAuthorize = true;
+
+          // call the change:canAuthorize callback (should be call 2)
+          services.listenToOnce.getCall(2).args[2]();
+          await waitForAsync();
+
+          sinon.assert.calledWith(
+            services.logger.error,
+            'services: failed to init service catalogs after auth, auth failed'
+          );
+        });
       });
     });
 
@@ -159,7 +309,7 @@ describe('webex-core', () => {
 
         services.collectPreauthCatalog = sinon.stub().callsFake(() => {
           return Promise.resolve();
-        })
+        });
 
         services.updateServices = sinon.stub().callsFake(() => {
           return Promise.reject(error);
@@ -254,7 +404,7 @@ describe('webex-core', () => {
             discovery: {
               sqdiscovery: 'https://test.ciscospark.com/v1/region',
             },
-          }
+          },
         };
       });
 
@@ -274,8 +424,8 @@ describe('webex-core', () => {
           assert.calledWith(webex.request, {
             uri: 'https://test.ciscospark.com/v1/region',
             addAuthHeader: false,
-            headers: { 'spark-user-agent': null },
-            timeout: 5000
+            headers: {'spark-user-agent': null},
+            timeout: 5000,
           });
         });
       });
@@ -332,7 +482,6 @@ describe('webex-core', () => {
     });
 
     describe('#_fetchNewServiceHostmap()', () => {
-
       beforeEach(() => {
         sinon.spy(webex.internal.newMetrics.callDiagnosticLatencies, 'measureLatency');
       });
@@ -346,17 +495,20 @@ describe('webex-core', () => {
 
         sinon.stub(services, '_formatReceivedHostmap').resolves(mapResponse);
         sinon.stub(services, 'request').resolves({});
-        
+
         const mapResult = await services._fetchNewServiceHostmap({from: 'limited'});
 
         assert.calledOnceWithExactly(services.request, {
           method: 'GET',
           service: 'u2c',
           resource: '/limited/catalog',
-          qs: {format: 'hostmap'}
-        }
+          qs: {format: 'hostmap'},
+        });
+        assert.calledOnceWithExactly(
+          webex.internal.newMetrics.callDiagnosticLatencies.measureLatency,
+          sinon.match.func,
+          'internal.get.u2c.time'
         );
-        assert.calledOnceWithExactly(webex.internal.newMetrics.callDiagnosticLatencies.measureLatency, sinon.match.func, 'internal.get.u2c.time');
       });
 
       it('checks service request rejects', async () => {
@@ -364,7 +516,7 @@ describe('webex-core', () => {
 
         sinon.spy(services, '_formatReceivedHostmap');
         sinon.stub(services, 'request').rejects(error);
-        
+
         const promise = services._fetchNewServiceHostmap({from: 'limited'});
         const rejectedValue = await assert.isRejected(promise);
 
@@ -376,10 +528,13 @@ describe('webex-core', () => {
           method: 'GET',
           service: 'u2c',
           resource: '/limited/catalog',
-          qs: {format: 'hostmap'}
-        }
+          qs: {format: 'hostmap'},
+        });
+        assert.calledOnceWithExactly(
+          webex.internal.newMetrics.callDiagnosticLatencies.measureLatency,
+          sinon.match.func,
+          'internal.get.u2c.time'
         );
-        assert.calledOnceWithExactly(webex.internal.newMetrics.callDiagnosticLatencies.measureLatency, sinon.match.func, 'internal.get.u2c.time');
       });
     });
 
@@ -410,7 +565,6 @@ describe('webex-core', () => {
       });
 
       it('returns the original uri if the hostmap has no hosts for the host', () => {
-
         services._hostCatalog = {
           'example.com': [],
         };
@@ -574,8 +728,7 @@ describe('webex-core', () => {
                 id: '0:0:0:different-e-x',
               },
             ],
-            'example-f.com': [
-            ],
+            'example-f.com': [],
           },
           format: 'hostmap',
         };
@@ -783,7 +936,7 @@ describe('webex-core', () => {
             defaultUrl: 'https://example-g.com/api/v1',
             hosts: [],
             name: 'example-g',
-          }
+          },
         ]);
       });
 
@@ -837,34 +990,59 @@ describe('webex-core', () => {
         assert.equal(webex.config.credentials.authorizeUrl, authUrl);
       });
     });
-    
+
     describe('#getMobiusClusters', () => {
       it('returns unique mobius host entries from hostCatalog', () => {
         // Arrange: two hostCatalog keys, with duplicate mobius host across keys
         services._hostCatalog = {
           'mobius-us-east-2.prod.infra.webex.com': [
-            {host: 'mobius-us-east-2.prod.infra.webex.com', ttl: -1, priority: 5, id: 'urn:TEAM:xyz:mobius'},
-            {host: 'mobius-eu-central-1.prod.infra.webex.com', ttl: -1, priority: 10, id: 'urn:TEAM:xyz:mobius'},
-            ],
+            {
+              host: 'mobius-us-east-2.prod.infra.webex.com',
+              ttl: -1,
+              priority: 5,
+              id: 'urn:TEAM:xyz:mobius',
+            },
+            {
+              host: 'mobius-eu-central-1.prod.infra.webex.com',
+              ttl: -1,
+              priority: 10,
+              id: 'urn:TEAM:xyz:mobius',
+            },
+          ],
 
           'mobius-eu-central-1.prod.infra.webex.com': [
-            {host: 'mobius-us-east-2.prod.infra.webex.com', ttl: -1, priority: 7, id: 'urn:TEAM:xyz:mobius'}, // duplicate host
-            ],
-          'wdm-a.webex.com' : [
+            {
+              host: 'mobius-us-east-2.prod.infra.webex.com',
+              ttl: -1,
+              priority: 7,
+              id: 'urn:TEAM:xyz:mobius',
+            }, // duplicate host
+          ],
+          'wdm-a.webex.com': [
             {host: 'wdm-a.webex.com', ttl: -1, priority: 5, id: 'urn:TEAM:xyz:wdm'},
-          ]
+          ],
         };
-    
+
         // Act
         const clusters = services.getMobiusClusters();
-    
+
         // Assert
         // deduped; only mobius entries; keeps first seen mobius-a, then mobius-b
         assert.deepEqual(
           clusters.map(({host, id, ttl, priority}) => ({host, id, ttl, priority})),
           [
-            {host: 'mobius-us-east-2.prod.infra.webex.com', id: 'urn:TEAM:xyz:mobius', ttl: -1, priority: 5},
-            {host: 'mobius-eu-central-1.prod.infra.webex.com', id: 'urn:TEAM:xyz:mobius', ttl: -1, priority: 10},
+            {
+              host: 'mobius-us-east-2.prod.infra.webex.com',
+              id: 'urn:TEAM:xyz:mobius',
+              ttl: -1,
+              priority: 5,
+            },
+            {
+              host: 'mobius-eu-central-1.prod.infra.webex.com',
+              id: 'urn:TEAM:xyz:mobius',
+              ttl: -1,
+              priority: 10,
+            },
           ]
         );
       });
@@ -873,31 +1051,31 @@ describe('webex-core', () => {
     describe('#isValidHost', () => {
       beforeEach(() => {
         // Setting up a mock host catalog
-          services._hostCatalog = {
-            "audit-ci-m.wbx2.com": [
-              {
-                  "host": "audit-ci-m.wbx2.com",
-                  "ttl": -1,
-                  "priority": 5,
-                  "id": "urn:IDENTITY:PA61:adminAudit"
-              },
-              {
-                  "host": "audit-ci-m.wbx2.com",
-                  "ttl": -1,
-                  "priority": 5,
-                  "id": "urn:IDENTITY:PA61:adminAuditV2"
-              }
-            ],
-            "mercury-connection-partition0-r.wbx2.com": [
-                {
-                    "host": "mercury-connection-partition0-r.wbx2.com",
-                    "ttl": -1,
-                    "priority": 5,
-                    "id": "urn:TEAM:us-west-2_r:mercuryConnectionPartition0"
-                }
-            ],
-            "empty.com": []
-          };
+        services._hostCatalog = {
+          'audit-ci-m.wbx2.com': [
+            {
+              host: 'audit-ci-m.wbx2.com',
+              ttl: -1,
+              priority: 5,
+              id: 'urn:IDENTITY:PA61:adminAudit',
+            },
+            {
+              host: 'audit-ci-m.wbx2.com',
+              ttl: -1,
+              priority: 5,
+              id: 'urn:IDENTITY:PA61:adminAuditV2',
+            },
+          ],
+          'mercury-connection-partition0-r.wbx2.com': [
+            {
+              host: 'mercury-connection-partition0-r.wbx2.com',
+              ttl: -1,
+              priority: 5,
+              id: 'urn:TEAM:us-west-2_r:mercuryConnectionPartition0',
+            },
+          ],
+          'empty.com': [],
+        };
       });
       afterAll(() => {
         // Clean up the mock host catalog
@@ -979,7 +1157,7 @@ describe('webex-core', () => {
       let catalog;
       let localStorageBackup;
       let windowBackup;
-  
+
       const makeLocalStorageShim = () => {
         const store = new Map();
         return {
@@ -989,13 +1167,16 @@ describe('webex-core', () => {
           _store: store,
         };
       };
-  
+
       beforeEach(() => {
         // Build a fresh webex instance
-        webex = new MockWebex({children: {services: Services}, config: {credentials: {federation: true}}});
+        webex = new MockWebex({
+          children: {services: Services},
+          config: {credentials: {federation: true}},
+        });
         services = webex.internal.services;
         catalog = services._getCatalog();
-  
+
         // enable U2C caching feature flag in tests that rely on localStorage writes/reads
         services.webex.config = services.webex.config || {};
         services.webex.config.calling = {...(services.webex.config.calling || {}), cacheU2C: true};
@@ -1007,13 +1188,15 @@ describe('webex-core', () => {
         global.window.localStorage = makeLocalStorageShim();
         // Ensure code under test uses our shim via util method
         sinon.stub(services, '_getLocalStorageSafe').returns(global.window.localStorage);
-  
+
         // Stub the formatter so we don't need a full hostmap payload in tests
-        sinon.stub(services, '_formatReceivedHostmap').callsFake(() => [
-          {name: 'hydra', defaultUrl: 'https://api.ciscospark.com/v1', hosts: []},
-        ]);
+        sinon
+          .stub(services, '_formatReceivedHostmap')
+          .callsFake(() => [
+            {name: 'hydra', defaultUrl: 'https://api.ciscospark.com/v1', hosts: []},
+          ]);
       });
-  
+
       afterEach(() => {
         global.window.localStorage = localStorageBackup || undefined;
         if (!windowBackup) {
@@ -1026,7 +1209,7 @@ describe('webex-core', () => {
           services._getLocalStorageSafe.restore();
         }
       });
-  
+
       it('invokes initServiceCatalogs on ready, caches catalog, and stores in localStorage', async () => {
         // Arrange: authenticated credentials and spies
         services.webex.credentials = {
@@ -1038,10 +1221,20 @@ describe('webex-core', () => {
         const cacheSpy = sinon.spy(services, '_cacheCatalog');
         const setItemSpy = sinon.spy(global.window.localStorage, 'setItem');
         // Make fetch return a hostmap object and allow formatter to reduce it
-        sinon.stub(services, 'request').resolves({body: {services: [], activeServices: {}, timestamp: Date.now().toString(), orgId: 'urn:EXAMPLE:org', format: 'U2CV2'}});
-        // Cause ready callback to run immediately
+        sinon
+          .stub(services, 'request')
+          .resolves({
+            body: {
+              services: [],
+              activeServices: {},
+              timestamp: Date.now().toString(),
+              orgId: 'urn:EXAMPLE:org',
+              format: 'U2CV2',
+            },
+          });
+        // Cause loaded callback to run immediately
         services.listenToOnce = sinon.stub().callsFake((ctx, event, cb) => {
-          if (event === 'ready') cb();
+          if (event === 'loaded') cb();
         });
 
         // Act
@@ -1081,9 +1274,9 @@ describe('webex-core', () => {
 
         const initSpy = sinon.spy(services, 'initServiceCatalogs');
         const cacheSpy = sinon.spy(services, '_cacheCatalog');
-        // Cause ready callback to run immediately
+        // Cause loaded callback to run immediately
         services.listenToOnce = sinon.stub().callsFake((ctx, event, cb) => {
-          if (event === 'ready') cb();
+          if (event === 'loaded') cb();
         });
 
         // Act
@@ -1091,9 +1284,18 @@ describe('webex-core', () => {
         await waitForAsync();
 
         // Assert: ready path found cache and skipped initServiceCatalogs
-        assert.isFalse(initSpy.called, 'expected initServiceCatalogs to be skipped with cache present');
-        assert.isTrue(services._getCatalog().status.preauth.ready, 'preauth should be ready from cache');
-        assert.isTrue(services._getCatalog().status.postauth.ready, 'postauth should be ready from cache');
+        assert.isFalse(
+          initSpy.called,
+          'expected initServiceCatalogs to be skipped with cache present'
+        );
+        assert.isTrue(
+          services._getCatalog().status.preauth.ready,
+          'preauth should be ready from cache'
+        );
+        assert.isTrue(
+          services._getCatalog().status.postauth.ready,
+          'postauth should be ready from cache'
+        );
         assert.isFalse(cacheSpy.called, 'should not write cache during warm-up-only path');
 
         // Cleanup
@@ -1109,26 +1311,29 @@ describe('webex-core', () => {
           preauth: {serviceLinks: {}, hostCatalog: {}},
           postauth: {serviceLinks: {}, hostCatalog: {}},
         };
-  
+
         window.localStorage.setItem(CATALOG_CACHE_KEY_V1, JSON.stringify(staleCached));
-  
+
         const warmed = await services._loadCatalogFromCache();
-  
+
         assert.isFalse(warmed, 'stale cache must not warm');
-        assert.isNull(window.localStorage.getItem(CATALOG_CACHE_KEY_V1), 'expired cache must be cleared');
+        assert.isNull(
+          window.localStorage.getItem(CATALOG_CACHE_KEY_V1),
+          'expired cache must be cleared'
+        );
         assert.isFalse(catalog.status.preauth.ready);
         assert.isFalse(catalog.status.postauth.ready);
       });
-  
+
       it('clearCatalogCache() removes the cached entry', async () => {
         const CATALOG_CACHE_KEY_V1 = 'services.v1.u2cHostMap';
         window.localStorage.setItem(CATALOG_CACHE_KEY_V1, JSON.stringify({cachedAt: Date.now()}));
-  
+
         await services.clearCatalogCache();
-  
+
         assert.isNull(window.localStorage.getItem(CATALOG_CACHE_KEY_V1), 'cache should be cleared');
       });
-  
+
       it('still fetches when forceRefresh=true even if ready', async () => {
         const CATALOG_CACHE_KEY_V1 = 'services.v1.u2cHostMap';
         window.localStorage.setItem(
@@ -1140,20 +1345,24 @@ describe('webex-core', () => {
             postauth: {serviceLinks: {}, hostCatalog: {}},
           })
         );
-  
+
         // warm from cache
         const warmed = await services._loadCatalogFromCache();
         assert.isTrue(warmed);
         assert.isTrue(catalog.status.preauth.ready);
         assert.isTrue(catalog.status.postauth.ready);
-  
+
         const fetchSpy = sinon.spy(services, '_fetchNewServiceHostmap');
-  
+
         // with forceRefresh we should fetch despite ready=true
-        await services.updateServices({from: 'limited', query: {orgId: 'urn:EXAMPLE:org'}, forceRefresh: true});
+        await services.updateServices({
+          from: 'limited',
+          query: {orgId: 'urn:EXAMPLE:org'},
+          forceRefresh: true,
+        });
         // pass an empty query to avoid spreading undefined in qs construction
         await services.updateServices({forceRefresh: true});
-  
+
         assert.isTrue(fetchSpy.called, 'forceRefresh should bypass cache short-circuit');
         fetchSpy.restore();
       });
@@ -1214,9 +1423,11 @@ describe('webex-core', () => {
         );
         // formatter returns at least one entry to mark ready
         services._formatReceivedHostmap.restore && services._formatReceivedHostmap.restore();
-        sinon.stub(services, '_formatReceivedHostmap').callsFake(() => [
-          {name: 'hydra', defaultUrl: 'https://api.ciscospark.com/v1', hosts: []},
-        ]);
+        sinon
+          .stub(services, '_formatReceivedHostmap')
+          .callsFake(() => [
+            {name: 'hydra', defaultUrl: 'https://api.ciscospark.com/v1', hosts: []},
+          ]);
 
         const warmed = await services._loadCatalogFromCache();
         assert.isTrue(warmed);
@@ -1238,9 +1449,11 @@ describe('webex-core', () => {
           })
         );
         services._formatReceivedHostmap.restore && services._formatReceivedHostmap.restore();
-        sinon.stub(services, '_formatReceivedHostmap').callsFake(() => [
-          {name: 'hydra', defaultUrl: 'https://api.ciscospark.com/v1', hosts: []},
-        ]);
+        sinon
+          .stub(services, '_formatReceivedHostmap')
+          .callsFake(() => [
+            {name: 'hydra', defaultUrl: 'https://api.ciscospark.com/v1', hosts: []},
+          ]);
 
         const warmed = await services._loadCatalogFromCache();
         // function returns true if overall cache path succeeded; we only verify group readiness
@@ -1267,12 +1480,17 @@ describe('webex-core', () => {
           })
         );
         services._formatReceivedHostmap.restore && services._formatReceivedHostmap.restore();
-        sinon.stub(services, '_formatReceivedHostmap').callsFake(() => [
-          {name: 'hydra', defaultUrl: 'https://api.ciscospark.com/v1', hosts: []},
-        ]);
+        sinon
+          .stub(services, '_formatReceivedHostmap')
+          .callsFake(() => [
+            {name: 'hydra', defaultUrl: 'https://api.ciscospark.com/v1', hosts: []},
+          ]);
 
         await services._loadCatalogFromCache();
-        assert.isFalse(catalog.status.preauth.ready, 'preauth should not warm on selection mismatch');
+        assert.isFalse(
+          catalog.status.preauth.ready,
+          'preauth should not warm on selection mismatch'
+        );
       });
 
       it('skips warming when environment fingerprint mismatches', async () => {
@@ -1283,7 +1501,10 @@ describe('webex-core', () => {
           JSON.stringify({
             cachedAt: Date.now(),
             env: {fedramp: false, u2cDiscoveryUrl: 'https://u2c.other.com/u2c/api/v1'},
-            preauth: {hostMap: {serviceLinks: {}, hostCatalog: {}}, meta: {selectionType: 'mode', selectionValue: 'DEFAULT_BY_PROXIMITY'}},
+            preauth: {
+              hostMap: {serviceLinks: {}, hostCatalog: {}},
+              meta: {selectionType: 'mode', selectionValue: 'DEFAULT_BY_PROXIMITY'},
+            },
           })
         );
         // current env
@@ -1298,7 +1519,6 @@ describe('webex-core', () => {
         assert.isFalse(catalog.status.postauth.ready);
       });
     });
-    
   });
 });
 /* eslint-enable no-underscore-dangle */