fixed tests

descope · Dec 18, 2024 · a9bbf39 · a9bbf39
1 parent f002ee1
commit a9bbf39
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/packages/sdks/web-js-sdk/src/sdk/fedcm.ts b/packages/sdks/web-js-sdk/src/sdk/fedcm.ts
@@ -195,9 +195,6 @@ const createFedCM = (sdk: CoreSdk, projectId: string) => ({
   async isLoggedIn(
     context?: IdentityCredentialRequestOptionsContext,
   ): Promise<boolean> {
-    if (!this.isSupported()) {
-      return false;
-    }
     const configURL = sdk.httpClient.buildUrl(
       projectId + apiPaths.fedcm.config,
     );

diff --git a/packages/sdks/web-js-sdk/test/fedcm.test.ts b/packages/sdks/web-js-sdk/test/fedcm.test.ts
@@ -242,6 +242,8 @@ describe('fedcm', () => {
 
     it('should return false if navigator.credentials.get returns null', async () => {
       const mockGet = jest.fn();
+      // @ts-ignore
+      global.navigator.credentials = { get: mockGet };
       mockGet.mockResolvedValue(null);
 
       const result = await sdk.fedcm.isLoggedIn();
@@ -252,6 +254,8 @@ describe('fedcm', () => {
 
     it('should return false if navigator.credentials.get throws an error', async () => {
       const mockGet = jest.fn();
+      // @ts-ignore
+      global.navigator.credentials = { get: mockGet };
       mockGet.mockRejectedValue(new Error('Test Error'));
 
       const result = await sdk.fedcm.isLoggedIn();