PR feedback

Author: avolkovi

packages-exp/auth-exp/src/core/auth/auth_impl.ts

@@ -398,7 +398,7 @@ export class AuthImpl implements Auth, _FirebaseService {
   }
 
   async _redirectUserForId(id: string): Promise<User | null> {
-    // Make sure we've cleared any pending ppersistence actions if we're not in
+    // Make sure we've cleared any pending persistence actions if we're not in
     // the initializer
     if (this._isInitialized) {
       await this.queue(async () => {});

packages-exp/auth-exp/src/core/user/token_manager.ts

@@ -21,7 +21,7 @@ import { Auth } from '../../model/auth';
 import { IdTokenResponse } from '../../model/id_token';
 import { AuthErrorCode } from '../errors';
 import { PersistedBlob } from '../persistence';
-import { assert, debugFail, fail } from '../util/assert';
+import { assert, debugFail } from '../util/assert';
 
 /**
  * The number of milliseconds before the official expiration time of a token
@@ -56,11 +56,13 @@ export class StsTokenManager {
       return this.accessToken;
     }
 
-    if (this.accessToken && !this.refreshToken) {
-      fail(AuthErrorCode.TOKEN_EXPIRED, {
+    assert(
+      this.accessToken && !this.refreshToken,
+      AuthErrorCode.TOKEN_EXPIRED,
+      {
         appName: auth.name
-      });
-    }
+      }
+    ); /*  */
 
     await this.refresh(auth, this.refreshToken!);
     return this.accessToken;