PR feedback

Author: sam-gc

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

@@ -74,6 +74,14 @@ describe('core/user/token_manager', () => {
     });
   });
 
+  describe('#clearRefreshToken', () => {
+    it('sets refresh token to null', () => {
+      stsTokenManager.refreshToken = 'refresh-token';
+      stsTokenManager.clearRefreshToken();
+      expect(stsTokenManager.refreshToken).to.be.null;
+    });
+  });
+
   describe('#getToken', () => {
     context('with endpoint setup', () => {
       let mock: fetch.Route;

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

@@ -81,6 +81,10 @@ export class StsTokenManager {
     };
   }
 
+  clearRefreshToken(): void {
+    this.refreshToken = null;
+  }
+
   toPlainObject(): object {
     return {
       refreshToken: this.refreshToken,

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

@@ -103,7 +103,7 @@ export class UserImpl implements User {
   async delete(): Promise<void> {
     const idToken = await this.getIdToken();
     await deleteAccount(this.auth, { idToken });
-    this.stsTokenManager.refreshToken = null;
+    this.stsTokenManager.clearRefreshToken();
 
     // TODO: Determine if cancellable-promises are necessary to use in this class so that delete()
     //       cancels pending actions...