Add shell input (#5)

* Add miniconda-setup to workflow for testing

* Testing own shell impl

* Debug shell input

* Add tests for shell input

Author: aganders3

.github/workflows/test.yml

@@ -17,17 +17,34 @@ jobs:
           npm run all
 
   test: # make sure the action works on a clean machine without building
+
+    defaults:
+      run:
+        shell: bash -l {0}  # for setup-miniconda
+
     strategy:
       matrix:
         os: [ubuntu-latest, windows-latest, macos-latest]
     runs-on: ${{ matrix.os }}
+
     steps:
       - uses: actions/checkout@v2
 
+      - uses: conda-incubator/setup-miniconda@v2
+        with:
+          mamba-version: "*"
+          channels: conda-forge
+          channel-priority: strict
+          python-version: '3.10'
+
       - uses: ./
         name: "Run action from local source (./)"
         with:
+          shell: bash -el {0}
           run: |
+            env
+            which python
+            conda run which python
             echo "this is where your commands would run"
             echo "multiple commands are entered as multi-line input"
           linux-setup-delay: 500

__tests__/main.test.ts

@@ -68,6 +68,36 @@ test("test working dir", () => {
   );
 });
 
+test("test shell input", () => {
+  const env: { [key: string]: string } = { ...process.env } as {
+    [key: string]: string;
+  };
+  env["INPUT_RUN"] = "shopt -q login_shell && echo 'login shell'";
+  env["INPUT_SHELL"] = "bash -l {0}";
+  const options: ExecFileSyncOptions = {
+    env: env,
+  };
+
+  const np = process.execPath;
+  const ip = path.join(__dirname, "..", "lib", "main.js");
+  expect(execFileSync(np, [ip], options).toString()).toContain("login shell");
+});
+
+test("test shell input middle", () => {
+  const env: { [key: string]: string } = { ...process.env } as {
+    [key: string]: string;
+  };
+  env["INPUT_RUN"] = "shopt -q login_shell && echo 'login shell'";
+  env["INPUT_SHELL"] = "bash -l {0} -e";
+  const options: ExecFileSyncOptions = {
+    env: env,
+  };
+
+  const np = process.execPath;
+  const ip = path.join(__dirname, "..", "lib", "main.js");
+  expect(execFileSync(np, [ip], options).toString()).toContain("login shell");
+});
+
 test("test failing command", () => {
   const env: { [key: string]: string } = { ...process.env } as {
     [key: string]: string;

action.yml

@@ -4,6 +4,10 @@ inputs:
   run:
     description: 'commands to execute'
     required: true
+  shell:
+    description: 'shell in which to execute commands from `run` input'
+    required: false
+    default: ""
   linux-pkgs:
     description: 'additional packages to install on linux runner'
     required: false

dist/index.js

@@ -81,11 +81,12 @@ async function startXvfb(env: { [key: string]: string }): Promise<string[]> {
     const result = output.stdout.split("\n");
     console.log("sleep for 1000ms");
     await sleep(1000);
+    console.log("::endgroup::");
     return [result[0], result[1]];
   } else {
+    console.log("::endgroup::");
     throw new Error(`failed to start Xvfb, exit code '${output.exitCode}'`);
   }
-  console.log("::endgroup::");
 }
 
 async function killAllXvfb(sig = 15) {
@@ -100,8 +101,25 @@ async function runCommands(commands: string[], env: { [key: string]: string }) {
     options.cwd = working_dir;
   }
 
-  for (const command of commands) {
-    await exec.exec(command, [], options);
+  const shell = core.getInput("shell");
+  if (shell) {
+    const [shell_bin, ...shell_args] = shell.split(" ");
+    const placeholderIndex = shell_args.indexOf("{0}");
+
+    for (const command of commands) {
+      const args = [...shell_args];
+      if (placeholderIndex > 0) {
+        args.splice(placeholderIndex, 1, "-c", command);
+      } else {
+        args.push("-c", command);
+      }
+      core.debug(`exec: args: ${shell_bin}, args: ${args}`);
+      await exec.exec(shell_bin, args, options);
+    }
+  } else {
+    for (const command of commands) {
+      await exec.exec(command, [], options);
+    }
   }
 }