change to resolve

Author: roger-zhangg

packages/core/src/shared/env/resolveEnv.ts

@@ -20,7 +20,7 @@ import { asEnvironmentVariables } from '../../auth/credentials/utils'
 import { getIAMConnection } from '../../auth/utils'
 import { ChildProcess } from '../utilities/processUtils'
 
-let unixShellEnvPromise: Promise<typeof process.env> | undefined = undefined
+let unixShellEnvPromise: Promise<typeof process.env | undefined> | undefined = undefined
 let envCacheExpireTime: number
 
 export interface IProcessEnvironment {
@@ -176,7 +176,7 @@ export async function getResolvedShellEnv(env?: IProcessEnvironment): Promise<ty
         if (!unixShellEnvPromise || Date.now() > envCacheExpireTime) {
             // cache valid for 5 minutes
             envCacheExpireTime = Date.now() + 5 * 60 * 1000
-            unixShellEnvPromise = new Promise<NodeJS.ProcessEnv>(async (resolve, reject) => {
+            unixShellEnvPromise = new Promise<NodeJS.ProcessEnv | undefined>(async (resolve, reject) => {
                 const timeout = new Timeout(10000)
 
                 // Resolve shell env and handle errors
@@ -185,11 +185,11 @@ export async function getResolvedShellEnv(env?: IProcessEnvironment): Promise<ty
                     if (shellEnv && Object.keys(shellEnv).length > 0) {
                         resolve(shellEnv)
                     } else {
-                        return undefined
+                        resolve(undefined)
                     }
                 } catch {
                     // failed resolve should not affect other feature.
-                    return undefined
+                    resolve(undefined)
                 }
             })
         }