Skip to content

Fix test step summary with verbose go test output#185

Merged
dcbickfo merged 4 commits intomainfrom
fix/test-step-summary
Mar 24, 2026
Merged

Fix test step summary with verbose go test output#185
dcbickfo merged 4 commits intomainfrom
fix/test-step-summary

Conversation

@dcbickfo
Copy link
Copy Markdown
Contributor

@dcbickfo dcbickfo commented Mar 23, 2026

  • Added -v flag to go test in test.sh so CI gets individual test names
  • Step summary now correctly counts individual tests via ^--- PASS:, ^--- FAIL:, ^--- SKIP: patterns
  • Failed tests are listed by name in a collapsible details section
  • Previous summary was broken: grepped for verbose patterns but test.sh ran without -v, so all counts showed 0

Tested on inventory-api: https://github.com/wishabi/inventory-api/actions/runs/23457990255

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Go composite action’s GitHub step summary to correctly parse default (non--v) go test output, reporting results at the package level.

Changes:

  • Switch summary parsing from verbose test-case markers (--- PASS/FAIL/SKIP) to package-level go test lines (ok, FAIL, [no test files]).
  • Rename the summary table columns/labels to reflect package-level counts.
  • Update the failure details section to list failed packages rather than failed tests.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@dcbickfo dcbickfo changed the title Fix test step summary to parse non-verbose go test output Fix test step summary with verbose go test output Mar 23, 2026
@dcbickfo dcbickfo requested a review from Copilot March 23, 2026 20:20
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@dcbickfo dcbickfo requested a review from chriskarlin March 23, 2026 20:25
@dcbickfo dcbickfo merged commit 58b4466 into main Mar 24, 2026
6 checks passed
@dcbickfo dcbickfo deleted the fix/test-step-summary branch March 24, 2026 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants