🐛 access token not saved after refresh #74

Author: Crhistian

codegen/templates/api/Tokens.ts

@@ -153,7 +153,9 @@ class Tokens {
     if (clientID) {
       try {
         const refreshRequest = await Auth.RefreshToken(refreshToken, clientID)
-        return refreshRequest.access_token
+        const accessToken = refreshRequest.access_token
+        this.SetAccessToken(accessToken)
+        return accessToken
       } catch (e) {
         return ''
       }

src/api/Tokens.ts

@@ -153,7 +153,9 @@ class Tokens {
     if (clientID) {
       try {
         const refreshRequest = await Auth.RefreshToken(refreshToken, clientID)
-        return refreshRequest.access_token
+        const accessToken = refreshRequest.access_token
+        this.SetAccessToken(accessToken)
+        return accessToken
       } catch (e) {
         return ''
       }

tests/invalid-token.test.ts

@@ -31,7 +31,7 @@ beforeEach(() => {
     baseApiUrl: 'https://api.ordercloud.io',
     apiVersion: 'v1',
     timeoutInMilliseconds: 10 * 1000,
-    clientID: null,
+    clientID: undefined,
   })
 })
 
@@ -72,7 +72,7 @@ describe('has expired access token', () => {
               access_token: testdata.accessTokenFromRefresh,
               expires_in: 32000000000,
               token_type: 'bearer',
-              refresh_token: undefined,
+              refresh_token: 'mockAccessToken',
             }
             return Promise.resolve(response)
           })
@@ -110,7 +110,7 @@ describe('has expired access token', () => {
               access_token: testdata.accessTokenFromRefresh,
               expires_in: 32000000000,
               token_type: 'bearer',
-              refresh_token: undefined,
+              refresh_token: 'mockAccessToken',
             }
             return Promise.resolve(response)
           })
@@ -165,14 +165,15 @@ describe('has no access token', () => {
         Configuration.Set({ clientID: testdata.clientID })
 
         const GetRefreshTokenSpy = jest.spyOn(Tokens, 'GetRefreshToken')
+        const SetAccessTokenSpy = jest.spyOn(Tokens, 'SetAccessToken')
         const RefreshTokenSpy = jest
           .spyOn(Auth, 'RefreshToken')
           .mockImplementationOnce(() => {
             const response: RequiredDeep<AccessToken> = {
               access_token: testdata.accessTokenFromRefresh,
               expires_in: 32000000000,
               token_type: 'bearer',
-              refresh_token: undefined,
+              refresh_token: 'mockAccessToken',
             }
             return Promise.resolve(response)
           })
@@ -186,6 +187,10 @@ describe('has no access token', () => {
           testdata.refreshToken,
           testdata.clientID
         )
+        expect(SetAccessTokenSpy).toHaveBeenCalledTimes(1)
+        expect(SetAccessTokenSpy).toHaveBeenCalledWith(
+          testdata.accessTokenFromRefresh
+        )
         expect(mockAxios.delete).toHaveBeenCalledWith(
           `${apiUrl}/products/${testdata.productID}`,
           {