fix: avoid duplicated content in summary (#5939)

- Remove the previous summary file is present to avoid showing stale
  contents from old summaries.
- Extract the logic to write summary heading and lines to dedicated
  functions in output.sh to make it easier adding new formats in the
  future.

Author: ferrarimarco

Makefile

@@ -238,6 +238,7 @@ lint-codebase: ## Lint the entire codebase
 		-e FILTER_REGEX_EXCLUDE=".*(/test/linters/|CHANGELOG.md).*" \
 		-e GITLEAKS_CONFIG_FILE=".gitleaks-ignore-tests.toml" \
 		-e RENOVATE_SHAREABLE_CONFIG_PRESET_FILE_NAMES="default.json,hoge.json" \
+		-e SAVE_SUPER_LINTER_SUMMARY=true \
 		-e VALIDATE_ALL_CODEBASE=true \
 		-v "$(CURDIR):/tmp/lint" \
 		$(SUPER_LINTER_TEST_CONTAINER_URL)
@@ -280,7 +281,7 @@ lint-subset-files-enable-expensive-io-checks: ## Lint a small subset of files in
 		$(SUPER_LINTER_TEST_CONTAINER_URL)
 
 .PHONY: test-lib
-test-lib: test-build-file-list test-detect-files test-github-event test-setup-ssh test-validation ## Test super-linter
+test-lib: test-build-file-list test-detect-files test-github-event test-setup-ssh test-validation test-output ## Test super-linter
 
 .PHONY: test-build-file-list
 test-build-file-list: ## Test buildFileList
@@ -323,6 +324,14 @@ test-validation: ## Test validation
 		--entrypoint /tmp/lint/test/lib/validationTest.sh \
 		$(SUPER_LINTER_TEST_CONTAINER_URL)
 
+.PHONY: test-output
+test-output: ## Test output
+	docker run \
+		-v "$(CURDIR):/tmp/lint" \
+		-w /tmp/lint \
+		--entrypoint /tmp/lint/test/lib/outputTest.sh \
+		$(SUPER_LINTER_TEST_CONTAINER_URL)
+
 # Run this test against a small directory because we're only interested in
 # loading default configuration files. The directory that we run super-linter
 # against should not be .github because that includes default linter rules.

README.md

@@ -476,6 +476,8 @@ The summary is in Markdown format. Super-linter supports the following formats:
 
 - Table (default)
 
+The summary output of previous Super-linter runs is not preserved.
+
 ### Super-linter outputs
 
 If you set `SAVE_SUPER_LINTER_OUTPUT` to `true`, Super-linter saves its output

lib/functions/output.sh

@@ -0,0 +1,40 @@
+#!/usr/bin/env bash
+
+WriteSummaryHeader() {
+  local SUPER_LINTER_SUMMARY_OUTPUT_PATH="${1}"
+
+  {
+    echo "# Super-linter summary"
+    echo ""
+    echo "| Language               | Validation result |"
+    echo "| -----------------------|-------------------|"
+  } >>"${SUPER_LINTER_SUMMARY_OUTPUT_PATH}"
+}
+
+WriteSummaryLineSuccess() {
+  local SUPER_LINTER_SUMMARY_OUTPUT_PATH="${1}"
+  local LANGUAGE_NAME="${2}"
+  echo "| ${LANGUAGE_NAME} | Pass ✅ |" >>"${SUPER_LINTER_SUMMARY_OUTPUT_PATH}"
+}
+
+WriteSummaryLineFailure() {
+  local SUPER_LINTER_SUMMARY_OUTPUT_PATH="${1}"
+  local LANGUAGE_NAME="${2}"
+  echo "| ${LANGUAGE_NAME} | Fail ❌ |" >>"${SUPER_LINTER_SUMMARY_OUTPUT_PATH}"
+}
+
+WriteSummaryFooterSuccess() {
+  local SUPER_LINTER_SUMMARY_OUTPUT_PATH="${1}"
+  {
+    echo ""
+    echo "All files and directories linted successfully"
+  } >>"${SUPER_LINTER_SUMMARY_OUTPUT_PATH}"
+}
+
+WriteSummaryFooterFailure() {
+  local SUPER_LINTER_SUMMARY_OUTPUT_PATH="${1}"
+  {
+    echo ""
+    echo "Super-linter detected linting errors"
+  } >>"${SUPER_LINTER_SUMMARY_OUTPUT_PATH}"
+}

lib/linter.sh

@@ -31,6 +31,8 @@ source /action/lib/functions/setupSSH.sh # Source the function script(s)
 source /action/lib/functions/githubEvent.sh
 # shellcheck source=/dev/null
 source /action/lib/functions/githubDomain.sh
+# shellcheck source=/dev/null
+source /action/lib/functions/output.sh
 
 if ! ValidateGitHubUrls; then
   fatal "GitHub URLs failed validation"
@@ -565,12 +567,7 @@ Footer() {
 
   if [[ "${SAVE_SUPER_LINTER_SUMMARY}" == "true" ]]; then
     debug "Saving Super-linter summary to ${SUPER_LINTER_SUMMARY_OUTPUT_PATH}"
-    {
-      echo "# Super-linter summary"
-      echo ""
-      echo "| Language               | Validation result |"
-      echo "| -----------------------|-------------------|"
-    } >>"${SUPER_LINTER_SUMMARY_OUTPUT_PATH}"
+    WriteSummaryHeader "${SUPER_LINTER_SUMMARY_OUTPUT_PATH}"
   fi
 
   for LANGUAGE in "${LANGUAGE_ARRAY[@]}"; do
@@ -591,7 +588,7 @@ Footer() {
         error "Errors found in ${LANGUAGE}"
 
         if [[ "${SAVE_SUPER_LINTER_SUMMARY}" == "true" ]]; then
-          echo "| ${LANGUAGE} | Fail ❌ |" >>"${SUPER_LINTER_SUMMARY_OUTPUT_PATH}"
+          WriteSummaryLineFailure "${SUPER_LINTER_SUMMARY_OUTPUT_PATH}" "${LANGUAGE}"
         fi
 
         # Print output as error in case users disabled the INFO level so they
@@ -619,7 +616,7 @@ Footer() {
       elif [[ ${ERROR_COUNTER} -eq 0 ]]; then
         notice "Successfully linted ${LANGUAGE}"
         if [[ "${SAVE_SUPER_LINTER_SUMMARY}" == "true" ]]; then
-          echo "| ${LANGUAGE} | Pass ✅ |" >>"${SUPER_LINTER_SUMMARY_OUTPUT_PATH}"
+          WriteSummaryLineSuccess "${SUPER_LINTER_SUMMARY_OUTPUT_PATH}" "${LANGUAGE}"
         fi
         CallStatusAPI "${LANGUAGE}" "success"
         ANY_LINTER_SUCCESS="true"
@@ -641,18 +638,12 @@ Footer() {
   if [[ ${SUPER_LINTER_EXIT_CODE} -eq 0 ]]; then
     notice "All files and directories linted successfully"
     if [[ "${SAVE_SUPER_LINTER_SUMMARY}" == "true" ]]; then
-      {
-        echo ""
-        echo "All files and directories linted successfully"
-      } >>"${SUPER_LINTER_SUMMARY_OUTPUT_PATH}"
+      WriteSummaryFooterSuccess "${SUPER_LINTER_SUMMARY_OUTPUT_PATH}"
     fi
   else
     error "Super-linter detected linting errors"
     if [[ "${SAVE_SUPER_LINTER_SUMMARY}" == "true" ]]; then
-      {
-        echo ""
-        echo "Super-linter detected linting errors"
-      } >>"${SUPER_LINTER_SUMMARY_OUTPUT_PATH}"
+      WriteSummaryFooterFailure "${SUPER_LINTER_SUMMARY_OUTPUT_PATH}"
     fi
   fi
 
@@ -820,6 +811,9 @@ if [[ "${SAVE_SUPER_LINTER_OUTPUT}" = "true" ]] ||
 fi
 
 if [[ "${SAVE_SUPER_LINTER_SUMMARY}" == "true" ]]; then
+  debug "Remove eventual ${SUPER_LINTER_SUMMARY_OUTPUT_PATH} leftover"
+  rm -f "${SUPER_LINTER_SUMMARY_OUTPUT_PATH}"
+
   debug "Ensuring that ${SUPER_LINTER_SUMMARY_OUTPUT_PATH} exists."
   if ! touch "${SUPER_LINTER_SUMMARY_OUTPUT_PATH}"; then
     fatal "Cannot create Super-linter summary file: ${SUPER_LINTER_SUMMARY_OUTPUT_PATH}"

test/data/super-linter-summary/markdown/table/expected-summary-footer-failure.md

@@ -0,0 +1,3 @@
+<!-- markdownlint-disable -->
+
+Super-linter detected linting errors

test/data/super-linter-summary/markdown/table/expected-summary-footer-success.md

@@ -0,0 +1,3 @@
+<!-- markdownlint-disable -->
+
+All files and directories linted successfully

test/data/super-linter-summary/markdown/table/expected-summary-heading.md

@@ -0,0 +1,4 @@
+# Super-linter summary
+
+| Language               | Validation result |
+| -----------------------|-------------------|

test/data/super-linter-summary/markdown/table/expected-summary-line-failure.md

@@ -0,0 +1,2 @@
+<!-- markdownlint-disable -->
+| Test Language | Fail ❌ |

test/data/super-linter-summary/markdown/table/expected-summary-line-success.md

@@ -0,0 +1,2 @@
+<!-- markdownlint-disable -->
+| Test Language | Pass ✅ |