From 794e8c78eaf98b86f23c71a9aa42a70adbb5088c Mon Sep 17 00:00:00 2001 From: David Bickford Date: Mon, 23 Mar 2026 14:03:20 -0400 Subject: [PATCH 1/4] Fix test step summary to match non-verbose go test output --- go/test/action.yml | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/go/test/action.yml b/go/test/action.yml index 2a5dd3c..9e13e63 100644 --- a/go/test/action.yml +++ b/go/test/action.yml @@ -30,20 +30,20 @@ runs: run: | echo "## Go Test Results" >> $GITHUB_STEP_SUMMARY if [ -f test-report.out ]; then - PASS=$(grep -c '^--- PASS:' test-report.out || true) - FAIL=$(grep -c '^--- FAIL:' test-report.out || true) - SKIP=$(grep -c '^--- SKIP:' test-report.out || true) - echo "| Status | Count |" >> $GITHUB_STEP_SUMMARY - echo "|--------|-------|" >> $GITHUB_STEP_SUMMARY + PASS=$(grep -c '^ok' test-report.out || true) + FAIL=$(grep -c '^FAIL' test-report.out || true) + SKIP=$(grep -c '\[no test files\]' test-report.out || true) + echo "| Status | Packages |" >> $GITHUB_STEP_SUMMARY + echo "|--------|----------|" >> $GITHUB_STEP_SUMMARY echo "| Passed | ${PASS} |" >> $GITHUB_STEP_SUMMARY echo "| Failed | ${FAIL} |" >> $GITHUB_STEP_SUMMARY - echo "| Skipped | ${SKIP} |" >> $GITHUB_STEP_SUMMARY + echo "| No Tests | ${SKIP} |" >> $GITHUB_STEP_SUMMARY if [ "$FAIL" -gt 0 ]; then echo "" >> $GITHUB_STEP_SUMMARY - echo "
Failed Tests" >> $GITHUB_STEP_SUMMARY + echo "
Failed Packages" >> $GITHUB_STEP_SUMMARY echo "" >> $GITHUB_STEP_SUMMARY echo '```' >> $GITHUB_STEP_SUMMARY - grep '^--- FAIL:' test-report.out >> $GITHUB_STEP_SUMMARY + grep '^FAIL' test-report.out >> $GITHUB_STEP_SUMMARY echo '```' >> $GITHUB_STEP_SUMMARY echo "
" >> $GITHUB_STEP_SUMMARY fi From 7a74fc0b137b125c125040d39caa40acf69e2f9c Mon Sep 17 00:00:00 2001 From: David Bickford Date: Mon, 23 Mar 2026 15:57:45 -0400 Subject: [PATCH 2/4] Tighten grep patterns to avoid double-counting failures --- go/test/action.yml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/go/test/action.yml b/go/test/action.yml index 9e13e63..11768dd 100644 --- a/go/test/action.yml +++ b/go/test/action.yml @@ -30,8 +30,8 @@ runs: run: | echo "## Go Test Results" >> $GITHUB_STEP_SUMMARY if [ -f test-report.out ]; then - PASS=$(grep -c '^ok' test-report.out || true) - FAIL=$(grep -c '^FAIL' test-report.out || true) + PASS=$(grep -c '^ok ' test-report.out || true) + FAIL=$(grep -cP '^FAIL\t' test-report.out || true) SKIP=$(grep -c '\[no test files\]' test-report.out || true) echo "| Status | Packages |" >> $GITHUB_STEP_SUMMARY echo "|--------|----------|" >> $GITHUB_STEP_SUMMARY @@ -43,7 +43,7 @@ runs: echo "
Failed Packages" >> $GITHUB_STEP_SUMMARY echo "" >> $GITHUB_STEP_SUMMARY echo '```' >> $GITHUB_STEP_SUMMARY - grep '^FAIL' test-report.out >> $GITHUB_STEP_SUMMARY + grep -P '^FAIL\t' test-report.out >> $GITHUB_STEP_SUMMARY echo '```' >> $GITHUB_STEP_SUMMARY echo "
" >> $GITHUB_STEP_SUMMARY fi From 433e804004ee79222cc3a168a17c7f7bd0a017a1 Mon Sep 17 00:00:00 2001 From: David Bickford Date: Mon, 23 Mar 2026 16:05:12 -0400 Subject: [PATCH 3/4] Use POSIX character class instead of grep -P for portability --- go/test/action.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/go/test/action.yml b/go/test/action.yml index 11768dd..7ea994e 100644 --- a/go/test/action.yml +++ b/go/test/action.yml @@ -31,7 +31,7 @@ runs: echo "## Go Test Results" >> $GITHUB_STEP_SUMMARY if [ -f test-report.out ]; then PASS=$(grep -c '^ok ' test-report.out || true) - FAIL=$(grep -cP '^FAIL\t' test-report.out || true) + FAIL=$(grep -c '^FAIL[[:space:]]' test-report.out || true) SKIP=$(grep -c '\[no test files\]' test-report.out || true) echo "| Status | Packages |" >> $GITHUB_STEP_SUMMARY echo "|--------|----------|" >> $GITHUB_STEP_SUMMARY @@ -43,7 +43,7 @@ runs: echo "
Failed Packages" >> $GITHUB_STEP_SUMMARY echo "" >> $GITHUB_STEP_SUMMARY echo '```' >> $GITHUB_STEP_SUMMARY - grep -P '^FAIL\t' test-report.out >> $GITHUB_STEP_SUMMARY + grep '^FAIL[[:space:]]' test-report.out >> $GITHUB_STEP_SUMMARY echo '```' >> $GITHUB_STEP_SUMMARY echo "
" >> $GITHUB_STEP_SUMMARY fi From 8a49f1cd5d07d788b62bbf76c3161da64fda134a Mon Sep 17 00:00:00 2001 From: David Bickford Date: Mon, 23 Mar 2026 16:14:35 -0400 Subject: [PATCH 4/4] Add -v to go test for individual test names in summary --- go/test/action.yml | 16 ++++++++-------- go/test/test.sh | 2 +- 2 files changed, 9 insertions(+), 9 deletions(-) diff --git a/go/test/action.yml b/go/test/action.yml index 7ea994e..2a5dd3c 100644 --- a/go/test/action.yml +++ b/go/test/action.yml @@ -30,20 +30,20 @@ runs: run: | echo "## Go Test Results" >> $GITHUB_STEP_SUMMARY if [ -f test-report.out ]; then - PASS=$(grep -c '^ok ' test-report.out || true) - FAIL=$(grep -c '^FAIL[[:space:]]' test-report.out || true) - SKIP=$(grep -c '\[no test files\]' test-report.out || true) - echo "| Status | Packages |" >> $GITHUB_STEP_SUMMARY - echo "|--------|----------|" >> $GITHUB_STEP_SUMMARY + PASS=$(grep -c '^--- PASS:' test-report.out || true) + FAIL=$(grep -c '^--- FAIL:' test-report.out || true) + SKIP=$(grep -c '^--- SKIP:' test-report.out || true) + echo "| Status | Count |" >> $GITHUB_STEP_SUMMARY + echo "|--------|-------|" >> $GITHUB_STEP_SUMMARY echo "| Passed | ${PASS} |" >> $GITHUB_STEP_SUMMARY echo "| Failed | ${FAIL} |" >> $GITHUB_STEP_SUMMARY - echo "| No Tests | ${SKIP} |" >> $GITHUB_STEP_SUMMARY + echo "| Skipped | ${SKIP} |" >> $GITHUB_STEP_SUMMARY if [ "$FAIL" -gt 0 ]; then echo "" >> $GITHUB_STEP_SUMMARY - echo "
Failed Packages" >> $GITHUB_STEP_SUMMARY + echo "
Failed Tests" >> $GITHUB_STEP_SUMMARY echo "" >> $GITHUB_STEP_SUMMARY echo '```' >> $GITHUB_STEP_SUMMARY - grep '^FAIL[[:space:]]' test-report.out >> $GITHUB_STEP_SUMMARY + grep '^--- FAIL:' test-report.out >> $GITHUB_STEP_SUMMARY echo '```' >> $GITHUB_STEP_SUMMARY echo "
" >> $GITHUB_STEP_SUMMARY fi diff --git a/go/test/test.sh b/go/test/test.sh index baff573..cd0fde1 100755 --- a/go/test/test.sh +++ b/go/test/test.sh @@ -5,7 +5,7 @@ TAGS=${1:-} TIMEOUT=${2:-} PARALLEL=${3:-} -cmd="go test ./... -coverprofile=coverage.out" +cmd="go test -v ./... -coverprofile=coverage.out" # find non-ignored subdirectories file_list=()