Fix popup/redirect stuff

Author: sam-gc

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

@@ -29,7 +29,9 @@ describe('popup_redirect/CompatPopupRedirectResolver', () => {
   // Do not run these tests in node; in node, this resolver
   // is never instantiated.
   if (typeof window === 'undefined') {
-    console.log('Skipping popup/redirect resolver tests in non-browser environment');
+    console.log(
+      'Skipping popup/redirect resolver tests in non-browser environment'
+    );
     return;
   }
 
@@ -82,9 +84,11 @@ describe('popup_redirect/CompatPopupRedirectResolver', () => {
 
     beforeEach(() => {
       underlyingResolver = sinon.createStubInstance(FakeResolver);
-      ((compatResolver as unknown) as {
-        underlyingResolver: exp.PopupRedirectResolverInternal;
-      }).underlyingResolver = underlyingResolver;
+      (
+        compatResolver as unknown as {
+          underlyingResolver: exp.PopupRedirectResolverInternal;
+        }
+      ).underlyingResolver = underlyingResolver;
       provider = new exp.GoogleAuthProvider();
     });
 

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

@@ -22,14 +22,13 @@ const _assert: typeof exp._assert = exp._assert;
 
 /** Platform-agnostic popup-redirect resolver */
 export class CompatPopupRedirectResolver
-  implements exp.PopupRedirectResolverInternal {
+  implements exp.PopupRedirectResolverInternal
+{
   // Create both resolvers for dynamic resolution later
-  private readonly browserResolver: exp.PopupRedirectResolverInternal = exp._getInstance(
-    exp.browserPopupRedirectResolver
-  ); 
-  private readonly cordovaResolver: exp.PopupRedirectResolverInternal = exp._getInstance(
-    exp.cordovaPopupRedirectResolver
-  );
+  private readonly browserResolver: exp.PopupRedirectResolverInternal =
+    exp._getInstance(exp.browserPopupRedirectResolver);
+  private readonly cordovaResolver: exp.PopupRedirectResolverInternal =
+    exp._getInstance(exp.cordovaPopupRedirectResolver);
   // The actual resolver in use: either browserResolver or cordovaResolver.
   private underlyingResolver: exp.PopupRedirectResolverInternal | null = null;
   _redirectPersistence = exp.browserSessionPersistence;
@@ -103,6 +102,8 @@ export class CompatPopupRedirectResolver
     // We haven't yet determined whether or not we're in Cordova; go ahead
     // and determine that state now.
     const isCordova = await _isCordova();
-    this.underlyingResolver = isCordova ? this.cordovaResolver : this.browserResolver;
+    this.underlyingResolver = isCordova
+      ? this.cordovaResolver
+      : this.browserResolver;
   }
 }