refactor: change return type of refreshAccessToken to boolean

Author: akalia-atlassian

src/atlclients/authStore.ts

@@ -26,7 +26,6 @@ import {
     UpdateAuthInfoEvent,
 } from './authInfo';
 import { OAuthRefesher } from './oauthRefresher';
-import { Tokens } from './tokens';
 const keychainServiceNameV3 = version.endsWith('-insider') ? 'atlascode-insiders-authinfoV3' : 'atlascode-authinfoV3';
 
 enum Priority {
@@ -306,15 +305,14 @@ export class CredentialManager implements Disposable {
     /**
      * Calls the OAuth provider and updates the access token.
      */
-    public async refreshAccessToken(site: DetailedSiteInfo): Promise<Tokens | undefined> {
+    public async refreshAccessToken(site: DetailedSiteInfo): Promise<boolean> {
         const credentials = await this.getAuthInfo(site);
         if (!isOAuthInfo(credentials)) {
-            return undefined;
+            return false;
         }
         Logger.debug(`refreshingAccessToken for ${site.baseApiUrl} credentialID: ${site.credentialId}`);
 
         const provider: OAuthProvider | undefined = oauthProviderForSite(site);
-        const newTokens = undefined;
         if (provider && credentials) {
             const tokenResponse = await this._refresher.getNewTokens(provider, credentials.refresh);
             if (tokenResponse.tokens) {
@@ -333,7 +331,7 @@ export class CredentialManager implements Disposable {
                 this.saveAuthInfo(site, credentials);
             }
         }
-        return newTokens;
+        return true;
     }
 
     /**