Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 31, 2026

The unit test coverage step outputs verbose test results to both stdout and a JSON file using tee, causing GitHub Actions stdout buffer overflow.

Changes

  • Replace | tee test-result-unit.json with > test-result-unit.json to redirect output directly to file without stdout output

Before:

go test -v -parallel=8 -timeout=3m -run='^Test' -tags '!integration' -coverprofile=coverage.out -json ./... | tee test-result-unit.json

After:

go test -v -parallel=8 -timeout=3m -run='^Test' -tags '!integration' -coverprofile=coverage.out -json ./... > test-result-unit.json

Test results JSON and coverage HTML artifacts continue to be generated and uploaded as before.

Original prompt

Update run test with coverage step in ci.yml to pipe results to a file and not show the output to stdout. The output is too large and overflows the GitHub actions stdout buffer.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 2 commits January 31, 2026 15:04
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Replace tee with direct file redirect (>) to pipe test results to file without displaying to stdout. This prevents GitHub Actions stdout buffer overflow while maintaining the same behavior for coverage report generation and artifact uploads.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Update run test with coverage step in ci.yml Fix stdout buffer overflow in unit test coverage step Jan 31, 2026
Copilot AI requested a review from pelikhan January 31, 2026 15:06
@pelikhan pelikhan marked this pull request as ready for review January 31, 2026 15:07
@pelikhan pelikhan merged commit ff02eb6 into main Jan 31, 2026
121 checks passed
@pelikhan pelikhan deleted the copilot/update-ci-yml-test-coverage branch January 31, 2026 15:09
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.

2 participants