remove debugging statement and fix spacing

Author: Hweinstock

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

@@ -349,7 +349,7 @@ export class Ec2Connecter implements vscode.Disposable {
  */
 export function getRemoveLinesCommand(pattern: string, hostOS: Ec2OS, filepath: string): string {
     // Linux allows not passing extension to -i, whereas macOS requires zero length extension.
-    return `sed -i ${isLinux(hostOS) ? '' : "''"} /${pattern}/d ${filepath}`
+    return `sed -i${isLinux(hostOS) ? '' : " ''"} /${pattern}/d ${filepath}`
 }
 
 function isLinux(os: Ec2OS): boolean {

packages/core/src/test/awsService/ec2/model.test.ts

@@ -18,6 +18,7 @@ import { fs } from '../../../shared'
 import path from 'path'
 import { ChildProcess } from '../../../shared/utilities/processUtils'
 import { isMac, isWin } from '../../../shared/vscode/env'
+import { inspect } from '../../../shared/utilities/collectionUtils'
 
 describe('Ec2ConnectClient', function () {
     let client: Ec2Connecter
@@ -227,12 +228,14 @@ describe('getRemoveLinesCommand', async function () {
         const textFile = path.join(tempPath.uri.fsPath, 'test.txt')
         const originalContent = lineToStr(lines)
         await fs.writeFile(textFile, originalContent)
-        console.log('running on os: %s', hostOS)
         const [command, ...args] = getRemoveLinesCommand('pattern', hostOS, textFile).split(' ')
-        console.log('running command: %s with args %s', command, args.join(' '))
         const process = new ChildProcess(command, args, { collect: true })
         const result = await process.run()
-        assert.strictEqual(result.exitCode, 0, `ChildProcess failed with error=${result.error}`)
+        assert.strictEqual(
+            result.exitCode,
+            0,
+            `Ran command '${command} ${args.join(' ')}' and failed with result ${inspect(result)}`
+        )
 
         const newContent = await fs.readFileText(textFile)
         assert.notStrictEqual(newContent, originalContent)