Merge pull request #8133 from romayalon/romy-fix-exec-tests-and-remov…

…e-log-message

NC | remove input from spawn options print and fix exec test

src/test/unit_tests/jest_tests/test_nc_master_keys_exec.test.js

@@ -21,10 +21,15 @@ key="$1"
 file_name="${TMP_PATH}/noobaa.$key"
 current_version="$2"
 
+TMPFILE="$(mktemp)"
+trap 'rm -f &>/dev/null -- "$TMPFILE"' EXIT
+# write stdin of the script to temp file
+cat > "$TMPFILE"
+
 if [ -e "$file_name" ]; then
     echo '{ "status": "VERSION_MISMATCH" }'
 else
-    out=$(cat > "$file_name" 2>/dev/null)
+    out=$(cat "$TMPFILE" > "$file_name" 2>/dev/null)
     rc=$?
     if [ $rc -eq 0 ]; then
         echo '{ "status": "OK" }'

src/util/os_utils.js

@@ -123,7 +123,7 @@ function spawn(command, args, options, ignore_rc, unref, timeout_ms) {
     return new Promise((resolve, reject) => {
         options = options || {};
         args = args || [];
-        dbg.log0('spawn:', command, args.join(' '), options, ignore_rc);
+        dbg.log0('spawn:', command, args.join(' '), _.omit(options, 'input'), ignore_rc);
         options.stdio = options.stdio || 'inherit';
         const return_stdout = options.return_stdout;
         const proc = child_process.spawn(command, args, options);