ci: add JaCoCo code coverage with Codecov integration by ex172000 · Pull Request #2623 · apache/iggy

ex172000 · 2026-01-27T07:40:17Z

Add code coverage reporting for Java modules using JaCoCo and integrate with Codecov for PR coverage reports.

Changes:

Add JaCoCo plugin (v0.8.14) to common conventions for all modules
Create aggregated coverage report task combining all module coverage
Update GitHub Actions to generate coverage reports and upload to Codecov

hubcio · 2026-01-27T19:36:47Z

@ex172000 can you modify single file java file under this PR so we can see how does the codecov comment looks like?

ex172000 · 2026-01-27T19:40:07Z

@ex172000 can you modify single file java file under this PR so we can see how does the codecov comment looks like?

Yes definitely. Was trying to fix issues first. Will add it in the next commit(s).

.github/actions/java-gradle/pre-merge/action.yml

mmodzelewski · 2026-02-05T14:26:05Z

Hey @ex172000, we now have the codecov integration setup for the rust code (done by @hubcio), so adding the java part should be much easier now :) Let us know if you want to continue with this PR.

ex172000 · 2026-02-05T20:47:31Z

Hey @ex172000, we now have the codecov integration setup for the rust code (done by @hubcio), so adding the java part should be much easier now :) Let us know if you want to continue with this PR.

Hi @mmodzelewski @hubcio , sorry it took a while. Something came up from work. I will return to fix the Java part tomorrow afternoon Pacific time. Will update to OPEN state as soon as it's ready for review.

Add comprehensive code coverage reporting for Java modules using JaCoCo and integrate with Codecov for PR coverage reports. Changes: - Add JaCoCo plugin (v0.8.14) to common conventions for all modules - Create aggregated coverage report task combining all module coverage - Update GitHub Actions to generate coverage reports and upload to Codecov - Add .codecov.yml configuration with 70% threshold for new code - Configure XML reports for Codecov and HTML for local viewing Coverage reports will automatically appear as PR comments showing: - Overall coverage percentage and trends - Coverage diff for changed files - File-by-file coverage breakdown Coverage can be viewed locally with: ./gradlew test jacocoAggregatedReport open build/reports/jacoco/aggregate/html/index.html

Move codecov upload from composite action to _test.yml where secrets are accessible, upgrade to codecov-action@v5, add Rust coverage flag, and set require_changes to suppress comments on unrelated PRs. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

codecov · 2026-02-07T06:34:23Z

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 67.87%. Comparing base (0b4cc90) to head (dcbe6c8).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #2623      +/-   ##
============================================
- Coverage     68.85%   67.87%   -0.98%     
- Complexity        0      637     +637     
============================================
  Files           575      734     +159     
  Lines         55672    59374    +3702     
  Branches      55672    55787     +115     
============================================
+ Hits          38334    40303    +1969     
- Misses        15467    17113    +1646     
- Partials       1871     1958      +87

Flag	Coverage Δ
java	`51.41% <100.00%> (?)`
rust	`69.02% <ø> (+0.16%)`	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
...in/java/org/apache/iggy/serde/BytesSerializer.java	`97.03% <100.00%> (ø)`

... and 172 files with indirect coverage changes

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Add comprehensive tests for HeaderValue (all 12 types, factory methods, accessors, validation, toString, equals/hashCode) and HeaderKey (fromString, validation, equals/hashCode, UTF-8 handling). Also raise codecov patch target from 70% to 80%. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

The build.sh script was removed in an upstream refactor. Use ./gradlew jacocoAggregatedReport directly, matching the pattern used by other steps in the composite action. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Remove HeaderKey/HeaderValue tests and increase the codecov project threshold from 0.5% to 6% to accommodate the coverage drop from adding Java reporting. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

.codecov.yml

mmodzelewski

Hey @ex172000, please have a look at the few suggestions I've added.

.codecov.yml

foreign/java/build.gradle.kts

ex172000 · 2026-02-08T04:02:13Z

Hey @ex172000, please have a look at the few suggestions I've added.

Thanks for the review, please take another look?

codecov.yml

mmodzelewski · 2026-02-08T19:28:34Z

@hubcio, would you mind checking it as well?

ex172000 force-pushed the qichao/add-java-cov branch from 1509d55 to 8bbdf37 Compare January 27, 2026 07:41

ex172000 changed the title ~~ci(coverage): add JaCoCo code coverage with Codecov integration~~ ci: add JaCoCo code coverage with Codecov integration Jan 27, 2026

ex172000 marked this pull request as draft January 27, 2026 23:57

hubcio reviewed Jan 29, 2026

View reviewed changes

.github/actions/java-gradle/pre-merge/action.yml Outdated Show resolved Hide resolved

ex172000 and others added 4 commits February 6, 2026 22:20

add the missing repo

c08dbb8

Add an example & fix minor things

2fc55c5

ex172000 force-pushed the qichao/add-java-cov branch from 690574d to e0a1c91 Compare February 7, 2026 06:54

ex172000 and others added 2 commits February 6, 2026 23:35

fix(ci): replace removed build.sh with direct gradlew call

1b324fe

The build.sh script was removed in an upstream refactor. Use ./gradlew jacocoAggregatedReport directly, matching the pattern used by other steps in the composite action. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

fix(ci): remove test files and raise codecov project threshold

63b15ea

Remove HeaderKey/HeaderValue tests and increase the codecov project threshold from 0.5% to 6% to accommodate the coverage drop from adding Java reporting. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

ex172000 commented Feb 7, 2026

View reviewed changes

.codecov.yml Outdated Show resolved Hide resolved

ex172000 marked this pull request as ready for review February 7, 2026 08:00

mmodzelewski requested changes Feb 7, 2026

View reviewed changes

.codecov.yml Show resolved Hide resolved

foreign/java/build.gradle.kts Show resolved Hide resolved

foreign/java/build.gradle.kts Outdated Show resolved Hide resolved

ex172000 added 5 commits February 7, 2026 08:59

address comments

f36b9a3

fix jacoco version

c3a66dc

Update codecov.yml

f3729c5

fix versioning

fd9ff50

change back threshold

075e10a

mmodzelewski reviewed Feb 8, 2026

View reviewed changes

codecov.yml Outdated Show resolved Hide resolved

Merge branch 'master' into qichao/add-java-cov

d0fe656

remove flags section

2cb520c

ex172000 added 2 commits February 8, 2026 19:36

Fix: add the required CI runtime env

3ac2efe

Fix path problem and disable search

2e176b6

hubcio approved these changes Feb 9, 2026

View reviewed changes

spetz approved these changes Feb 9, 2026

View reviewed changes

Merge branch 'master' into qichao/add-java-cov

dcbe6c8

mmodzelewski approved these changes Feb 9, 2026

View reviewed changes

mmodzelewski merged commit 2e74a3e into apache:master Feb 9, 2026
92 of 127 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

ci: add JaCoCo code coverage with Codecov integration#2623

ci: add JaCoCo code coverage with Codecov integration#2623
mmodzelewski merged 17 commits intoapache:masterfrom
ex172000:qichao/add-java-cov

ex172000 commented Jan 27, 2026 •

edited

Loading

Uh oh!

hubcio commented Jan 27, 2026

Uh oh!

ex172000 commented Jan 27, 2026

Uh oh!

Uh oh!

mmodzelewski commented Feb 5, 2026

Uh oh!

ex172000 commented Feb 5, 2026

Uh oh!

codecov bot commented Feb 7, 2026 •

edited

Loading

Uh oh!

Uh oh!

mmodzelewski left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

ex172000 commented Feb 8, 2026

Uh oh!

Uh oh!

mmodzelewski commented Feb 8, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

ex172000 commented Jan 27, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

hubcio commented Jan 27, 2026

Uh oh!

ex172000 commented Jan 27, 2026

Uh oh!

Uh oh!

mmodzelewski commented Feb 5, 2026

Uh oh!

ex172000 commented Feb 5, 2026

Uh oh!

codecov bot commented Feb 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Uh oh!

mmodzelewski left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

ex172000 commented Feb 8, 2026

Uh oh!

Uh oh!

mmodzelewski commented Feb 8, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

ex172000 commented Jan 27, 2026 •

edited

Loading

codecov bot commented Feb 7, 2026 •

edited

Loading