Formatting

Author: sam-gc

packages-exp/auth-exp/src/model/popup_redirect.ts

@@ -87,6 +87,9 @@ export interface PopupRedirectResolver extends externs.PopupRedirectResolver {
     authType: AuthEventType,
     eventId?: string
   ): Promise<never>;
-  _isIframeWebStorageSupported(auth: AuthCore, cb: (support: boolean) => unknown): void;
+  _isIframeWebStorageSupported(
+    auth: AuthCore,
+    cb: (support: boolean) => unknown
+  ): void;
   _redirectPersistence: externs.Persistence;
 }

packages-exp/auth-exp/src/platform_browser/popup_redirect.test.ts

@@ -70,7 +70,7 @@ describe('src/platform_browser/popup_redirect', () => {
               _message: string,
               cb: (event: GapiAuthEvent) => Promise<void>
             ) => (onIframeMessage = cb),
-            send: iframeSendStub,
+            send: iframeSendStub
           })
       } as unknown) as gapi.iframes.Context)
     );
@@ -273,9 +273,15 @@ describe('src/platform_browser/popup_redirect', () => {
     });
 
     function setIframeResponse(value: unknown): void {
-      iframeSendStub.callsFake((_message: string, _event: unknown, callback: (response: unknown) => void) => {
-        callback(value);
-      });
+      iframeSendStub.callsFake(
+        (
+          _message: string,
+          _event: unknown,
+          callback: (response: unknown) => void
+        ) => {
+          callback(value);
+        }
+      );
     }
 
     it('calls the iframe send method with the correct parameters', () => {
@@ -284,21 +290,21 @@ describe('src/platform_browser/popup_redirect', () => {
       const args = iframeSendStub.getCalls()[0].args;
       expect(args[0]).to.eq('webStorageSupport');
       expect(args[1]).to.eql({
-        type: 'webStorageSupport',
+        type: 'webStorageSupport'
       });
       expect(args[3]).to.eq(gapi.iframes.CROSS_ORIGIN_IFRAMES_FILTER);
     });
 
     it('passes through true value from the response to the callback', done => {
-      setIframeResponse([{webStorageSupport: true}]);
+      setIframeResponse([{ webStorageSupport: true }]);
       resolver._isIframeWebStorageSupported(auth, supported => {
         expect(supported).to.be.true;
         done();
       });
     });
 
     it('passes through false value from the response to callback', done => {
-      setIframeResponse([{webStorageSupport: false}]);
+      setIframeResponse([{ webStorageSupport: false }]);
       resolver._isIframeWebStorageSupported(auth, supported => {
         expect(supported).to.be.false;
         done();
@@ -307,7 +313,9 @@ describe('src/platform_browser/popup_redirect', () => {
 
     it('throws an error if the response is malformed', () => {
       setIframeResponse({});
-      expect(() => resolver._isIframeWebStorageSupported(auth, () => {})).to.throw(FirebaseError, 'auth/internal-error');
+      expect(() =>
+        resolver._isIframeWebStorageSupported(auth, () => {})
+      ).to.throw(FirebaseError, 'auth/internal-error');
     });
   });
 });

packages-exp/auth-exp/src/platform_browser/popup_redirect.ts

@@ -135,18 +135,26 @@ class BrowserPopupRedirectResolver implements PopupRedirectResolver {
     return manager;
   }
 
-  _isIframeWebStorageSupported(auth: Auth, cb: (supported: boolean) => unknown): void {
+  _isIframeWebStorageSupported(
+    auth: Auth,
+    cb: (supported: boolean) => unknown
+  ): void {
     const iframe = this.iframes[auth._key()];
-    iframe.send<gapi.iframes.Message, WebStorageSupportMessage>(WEB_STORAGE_SUPPORT_KEY, {type: WEB_STORAGE_SUPPORT_KEY}, result => {
-      const isSupported = result?.[0]?.[WEB_STORAGE_SUPPORT_KEY];
-      if (isSupported !== undefined) {
-        cb(!!isSupported);
-      }
-
-      fail(AuthErrorCode.INTERNAL_ERROR, {
-        appName: auth.name
-      });
-    }, gapi.iframes.CROSS_ORIGIN_IFRAMES_FILTER);
+    iframe.send<gapi.iframes.Message, WebStorageSupportMessage>(
+      WEB_STORAGE_SUPPORT_KEY,
+      { type: WEB_STORAGE_SUPPORT_KEY },
+      result => {
+        const isSupported = result?.[0]?.[WEB_STORAGE_SUPPORT_KEY];
+        if (isSupported !== undefined) {
+          cb(!!isSupported);
+        }
+
+        fail(AuthErrorCode.INTERNAL_ERROR, {
+          appName: auth.name
+        });
+      },
+      gapi.iframes.CROSS_ORIGIN_IFRAMES_FILTER
+    );
   }
 
   private originValidation(auth: Auth): Promise<void> {

packages-exp/auth-exp/src/platform_browser/strategies/popup.test.ts

@@ -200,7 +200,9 @@ describe('src/core/strategies/popup', () => {
 
     it('errors if webstorage support comes back negative', async () => {
       resolver = makeMockPopupRedirectResolver(eventManager, authPopup, false);
-      await expect(signInWithPopup(auth, provider, resolver)).to.be.rejectedWith(FirebaseError, 'auth/web-storage-unsupported');
+      await expect(
+        signInWithPopup(auth, provider, resolver)
+      ).to.be.rejectedWith(FirebaseError, 'auth/web-storage-unsupported');
     });
 
     it('passes any errors from idp task', async () => {
@@ -354,7 +356,10 @@ describe('src/core/strategies/popup', () => {
 
     it('errors if webstorage support comes back negative', async () => {
       resolver = makeMockPopupRedirectResolver(eventManager, authPopup, false);
-      await expect(linkWithPopup(user, provider, resolver)).to.be.rejectedWith(FirebaseError, 'auth/web-storage-unsupported');
+      await expect(linkWithPopup(user, provider, resolver)).to.be.rejectedWith(
+        FirebaseError,
+        'auth/web-storage-unsupported'
+      );
     });
 
     it('passes any errors from idp task', async () => {
@@ -486,7 +491,9 @@ describe('src/core/strategies/popup', () => {
 
     it('errors if webstorage support comes back negative', async () => {
       resolver = makeMockPopupRedirectResolver(eventManager, authPopup, false);
-      await expect(reauthenticateWithPopup(user, provider, resolver)).to.be.rejectedWith(FirebaseError, 'auth/web-storage-unsupported');
+      await expect(
+        reauthenticateWithPopup(user, provider, resolver)
+      ).to.be.rejectedWith(FirebaseError, 'auth/web-storage-unsupported');
     });
 
     it('does error if the poll timeout and event timeout trip', async () => {

packages-exp/auth-exp/src/platform_browser/strategies/popup.ts

@@ -155,9 +155,11 @@ class PopupOperation extends AbstractPopupRedirectOperation {
     // the popup closed by user poll will reject into the void.
     this.resolver._isIframeWebStorageSupported(this.auth, isSupported => {
       if (!isSupported) {
-        this.reject(AUTH_ERROR_FACTORY.create(AuthErrorCode.WEB_STORAGE_UNSUPPORTED, {
-          appName: this.auth.name
-        }));
+        this.reject(
+          AUTH_ERROR_FACTORY.create(AuthErrorCode.WEB_STORAGE_UNSUPPORTED, {
+            appName: this.auth.name
+          })
+        );
       }
     });
 

packages-exp/auth-exp/test/helpers/mock_popup_redirect_resolver.ts

@@ -28,7 +28,7 @@ import { EventManager } from '../../src/model/popup_redirect';
 export function makeMockPopupRedirectResolver(
   eventManager?: EventManager,
   authPopup?: AuthPopup,
-  webStorageSupported = true,
+  webStorageSupported = true
 ): PopupRedirectResolver {
   return class implements PopupRedirectResolver {
     async _initialize(): Promise<EventManager> {
@@ -41,7 +41,10 @@ export function makeMockPopupRedirectResolver(
 
     async _openRedirect(): Promise<void> {}
 
-    _isIframeWebStorageSupported(_auth: unknown, cb: (result: boolean)=>void ): void {
+    _isIframeWebStorageSupported(
+      _auth: unknown,
+      cb: (result: boolean) => void
+    ): void {
       cb(webStorageSupported);
     }