Show individual test results in functional test logs#11917
Show individual test results in functional test logs#11917brooke-hamilton wants to merge 1 commit into
Conversation
Signed-off-by: Brooke Hamilton <45323234+brooke-hamilton@users.noreply.github.com>
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned FilesNone |
Radius functional test overviewClick here to see the test run details
Test Status⌛ Building Radius and pushing container images for functional tests... |
There was a problem hiding this comment.
Pull request overview
This PR updates the functional test workflows so CI logs show per-test pass/fail status and elapsed time from gotestsum, making slow functional tests easier to identify.
Changes:
- Adds
--format testnameto functional testGOTESTSUM_OPTS. - Adds
--format-hide-empty-pkgto suppress empty package noise.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
.github/workflows/functional-test-cloud.yaml |
Updates cloud functional test gotestsum output formatting. |
.github/workflows/functional-test-noncloud.yaml |
Updates non-cloud functional test gotestsum output formatting. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #11917 +/- ##
==========================================
- Coverage 51.72% 51.71% -0.02%
==========================================
Files 726 726
Lines 45608 45608
==========================================
- Hits 23593 23587 -6
- Misses 19792 19795 +3
- Partials 2223 2226 +3 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Closing PR. This change caused the non-cloud tests to hang. |
Purpose
Show pass/fail status and elapsed time for each individual functional test in CI logs, instead of one consolidated line per package. This will help us find and optimize tests with lengthy execution times.
Changes
Adds
--format testname --format-hide-empty-pkgtoGOTESTSUM_OPTSin both functional test workflows:.github/workflows/functional-test-cloud.yaml.github/workflows/functional-test-noncloud.yamlBefore
gotestsumdefaulted topkgnameformat, producing one line per package:Individual test names and timings were hidden.
After
testnameformat prints a line per test as it completes (with PASS/FAIL/SKIP and elapsed time).--format-hide-empty-pkgsuppresses noise from packages that contain no tests.References
gotestsum --help→--format-hide-empty-pkg: do not print empty packages in compact formats