Add lucene coverage to overall report #3538

ScottDugas · 2025-08-08T16:25:31Z

Currently Lucene and other classes are not being included in the overall coverage report on a PR. This fixes that by doing two things:

Publish the jars along with coverage data, as those are needed to process the .exec files that jacoco uses
download the coverage data from sub-jobs as separate tasks. Otherwise it removes the hierarchy.

other doesn't build lucene so it won't have the libs

… point

ScottDugas added 9 commits August 8, 2025 12:25

Log downloaded artifacts

d162862

Try not merging multiple

54c8d8d

Download individually

fe4b529

Put individually downloaded files in the root

6e4e537

Publish the jars for coverage too

b910377

other doesn't build lucene so it won't have the libs

Try compiling the java code

ef4ba29

Build jars in subprojects

1f4f3c4

Publish jars for other-tests

ff1dcea

We need the core jar for other tests to pass, may need lucene at some…

7e81c16

… point

ScottDugas added testing improvement Change that improves our testing build improvement Improvement to the build system labels Aug 11, 2025

ScottDugas changed the title ~~Log downloaded artifacts~~ Add lucene coverage to overall report Aug 11, 2025

Remove logging

30c4536

ScottDugas requested a review from alecgrieser August 11, 2025 19:05

ScottDugas marked this pull request as ready for review August 11, 2025 19:05

arnaud-lacurie approved these changes Aug 12, 2025

View reviewed changes

alecgrieser approved these changes Aug 14, 2025

View reviewed changes

alecgrieser merged commit 5fc17e0 into FoundationDB:main Aug 14, 2025
6 checks passed

Provide feedback