Display summary of warnings (#932)

zachborboa · web-flow · commit 41cc6997987b · 2025-03-08T13:44:18.000-05:00
diff --git a/tests/display_warnings.inc.sh b/tests/display_warnings.inc.sh
@@ -0,0 +1,11 @@
+warning_count="${#warnings[@]}"
+if [[ "${warning_count}" -ge 1 ]]; then
+    echo -e "\nWarnings found: ${warning_count}"
+
+    iter=0
+    for value in "${warnings[@]}"; do
+        ((iter++))
+        echo -e "\nWarning ${iter} of ${warning_count}:"
+        echo "&#9888;&#65039; ${value}" | perl -pe 's/^(.*)$/\t\1/'
+    done
+fi
diff --git a/tests/run.sh b/tests/run.sh
@@ -2,6 +2,7 @@
 
 set -x
 
+warnings=()
 errors=()
 
 if [[ "${CI}" == "true" ]]; then
@@ -55,6 +56,7 @@ source "run_static_analysis_check_psalm.sh"
 
 set +x
 
+source "display_warnings.inc.sh"
 source "display_errors.inc.sh"
 
 if [[ "${CI_PHP_FUTURE_RELEASE}" != "true" ]]; then
diff --git a/tests/run_coding_standards_check.sh b/tests/run_coding_standards_check.sh
@@ -132,6 +132,7 @@ if [[ $(echo "${CI_PHP_VERSION} < 8.4" | bc -l) -eq 1 ]]; then
     fi
 else
     echo "&#9888;&#65039; Skipped running PHP-CS-Fixer coding standards check"
+    warnings+=("Skipped running PHP-CS-Fixer coding standards check")
 fi
 
 popd
diff --git a/tests/run_static_analysis_check_phpstan.sh b/tests/run_static_analysis_check_phpstan.sh
@@ -27,6 +27,7 @@ if [[ $(echo "${CI_PHP_VERSION} >= 7.4" | bc -l) -eq 1 ]]; then
     fi
 else
     echo "&#9888;&#65039; Skipped running phpstan check"
+    warnings+=("Skipped running phpstan check")
 fi
 
 popd
diff --git a/tests/run_static_analysis_check_psalm.sh b/tests/run_static_analysis_check_psalm.sh
@@ -12,6 +12,7 @@ set -x
 
 if [[ ! -f "vendor/bin/psalm" ]]; then
     echo "&#9888;&#65039; Skipped running psalm static analysis check"
+    warnings+=("Skipped running psalm static analysis check")
 else
     vendor/bin/psalm --version
 

-Original file line number
+Diff line change
     fi
 else
     echo "⚠️ Skipped running PHP-CS-Fixer coding standards check"
 +    warnings+=("Skipped running PHP-CS-Fixer coding standards check")
 fi
 popd