add subprocess_run test

Author: scivision

+stdlib/+sys/is_windows_powershell.m

@@ -0,0 +1,16 @@
+function iswps = is_windows_powershell()
+%% detects Windows powershell vs. Command Prompt
+% Matlab on Windows Powershell vs Comspec for system()
+% would impact syntax of some system() commands
+%
+% adapted from: https://stackoverflow.com/a/34480405/7703794
+
+iswps = false;
+if ~ispc
+  return
+end
+
+[ret, ~] = system("dir 2>&1 *`");
+iswps = ret == 0;
+
+end

+stdlib/+sys/subprocess_run.m

@@ -27,13 +27,11 @@
   cmd = exe;
 end
 
-if ~isempty(fieldnames(opt.env))
-  for f = string(fieldnames(opt.env)).'
-    if ispc
-      cmd = append("set ", f, "=", opt.env.(f), " && ", cmd);
-    else
-      cmd = append(f, "=", opt.env.(f), " ", cmd);
-    end
+for f = string(fieldnames(opt.env)).'
+  if ispc
+    cmd = append("set ", f, "=", opt.env.(f), " && ", cmd);
+  else
+    cmd = append(f, "=", opt.env.(f), " ", cmd);
   end
 end
 

+stdlib/TestSys.m

@@ -2,13 +2,39 @@
 
 methods (Test)
 
-function test_run(tc)
+function test_simple_run(tc)
+
+import stdlib.sys.subprocess_run
+
 if ispc
-  cmd = 'dir';
+  c = 'dir';
 else
-  cmd = 'ls';
+  c = 'ls';
 end
-tc.verifyEqual(stdlib.sys.subprocess_run(cmd), 0)
+tc.verifyEqual(subprocess_run(c), 0)
+end
+
+
+function test_env_run(tc)
+
+import stdlib.sys.subprocess_run
+
+names = ["TEST1", "TEST2"];
+vals = ["test123", "test321"];
+
+env = struct(names(1), vals(1), names(2), vals(2));
+
+% NOTE: test function cannot get specific variables without invoking a
+% subshell, as echo is evaluated in current shell before &&
+if ispc
+  c = "set";
+else
+  c = "env";
+end
+[ret, msg] = subprocess_run(c, env=env);
+tc.verifyEqual(ret, 0)
+tc.verifyTrue(contains(msg, names(1) + "=" + vals(1)) && contains(msg, names(2) + "=" + vals(2)))
+
 end
 
 function test_find_fortran(tc)