Fix logic around test results.

echoboomer · Sep 9, 2020 · 093ffe3 · 093ffe3
1 parent 8045cdf
commit 093ffe3
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/hooks/command b/hooks/command
@@ -103,8 +103,10 @@ function terraform-opa-run() {
   if [[ "${FAIL_STEP}" == true && "${OPA_PASS}" == false ]]; then
     echo -e "${cRed}[!] Terraform plan does not meet the policy requirements.${cNone}"
     exit 1
+  elif [[ "${FAIL_STEP}" == false && "${OPA_PASS}" == false ]]; then
+    echo -e "${cRed}[!] Terraform plan does not meet the policy requirements. Option to fail pipeline step is not enabled. Proceed with caution.${cNone}"
   else
-    echo -e "${cRed}[!] Terraform plan does not meet the policy requirements, but the option to enforce failure is not enabled.${cNone}"
+    echo -e "${cGreen}[!] Terraform plan meets the policy requirements!${cNone}"
   fi
 }