yihua opened a new pull request, #19130:
URL: https://github.com/apache/hudi/pull/19130

   ### Describe the issue this Pull Request addresses
   
   The `integration-tests` GitHub Actions job runs the Hudi CLI and 
`hudi-integ-test` integration tests, but reported no code coverage to Codecov. 
The `integration-tests` Maven profile wired no JaCoCo agent (unlike the 
`unit-tests` / `functional-tests*` profiles), so no `.exec` data was produced, 
and the job had no coverage upload step. Because `hudi-cli` is excluded from 
the unit/functional test jobs, its production code was exercised only by these 
integration tests and its coverage was never reported.
   
   ### Summary and Changelog
   
   Report integration-test coverage to Codecov, closing the `hudi-cli` blind 
spot.
   
   - `pom.xml`: add a JaCoCo `prepare-agent` execution to the 
`integration-tests` profile, writing exec data under `target/jacoco-agent/` so 
the existing `scripts/jacoco/generate_merged_coverage_report.sh` picks it up. 
The agent is appended to the `argLine` property, which both failsafe (IT) and 
surefire (integ-test UT) already consume.
   - `.github/workflows/bot.yml`: enable JaCoCo on the `UT integ-test` and `IT` 
steps (`-Djacoco.skip=false`), and add the standard "Generate merged coverage 
report" + "Upload coverage to Codecov" steps (flag `integration-tests`), 
mirroring the other test jobs.
   
   Scope is limited to the `integration-tests` job; the Flink integration-test 
jobs are intentionally left out.
   
   ### Impact
   
   CI only; no production code or public API change. The `integration-tests` 
job now uploads coverage under the `integration-tests` flag.
   
   ### Risk Level
   
   low
   
   CI-only change. It reuses the same JaCoCo agent plus merge/upload path 
already proven by the unit/functional test jobs. Failsafe/surefire memory 
settings are unchanged, since the agent is appended to the existing `argLine` 
property rather than replacing it.
   
   ### Documentation Update
   
   none
   
   ### Contributor's checklist
   
   - [x] Read through [contributor's 
guide](https://hudi.apache.org/contribute/how-to-contribute)
   - [x] Enough context is provided in the sections above
   - [ ] Adequate tests were added if applicable
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to