From dd3a792f27ae1a8c3e33b12e4d1b0621e118254b Mon Sep 17 00:00:00 2001
From: Victor Martinez <VictorMartinezRubio@gmail.com>
Date: Wed, 24 Jun 2020 22:58:45 +0100
Subject: [PATCH] Use isTap function and fix the output format

---
 lib/linter.sh | 30 ++++++++++++++++++++----------
 1 file changed, 20 insertions(+), 10 deletions(-)

diff --git a/lib/linter.sh b/lib/linter.sh
index 32604fb9..a48f8bbe 100755
--- a/lib/linter.sh
+++ b/lib/linter.sh
@@ -500,7 +500,7 @@ LintAnsibleFiles()
     ####################################
     # Prepare context if OUTPUT_FORMAT #
     ####################################
-    if [ -n "${OUTPUT_FORMAT}" ] ; then
+    if IsTAP ; then
       TMPFILE=$(mktemp -q "/tmp/super-linter-${FILE_TYPE}.XXXXXX")
       INDEX=0
       mkdir -p "${REPORT_OUTPUT_FOLDER}"
@@ -562,7 +562,7 @@ LintAnsibleFiles()
         #######################################################
         # Store the linting as a temporary file in TAP format #
         #######################################################
-        if [ -n "${OUTPUT_FORMAT}" ] ; then
+        if IsTAP ; then
           echo "nok ok ${INDEX} - ${FILE}" >> "${TMPFILE}"
           printf "  ---\n  message:[%s]\n  ..." "$LINT_CMD" >> "${TMPFILE}"
         fi
@@ -575,7 +575,7 @@ LintAnsibleFiles()
         #######################################################
         # Store the linting as a temporary file in TAP format #
         #######################################################
-        if [ -n "${OUTPUT_FORMAT}" ] ; then
+        if IsTAP ; then
           echo "ok ${INDEX} - ${FILE}" >> "${TMPFILE}"
         fi
       fi
@@ -584,8 +584,8 @@ LintAnsibleFiles()
     #################################
     # Generate report in TAP format #
     #################################
-    if [ -n "${OUTPUT_FORMAT}" ] && [ ${INDEX} -gt 0 ] ; then
-      printf "TAP version 13\n1..%s" "${INDEX}" > "${REPORT_OUTPUT_FILE}"
+    if IsTAP && [ ${INDEX} -gt 0 ] ; then
+      printf "TAP version 13\n1..%s\n" "${INDEX}" > "${REPORT_OUTPUT_FILE}"
       cat "${TMPFILE}" >> "${REPORT_OUTPUT_FILE}"
     fi
 
@@ -1831,7 +1831,7 @@ LintCodebase()
     ####################################
     # Prepare context if OUTPUT_FORMAT #
     ####################################
-    if [ -n "${OUTPUT_FORMAT}" ] ; then
+    if IsTAP ; then
       TMPFILE=$(mktemp -q "/tmp/super-linter-${FILE_TYPE}.XXXXXX")
       INDEX=0
       mkdir -p "${REPORT_OUTPUT_FOLDER}"
@@ -1911,7 +1911,7 @@ LintCodebase()
         #######################################################
         # Store the linting as a temporary file in TAP format #
         #######################################################
-        if [ -n "${OUTPUT_FORMAT}" ] ; then
+        if IsTAP ; then
           echo "nok ok ${INDEX} - ${FILE}" >> "${TMPFILE}"
           printf "  ---\n  message:[%s]\n  ..." "$LINT_CMD" >> "${TMPFILE}"
         fi
@@ -1925,7 +1925,7 @@ LintCodebase()
         #######################################################
         # Store the linting as a temporary file in TAP format #
         #######################################################
-        if [ -n "${OUTPUT_FORMAT}" ] ; then
+        if IsTAP ; then
           echo "ok ${INDEX} - ${FILE}" >> "${TMPFILE}"
         fi
       fi
@@ -1934,8 +1934,8 @@ LintCodebase()
     #################################
     # Generate report in TAP format #
     #################################
-    if [ -n "${OUTPUT_FORMAT}" ] && [ ${INDEX} -gt 0 ] ; then
-      printf "TAP version 13\n1..%s" "${INDEX}" > "${REPORT_OUTPUT_FILE}"
+    if IsTAP && [ ${INDEX} -gt 0 ] ; then
+      printf "TAP version 13\n1..%s\n" "${INDEX}" > "${REPORT_OUTPUT_FILE}"
       cat "${TMPFILE}" >> "${REPORT_OUTPUT_FILE}"
     fi
   fi
@@ -2286,6 +2286,16 @@ RunTestCases()
   Footer
 }
 ################################################################################
+#### Function IsTap ############################################################
+IsTAP()
+{
+  if [ "${OUTPUT_FORMAT}" == "tap" ] ; then
+    return 0
+  else
+    return 1
+  fi
+}
+################################################################################
 ############################### MAIN ###########################################
 ################################################################################