[CALCITE-5475] Improve test coverage accuracy by aggregating modules

• Type: Task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.33.0
• Labels: pull-request-available
• Assignee: Stamatis Zampetakis
• Reporter: Stamatis Zampetakis
• Created: 13/Jan/23 15:11
• Updated: 07/Feb/23 22:33
• Resolved: 17/Jan/23 12:25

Description

Currently the test coverage report is computed per module (using jacocoTestReport task) and upload it to Sonar as is. Considering the modules in isolation leads to some modules have rather low coverage.

Example

For testkit module, Sonar reports only 11% coverage at the moment of submitting this ticket:
https://sonarcloud.io/component_measures?metric=coverage&selected=apache_calcite%3Atestkit%2Fsrc%2Fmain&id=apache_calcite

Some classes such as SqlOperatorTest are as low as 1%.

The testkit module does not have itself many tests covering its own classes (under src/main) so if we don't consider how it is used by other modules we get the false impression that it is not sufficiently covered.

The test coverage accuracy can be improved by aggregating the results from all modules together using the jacoco_report_aggregation_plugin.

Issue Links

• links to: GitHub Pull Request #3027