powershell fixes (#44)

* change powershell args to prepend EAP

* change powershell args to prepend EAP part 2

* change powershell args to prepend EAP part 3

* change powershell args to prepend EAP part 4

* remove failing test

* remove failing test part 2

Author: pirog

CHANGELOG.md

@@ -1,3 +1,7 @@
+## v1.0.0-beta.3 - [March 15, 2024](https://github.com/lando/leia/releases/tag/v1.0.0-beta.3)
+
+* Fixed bug with `powershell` not throwing errors correctly
+
 ## v1.0.0-beta.2 - [March 14, 2024](https://github.com/lando/leia/releases/tag/v1.0.0-beta.2)
 
 * Added support for `powershell` and `pwsh`

lib/parse.js

@@ -45,7 +45,7 @@ const parseTestDescribe = (describe) => _.lowerCase(_.trim(_.trimStart(describe,
 /*
  * Helper to parse a test command
  */
-const parseTestCommand = (test) => {
+const parseTestCommand = (test, shell) => {
   const newLine = detectNewline(test);
   // Start by removing trailing or fronting whitespace
   test = _(test.split(newLine)).map((line) => _.trim(line)).value().join(newLine);
@@ -57,6 +57,9 @@ const parseTestCommand = (test) => {
     .map((line) => _.trim(line))
     .value();
 
+  // for powershell we need to prepend the error condition stuff
+  if (shell === 'pwsh' || shell === 'powershell') lines.unshift('$ErrorActionPreference = "Stop"');
+
   // Concat and return
   return lines.join(os.EOL);
 };
@@ -73,7 +76,7 @@ const parseCodeBlock = ({text, file, shell}) => {
     return {
       args: scriptArgs.replace('{0}', scriptPath),
       shell: shell.binary.split(path.sep).join('/'),
-      command: parseTestCommand(test),
+      command: parseTestCommand(test, shell.name),
       script: scriptPath,
       describe: _.map(_.filter(test.split(newLine), (line) => _.startsWith(line, '#')), parseTestDescribe),
     };

lib/shell.js

@@ -57,6 +57,7 @@ module.exports = (shell = userShell()) => {
       args: ['-command', '.', '{0}'],
       extension: '.ps1',
     });
+
   case 'sh':
     return _.assign(data, {args: ['-e', '{0}']});
   case 'zsh':