Add comprehensive code coverage and test reporting to CI/CD #423
+256
−53
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
✅ Deploy Preview for angry-raman-7c44f6 canceled.
|
Add comprehensive testing infrastructure with code coverage tracking: Just Recipes: - test-coverage: Run tests with coverage, merge profiles, show summary - test-junit: Run tests with JUnit XML output using gotestsum CI/CD Enhancements: - Generate coverage on all test runs with atomic coverage mode - Upload coverage artifacts and test results - Publish test summaries using test-summary/action v2.4 - Add coverage-report job for PR comments with fgrosse/go-coverage-report - Continue uploading to Codecov for historical tracking Configuration: - Add coverage/, coverage.out, test-results/ to .gitignore - Update test job to use test-junit recipe for comprehensive reporting Documentation: - Add Testing and Code Coverage section to morphir-go-preview.md - Document local testing commands and coverage workflows - Explain CI/CD integration and PR coverage comments - Include emoji indicators for coverage changes Benefits: - Track code coverage trends over time - See coverage impact directly in PR reviews - Better visibility into test results with GitHub Actions summaries - Identify untested code paths - Automated test result reports Relates to morphir-rnt
The test-junit recipe was failing because the coverage directory didn't exist when tests tried to write coverage files. Now we create both test-results and coverage directories upfront. Also added coverage merging step to test-junit recipe to generate the final coverage.out file that CI expects.
The test-junit and test-coverage recipes were failing because they used relative paths (../coverage/) which don't work correctly when cd'ing into subdirectories. When in cmd/morphir, ../coverage/ points to morphir/cmd/coverage/ instead of morphir/coverage/. Now we capture the repo root path upfront and use absolute paths for all coverage and test result file locations.
DamianReeves
force-pushed
the
feat/code-coverage-reporting
branch
from
59f68d7 to
08fa21b
Compare
The coverage-report job fails when trying to download baseline coverage from the main branch if it doesn't exist yet (which is expected for the first time adding coverage support). Adding continue-on-error allows the job to proceed gracefully without failing the entire CI run. Once this PR is merged and main has coverage artifacts, future PRs will be able to compare against the baseline.
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.
Summary
Implements comprehensive testing infrastructure with code coverage tracking and reporting.
Changes
Just Recipes
test-coverage: Run tests with coverage, merge profiles, show summarytest-junit: Run tests with JUnit XML output using gotestsumCI/CD Enhancements
test-summary/[email protected]coverage-reportjob for automated PR comments withfgrosse/[email protected]Configuration
coverage/,coverage.out,test-results/to.gitignoretest-junitrecipe for comprehensive reportingDocumentation
morphir-go-preview.mdBenefits
Test Plan
test-coverage,test-junit)Related Issues
Closes morphir-rnt
🤖 Generated with Claude Code