fix: hashicorp/setup-terraform terraform_wrapper (#29)

This adds detection logic for the Hashicorp action's `terraform_wrapper`
output and fails the action run if it exists.

This is because `terraform_wrapper` output makes it impossible to
JSON.parse the `terraform show` command output used by
Open Policy Agent.

README.md

@@ -63,6 +63,14 @@ Use the following to control the action:
     terraform-init: -backend-config="region=ca-central-1"
 ```
 
+# Notes
+If you use the [hashicorp/setup-terraform](https://github.com/hashicorp/setup-terraform) action, `terraform_wrapper` needs to be disabled:
+```yaml
+- uses: hashicorp/setup-terraform@v1
+  with:
+    terraform_wrapper: false
+```
+
 # Contributing
 To setup your local dev environment:
 ```sh

dist/index.js

@@ -14882,9 +14882,17 @@ const action = async () => {
     if (!command.depends || results[command.depends].isSuccess) {
       results[command.key] = execCommand(command, directory);
     } else {
-      results[command.key] = { isSuccess: false };
+      results[command.key] = { isSuccess: false, output: "" };
     }
     isError = isError || !results[command.key].isSuccess;
+
+    // Check for hashicorp/setup-terraform action's terraform_wrapper output
+    if (results[command.key].output.indexOf("::debug::exitcode:") > -1) {
+      core.setFailed(
+        "Error: `hashicorp/setup-terraform` must have `terraform_wrapper: false`"
+      );
+      return;
+    }
   }
 
   // Delete previous PR comments

src/action.js

@@ -60,9 +60,17 @@ const action = async () => {
     if (!command.depends || results[command.depends].isSuccess) {
       results[command.key] = execCommand(command, directory);
     } else {
-      results[command.key] = { isSuccess: false };
+      results[command.key] = { isSuccess: false, output: "" };
     }
     isError = isError || !results[command.key].isSuccess;
+
+    // Check for hashicorp/setup-terraform action's terraform_wrapper output
+    if (results[command.key].output.indexOf("::debug::exitcode:") > -1) {
+      core.setFailed(
+        "Error: `hashicorp/setup-terraform` must have `terraform_wrapper: false`"
+      );
+      return;
+    }
   }
 
   // Delete previous PR comments

test/action.test.js

@@ -183,7 +183,7 @@ describe("action", () => {
   });
 
   test("failed command", async () => {
-    execCommand.mockReturnValue({ isSuccess: false });
+    execCommand.mockReturnValue({ isSuccess: false, output: "" });
 
     await action();
 
@@ -192,7 +192,7 @@ describe("action", () => {
   });
 
   test("allowed to fail", async () => {
-    execCommand.mockReturnValue({ isSuccess: false });
+    execCommand.mockReturnValue({ isSuccess: false, output: "" });
     when(core.getBooleanInput)
       .calledWith("allow-failure")
       .mockReturnValue(true);
@@ -206,6 +206,20 @@ describe("action", () => {
     expect(core.setFailed.mock.calls.length).toBe(0);
   });
 
+  test("detect terraform_wrapper", async () => {
+    execCommand.mockReturnValue({
+      isSuccess: false,
+      output: "Some command output ::debug::exitcode: 1",
+    });
+
+    await action();
+
+    expect(core.setFailed.mock.calls.length).toBe(1);
+    expect(core.setFailed.mock.calls[0]).toEqual([
+      "Error: `hashicorp/setup-terraform` must have `terraform_wrapper: false`",
+    ]);
+  });
+
   test("failed validation", async () => {
     when(core.getBooleanInput).calledWith("comment").mockReturnValue(true);
     when(core.getInput).calledWith("github-token").mockReturnValue("false");