avoid circular dependency

Author: Hweinstock

packages/core/src/awsService/ec2/sshKeyPair.ts

@@ -4,7 +4,7 @@
  */
 import { fs } from '../../shared'
 import { ToolkitError } from '../../shared/errors'
-import { tryRun } from '../../shared/utilities/processUtils'
+import { tryRun } from '../../shared/utilities/programUtils'
 import { Timeout } from '../../shared/utilities/timeoutUtils'
 import { findAsync } from '../../shared/utilities/collectionUtils'
 import { RunParameterContext } from '../../shared/utilities/processUtils'

packages/core/src/codewhisperer/commands/basicCommands.ts

@@ -43,7 +43,7 @@ import { isBuilderIdConnection } from '../../auth/connection'
 import globals from '../../shared/extensionGlobals'
 import { getVscodeCliPath } from '../../shared/utilities/pathFind'
 import { setContext } from '../../shared/vscode/setContext'
-import { tryRun } from '../../shared/utilities/processUtils'
+import { tryRun } from '../../shared/utilities/programUtils'
 
 const MessageTimeOut = 5_000
 

packages/core/src/shared/sam/cli/samCliLocator.ts

@@ -10,7 +10,7 @@ import { getLogger, Logger } from '../../logger'
 import { SamCliInfoInvocation } from './samCliInfo'
 import { DefaultSamCliValidator, SamCliValidatorContext, SamCliVersionValidation } from './samCliValidator'
 import { PerfLog } from '../../logger/perfLogger'
-import { tryRun } from '../../utilities/processUtils'
+import { tryRun } from '../../utilities/programUtils'
 
 export class SamCliLocationProvider {
     private static samCliLocator: BaseSamCliLocator | undefined

packages/core/src/shared/utilities/pathFind.ts

@@ -8,7 +8,7 @@ import * as path from 'path'
 import fs from '../../shared/fs/fs'
 import { GitExtension } from '../extensions/git'
 import { Settings } from '../settings'
-import * as processUtils from './processUtils'
+import * as programUtils from './programUtils'
 
 /** Full path to VSCode CLI. */
 let vscPath: string
@@ -50,7 +50,7 @@ export async function getVscodeCliPath(): Promise<string | undefined> {
         if (!vsc || (vsc !== 'code' && !(await fs.exists(vsc)))) {
             continue
         }
-        if (await processUtils.tryRun(vsc, ['--version'])) {
+        if (await programUtils.tryRun(vsc, ['--version'])) {
             vscPath = vsc
             return vsc
         }
@@ -75,7 +75,7 @@ export async function findTypescriptCompiler(): Promise<string | undefined> {
 
     for (const tsc of tscPaths) {
         // Try to run "tsc -v".
-        if (await processUtils.tryRun(tsc, ['-v'], 'yes', 'Version')) {
+        if (await programUtils.tryRun(tsc, ['-v'], 'yes', 'Version')) {
             return tsc
         }
     }
@@ -104,7 +104,7 @@ export async function findSshPath(useCache: boolean = true): Promise<string | un
         if (!p || ('ssh' !== p && !(await fs.exists(p)))) {
             continue
         }
-        if (await processUtils.tryRun(p, ['-G', 'x'], 'noresult' /* "ssh -G" prints quasi-sensitive info. */)) {
+        if (await programUtils.tryRun(p, ['-G', 'x'], 'noresult' /* "ssh -G" prints quasi-sensitive info. */)) {
             sshPath = p
             return p
         }
@@ -126,7 +126,7 @@ export async function findGitPath(): Promise<string | undefined> {
         if (!p || ('git' !== p && !(await fs.exists(p)))) {
             continue
         }
-        if (await processUtils.tryRun(p, ['--version'])) {
+        if (await programUtils.tryRun(p, ['--version'])) {
             gitPath = p
             return p
         }
@@ -146,7 +146,7 @@ export async function findBashPath(): Promise<string | undefined> {
         if (!p || ('bash' !== p && !(await fs.exists(p)))) {
             continue
         }
-        if (await processUtils.tryRun(p, ['--version'])) {
+        if (await programUtils.tryRun(p, ['--version'])) {
             bashPath = p
             return p
         }

packages/core/src/shared/utilities/processUtils.ts

@@ -7,7 +7,6 @@ import * as proc from 'child_process'
 import * as crossSpawn from 'cross-spawn'
 import * as logger from '../logger'
 import { Timeout, CancellationError, waitUntil } from './timeoutUtils'
-import { getLogger } from '..'
 
 export interface RunParameterContext {
     /** Reports an error parsed from the stdin/stdout streams. */
@@ -369,30 +368,3 @@ export class ChildProcess {
         return `${pid} [${this.#command} ${noparams ? '...' : this.#args.join(' ')}]`
     }
 }
-
-/**
- * Tries to execute a program at path `p` with the given args and
- * optionally checks the output for `expected`.
- *
- * @param p path to a program to execute
- * @param args program args
- * @param doLog log failures
- * @param expected output must contain this string
- */
-export async function tryRun(
-    p: string,
-    args: string[],
-    logging: 'yes' | 'no' | 'noresult' = 'yes',
-    expected?: string,
-    opt?: ChildProcessOptions
-): Promise<boolean> {
-    const proc = new ChildProcess(p, args, { logging: 'no' })
-    const r = await proc.run(opt)
-    const ok = r.exitCode === 0 && (expected === undefined || r.stdout.includes(expected))
-    if (logging === 'noresult') {
-        getLogger().info('tryRun: %s: %s', ok ? 'ok' : 'failed', proc)
-    } else if (logging !== 'no') {
-        getLogger().info('tryRun: %s: %s %O', ok ? 'ok' : 'failed', proc, proc.result())
-    }
-    return ok
-}

packages/core/src/shared/utilities/programUtils.ts

@@ -0,0 +1,34 @@
+/*!
+ * Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+import { getLogger } from '..'
+import { ChildProcess, ChildProcessOptions } from './processUtils'
+
+/**
+ * Tries to execute a program at path `p` with the given args and
+ * optionally checks the output for `expected`.
+ *
+ * @param p path to a program to execute
+ * @param args program args
+ * @param doLog log failures
+ * @param expected output must contain this string
+ */
+export async function tryRun(
+    p: string,
+    args: string[],
+    logging: 'yes' | 'no' | 'noresult' = 'yes',
+    expected?: string,
+    opt?: ChildProcessOptions
+): Promise<boolean> {
+    const proc = new ChildProcess(p, args, { logging: 'no' })
+    const r = await proc.run(opt)
+    const ok = r.exitCode === 0 && (expected === undefined || r.stdout.includes(expected))
+    if (logging === 'noresult') {
+        getLogger().info('tryRun: %s: %s', ok ? 'ok' : 'failed', proc)
+    } else if (logging !== 'no') {
+        getLogger().info('tryRun: %s: %s %O', ok ? 'ok' : 'failed', proc, proc.result())
+    }
+    return ok
+}

packages/core/src/test/shared/utilities/pathFind.test.ts

@@ -11,7 +11,7 @@ import * as path from 'path'
 import * as testutil from '../../testUtil'
 import { fs } from '../../../shared'
 import { findSshPath, findTypescriptCompiler, getVscodeCliPath } from '../../../shared/utilities/pathFind'
-import * as processUtils from '../../../shared/utilities/processUtils'
+import * as programUtils from '../../../shared/utilities/programUtils'
 
 describe('pathFind', function () {
     it('findTypescriptCompiler()', async function () {
@@ -47,7 +47,7 @@ describe('pathFind', function () {
     describe('findSshPath', function () {
         let tryRunStub: sinon.SinonStub
         before(function () {
-            tryRunStub = sinon.stub(processUtils, 'tryRun')
+            tryRunStub = sinon.stub(programUtils, 'tryRun')
         })
 
         after(function () {