@salesforce/cli-plugins-testkit 3.2.19 → 3.2.20

package/lib/execCmd.js

@@ -17,6 +17,7 @@ const debug_1 = require("debug");
 const shelljs = require("shelljs");
 const shelljs_1 = require("shelljs");
 const stripAnsi = require("strip-ansi");
+const genUniqueString_1 = require("./genUniqueString");
 const buildCmdOptions = (options) => {
     const defaults = {
         env: { ...process.env, ...options?.env },
@@ -34,10 +35,10 @@ const buildCmdOptions = (options) => {
 // Create a Duration instance from process.hrtime
 const hrtimeToMillisDuration = (hrTime) => kit_1.Duration.milliseconds(hrTime[0] * kit_1.Duration.MILLIS_IN_SECONDS + hrTime[1] / 1e6);
 // Add JSON output if json flag is set
-const addJsonOutput = (cmd, result) => {
+const addJsonOutput = (cmd, result, file) => {
     if (cmd.includes('--json')) {
         try {
-            result.jsonOutput = (0, kit_1.parseJson)(stripAnsi(result.shellOutput.stdout));
+            result.jsonOutput = (0, kit_1.parseJson)(stripAnsi(fs.readFileSync(file, 'utf-8')));
         }
         catch (parseErr) {
             result.jsonError = parseErr;
@@ -92,30 +93,49 @@ const execCmdSync = (cmd, options) => {
     const cmdOptions = buildCmdOptions(options);
     debug(`Running cmd: ${cmd}`);
     debug(`Cmd options: ${(0, util_1.inspect)(cmdOptions)}`);
+    const stdoutFile = `${(0, genUniqueString_1.genUniqueString)('stdout')}.txt`;
+    const stderrFile = `${(0, genUniqueString_1.genUniqueString)('stderr')}.txt`;
+    const stdoutFileLocation = (0, path_1.join)(process.cwd(), stdoutFile);
+    const stderrFileLocation = (0, path_1.join)(process.cwd(), stderrFile);
     const result = {
-        shellOutput: '',
+        shellOutput: new shelljs_1.ShellString(''),
         execCmdDuration: kit_1.Duration.seconds(0),
     };
     // Execute the command in a synchronous child process
     const startTime = process.hrtime();
-    result.shellOutput = shelljs.exec(cmd, cmdOptions);
-    result.shellOutput.stdout = stripAnsi(result.shellOutput.stdout);
-    result.shellOutput.stderr = stripAnsi(result.shellOutput.stderr);
+    const code = shelljs.exec(`${cmd} 1> ${stdoutFile} 2> ${stderrFile} `, cmdOptions).code;
+    if (code === 0) {
+        result.shellOutput = new shelljs_1.ShellString(stripAnsi(fs.readFileSync(stdoutFileLocation, 'utf-8')));
+        result.shellOutput.stdout = stripAnsi(result.shellOutput.stdout);
+    }
+    else {
+        result.shellOutput = new shelljs_1.ShellString(stripAnsi(fs.readFileSync(stderrFileLocation, 'utf-8')));
+        // The ShellString constructor sets the argument as stdout, so we strip 'stdout' and set as stderr
+        result.shellOutput.stderr = stripAnsi(result.shellOutput.stdout);
+    }
+    result.shellOutput.code = code;
     result.execCmdDuration = hrtimeToMillisDuration(process.hrtime(startTime));
     debug(`Command completed with exit code: ${result.shellOutput.code}`);
     if ((0, ts_types_1.isNumber)(cmdOptions.ensureExitCode) && result.shellOutput.code !== cmdOptions.ensureExitCode) {
         throw getExitCodeError(cmd, cmdOptions.ensureExitCode, result.shellOutput);
     }
-    return addJsonOutput(cmd, result);
+    const withJson = addJsonOutput(cmd, result, stdoutFileLocation);
+    fs.rmSync(stderrFileLocation);
+    fs.rmSync(stdoutFileLocation);
+    return withJson;
 };
 const execCmdAsync = async (cmd, options) => {
     const debug = (0, debug_1.default)('testkit:execCmdAsync');
     // Add on the bin path
     cmd = buildCmd(cmd, options);
-    const resultPromise = new Promise((resolve, reject) => {
+    return new Promise((resolve, reject) => {
         const cmdOptions = buildCmdOptions(options);
         debug(`Running cmd: ${cmd}`);
         debug(`Cmd options: ${(0, util_1.inspect)(cmdOptions)}`);
+        const stdoutFile = `${(0, genUniqueString_1.genUniqueString)('stdout')}.txt`;
+        const stderrFile = `${(0, genUniqueString_1.genUniqueString)('stderr')}.txt`;
+        const stdoutFileLocation = (0, path_1.join)(process.cwd(), stdoutFile);
+        const stderrFileLocation = (0, path_1.join)(process.cwd(), stderrFile);
         const callback = (code, stdout, stderr) => {
             const execCmdDuration = hrtimeToMillisDuration(process.hrtime(startTime));
             debug(`Command completed with exit code: ${code}`);
@@ -126,19 +146,28 @@ const execCmdAsync = async (cmd, options) => {
                 reject(getExitCodeError(cmd, cmdOptions.ensureExitCode, output));
             }
             const result = {
-                shellOutput: new shelljs_1.ShellString(stdout),
+                shellOutput: new shelljs_1.ShellString(fs.readFileSync(stdoutFileLocation, 'utf-8')),
                 execCmdDuration,
             };
             result.shellOutput.code = code;
-            result.shellOutput.stdout = stripAnsi(stdout);
-            result.shellOutput.stderr = stripAnsi(stderr);
-            resolve(addJsonOutput(cmd, result));
+            if (code === 0) {
+                result.shellOutput = new shelljs_1.ShellString(stripAnsi(fs.readFileSync(stdoutFileLocation, 'utf-8')));
+                result.shellOutput.stdout = stripAnsi(result.shellOutput.stdout);
+            }
+            else {
+                result.shellOutput = new shelljs_1.ShellString(stripAnsi(fs.readFileSync(stderrFileLocation, 'utf-8')));
+                // The ShellString constructor sets the argument as stdout, so we strip 'stdout' and set as stderr
+                result.shellOutput.stderr = stripAnsi(result.shellOutput.stdout);
+            }
+            const addJson = addJsonOutput(cmd, result, stdoutFileLocation);
+            fs.rmSync(stdoutFileLocation);
+            fs.rmSync(stderrFileLocation);
+            resolve(addJson);
         };
         // Execute the command async in a child process
         const startTime = process.hrtime();
-        shelljs.exec(cmd, cmdOptions, callback);
+        shelljs.exec(`${cmd} 1> ${stdoutFile} 2> ${stderrFile}`, cmdOptions, callback);
     });
-    return resultPromise;
 };
 function execCmd(cmd, options) {
     if (options?.async) {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@salesforce/cli-plugins-testkit",
   "description": "Provides test utilities to assist Salesforce CLI plug-in authors with writing non-unit tests (NUT).",
-  "version": "3.2.19",
+  "version": "3.2.20",
   "author": "Salesforce",
   "license": "BSD-3-Clause",
   "main": "lib/index.js",
@@ -62,7 +62,7 @@
     "@typescript-eslint/eslint-plugin": "^5.45.0",
     "@typescript-eslint/parser": "^5.48.2",
     "chai": "^4.3.7",
-    "eslint": "^8.31.0",
+    "eslint": "^8.32.0",
     "eslint-config-prettier": "^8.6.0",
     "eslint-config-salesforce": "^1.1.0",
     "eslint-config-salesforce-license": "^0.2.0",