Skip to content

Commit a1b8bda

Browse files
authored
fix: error when current context is undefined (#870)
1 parent 8db2d5d commit a1b8bda

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

src/lib/auth-helper.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ async function getAccessToken ({ useCachedToken = false } = {}) {
3535

3636
let contextName = CLI // default
3737
const currentContext = await context.getCurrent() // potential override
38-
if (currentContext !== CLI) {
38+
if (currentContext && currentContext !== CLI) {
3939
contextName = currentContext
4040
} else {
4141
await context.setCli({ 'cli.bare-output': true }, false) // set this globally

test/commands/lib/auth-helper.test.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,12 +11,12 @@ describe('getAccessToken', () => {
1111
jest.clearAllMocks()
1212
})
1313

14-
test('should get token using CLI context by default', async () => {
14+
test('should get token using CLI context (default) if current context undefined', async () => {
1515
const mockToken = 'mocked-token'
1616
const mockEnv = 'prod'
1717
getToken.mockResolvedValue(mockToken)
1818
getCliEnv.mockReturnValue(mockEnv)
19-
context.getCurrent.mockResolvedValue(CLI)
19+
context.getCurrent.mockResolvedValue(undefined)
2020

2121
const result = await getAccessToken()
2222

0 commit comments

Comments
 (0)