XXX

tools/gotestfmt

@@ -31,7 +31,8 @@ done
 
 LOGFILE="${LOGFILE:-$(mktemp -t gotestfmt.XXXXXX.log)}"
 echo >&2 "Logging to $LOGFILE"
-STDOUT_FILE=${LOGFILE}.raw
+STDERR_FILE=$(mktemp -t gotestfmt.XXXXXX.stderr)
+STDOUT_FILE=$(mktemp -t gotestfmt.XXXXXX.stdout)
 
 strip_empty_coverage_lines() {
     # This strips empty coverage lines from the output of `go test -cover -json`,
@@ -51,4 +52,42 @@ copy_output() {
     tee "$STDOUT_FILE"
 }
 
-copy_output | strip_empty_coverage_lines | copy_to_logfile | gotestfmt --hide all <&0
+copy_output | strip_empty_coverage_lines | copy_to_logfile | gotestfmt --hide all <&0 2> "$STDERR_FILE" || exit_code=$?
+
+if [ -n "${GITHUB_WORKFLOW:-}" ]; then
+    echo "::group::stderr"
+    cat "$STDERR_FILE"
+    echo "::endgroup::"
+    echo "::group::stdout"
+    cat "$STDOUT_FILE"
+    echo "::endgroup::"
+else
+    echo "##### Beginning of stderr #####"
+    cat "$STDERR_FILE"
+    echo "##### End of stderr #####"
+    echo "##### Beginning of stdout #####"
+    cat "$STDOUT_FILE"
+    echo "##### End of stdout #####"
+fi
+
+## If an error message was printed to stderr, that's an indicator that
+## parsing the JSON output failed, so we print the raw output for debugging.
+#if [ -n "${exit_code:-}" ] && [ -s "$STDERR_FILE" ]; then
+#    if [ -n "${GITHUB_WORKFLOW:-}" ]; then
+#        echo "::group::stderr"
+#        cat "$STDERR_FILE"
+#        echo "::endgroup::"
+#        echo "::group::stdout"
+#        cat "$STDOUT_FILE"
+#        echo "::endgroup::"
+#    else
+#        echo "##### Beginning of stderr #####"
+#        cat "$STDERR_FILE"
+#        echo "##### End of stderr #####"
+#        echo "##### Beginning of stdout #####"
+#        cat "$STDOUT_FILE"
+#        echo "##### End of stdout #####"
+#    fi
+#fi
+
+exit "${exit_code:-0}"