Commenting change

Author: mansisampat

packages/auth/src/core/auth/auth_impl.test.ts

@@ -308,16 +308,16 @@ describe('core/auth/auth_impl', () => {
       expect(persistenceStub._remove).to.have.been.called;
       expect(auth.currentUser).to.be.null;
     });
-    it('sets currentUser to null, calls remove', async () => {
-      const token: FirebaseToken = {
-        token: 'test-token',
-        expirationTime: 123456789
-      };
-      await auth._updateFirebaseToken(token);
-      await auth.signOut();
-      expect(persistenceStub._remove).to.have.been.called;
-      expect(auth.firebaseToken).to.be.null;
-    });
+    // it('sets currentUser to null, calls remove', async () => {
+    //   const token: FirebaseToken = {
+    //     token: 'test-token',
+    //     expirationTime: 123456789
+    //   };
+    //   await auth._updateFirebaseToken(token);
+    //   await auth.signOut();
+    //   expect(persistenceStub._remove).to.have.been.called;
+    //   expect(auth.firebaseToken).to.be.null;
+    // });
     it('is blocked if a beforeAuthStateChanged callback throws', async () => {
       await auth._updateCurrentUser(testUser(auth, 'test'));
       auth.beforeAuthStateChanged(sinon.stub().throws());

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

@@ -491,7 +491,7 @@ export class AuthImpl implements AuthInternal, _FirebaseService {
     }
 
     // Clear Firebase token for Regional Auth Instance when signOut is called
-    await this._updateFirebaseToken(null);
+    // await this._updateFirebaseToken(null);
     // Prevent callbacks from being called again in _updateCurrentUser, as
     // they were already called in the first line.
     return this._updateCurrentUser(null, /* skipBeforeStateCallbacks */ true);