Fix test coverage check to handle Go subtests correctly #12654
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
- Modified extract-executed-tests.sh to strip subtest names (after /) - This ensures only top-level test names are extracted - Fixes CI failure where 3406 subtests were incorrectly counted as missing tests Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Fix issues in test list
Fix test coverage check to handle Go subtests correctly
Jan 30, 2026
pelikhan
approved these changes
Jan 30, 2026
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The test coverage verification was reporting 3406 "missing" tests (44% coverage) because
go test -jsonemits both parent tests (TestFoo) and subtests (TestFoo/subtest_name), while the test definition scanner only finds parent test functions.Changes
scripts/extract-executed-tests.shto strip subtest suffixes (everything after/)list-all-tests.shThis eliminates false negatives in the coverage comparison, allowing CI to correctly validate that all defined tests are executed.
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.