fix: remove colour codes from commands (#34)

cds-snc · Jul 9, 2021 · 78f2fd2 · 78f2fd2
1 parent b120080
commit 78f2fd2
Show file tree

Hide file tree

Showing 4 changed files with 20 additions and 16 deletions.
diff --git a/dist/index.js b/dist/index.js
@@ -14847,11 +14847,13 @@ const action = async () => {
   const commands = [
     {
       key: "init",
-      exec: `${binary} init ${terraformInit ? terraformInit.join(" ") : ""}`,
+      exec: `${binary} init -no-color ${
+        terraformInit ? terraformInit.join(" ") : ""
+      }`,
     },
     {
       key: "validate",
-      exec: `${binary} validate`,
+      exec: `${binary} validate -no-color`,
     },
     {
       key: "fmt",
@@ -14863,7 +14865,7 @@ const action = async () => {
     },
     {
       key: "show",
-      exec: `${binary} show -json plan.tfplan`,
+      exec: `${binary} show -no-color -json plan.tfplan`,
       depends: "plan",
       output: false,
     },

diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/action.js b/src/action.js
@@ -25,11 +25,13 @@ const action = async () => {
   const commands = [
     {
       key: "init",
-      exec: `${binary} init ${terraformInit ? terraformInit.join(" ") : ""}`,
+      exec: `${binary} init -no-color ${
+        terraformInit ? terraformInit.join(" ") : ""
+      }`,
     },
     {
       key: "validate",
-      exec: `${binary} validate`,
+      exec: `${binary} validate -no-color`,
     },
     {
       key: "fmt",
@@ -41,7 +43,7 @@ const action = async () => {
     },
     {
       key: "show",
-      exec: `${binary} show -json plan.tfplan`,
+      exec: `${binary} show -no-color -json plan.tfplan`,
       depends: "plan",
       output: false,
     },

diff --git a/test/action.test.js b/test/action.test.js
@@ -36,14 +36,14 @@ describe("action", () => {
       [
         {
           key: "init",
-          exec: "terraform init -backend-config='bucket=some-bucket' -backend-config='region=ca-central-1'",
+          exec: "terraform init -no-color -backend-config='bucket=some-bucket' -backend-config='region=ca-central-1'",
         },
         "foo",
       ],
       [
         {
           key: "validate",
-          exec: "terraform validate",
+          exec: "terraform validate -no-color",
         },
         "foo",
       ],
@@ -64,7 +64,7 @@ describe("action", () => {
       [
         {
           key: "show",
-          exec: "terraform show -json plan.tfplan",
+          exec: "terraform show -no-color -json plan.tfplan",
           depends: "plan",
           output: false,
         },
@@ -87,14 +87,14 @@ describe("action", () => {
       [
         {
           key: "init",
-          exec: "terragrunt init ",
+          exec: "terragrunt init -no-color ",
         },
         "bar",
       ],
       [
         {
           key: "validate",
-          exec: "terragrunt validate",
+          exec: "terragrunt validate -no-color",
         },
         "bar",
       ],
@@ -115,7 +115,7 @@ describe("action", () => {
       [
         {
           key: "show",
-          exec: "terragrunt show -json plan.tfplan",
+          exec: "terragrunt show -no-color -json plan.tfplan",
           depends: "plan",
           output: false,
         },
@@ -189,11 +189,11 @@ describe("action", () => {
 
     expect(core.setFailed.mock.calls.length).toBe(1);
     expect(core.setFailed.mock.calls[0][0]).toBe(`The following commands failed:
-terraform init 
-terraform validate
+terraform init -no-color 
+terraform validate -no-color
 terraform fmt --check
 terraform plan -no-color -input=false -out=plan.tfplan
-terraform show -json plan.tfplan`);
+terraform show -no-color -json plan.tfplan`);
   });
 
   test("allowed to fail", async () => {