fix: Set tokenRefresh and refresh as possibly undefined

commercelayer · Nov 23, 2021 · cadd19c · cadd19c
1 parent 811de64
commit cadd19c
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/specs/index.spec.ts b/specs/index.spec.ts
@@ -39,6 +39,7 @@ describe('Sales Channel mode', () => {
     expect(auth).toHaveProperty('refreshToken')
     expect(auth).toHaveProperty('expires')
     expect(auth).toHaveProperty('tokenType')
+    expect(auth.refreshToken).not.toBeDefined()
     expect(typeof auth.accessToken).toBe('string')
     expect(auth.tokenType).toEqual('bearer')
     expect(auth.refreshToken).not.toBeDefined()
@@ -48,6 +49,7 @@ describe('Sales Channel mode', () => {
     expect(auth).toHaveProperty('accessToken')
     expect(auth).toHaveProperty('refresh')
     expect(auth).toHaveProperty('refreshToken')
+    expect(auth.refreshToken).toBeDefined()
     expect(auth).toHaveProperty('expires')
     expect(auth).toHaveProperty('tokenType')
     expect(auth.data).toHaveProperty('owner_id')

diff --git a/src/typings/index.ts b/src/typings/index.ts
@@ -13,10 +13,11 @@ type AuthData = TokenData & {
 }
 
 export type AuthReturnType = Promise<
-  | (Token & {
+  | ((Omit<Token, 'refreshToken' | 'refresh'> & {
       data: AuthData
       expires?: Date
-    })
+    }) &
+      Partial<Pick<Token, 'refreshToken' | 'refresh'>>)
   | null
 >