search

apache / gravitino

World's most powerful open data catalog for building a high-performance, geo-distributed and federated metadata lake.
https://gravitino.apache.org
Apache License 2.0
1.03k stars 320 forks source link

[Bug report] Code coverage tool `JacocoReport` does not work #1488

Open yuqi1129 opened 9 months ago

yuqi1129 commented 9 months ago

Describe what's wrong

The test coverage tool JacocoReport did not work at some point, we need to make it run normally.

Error message and/or stacktrace

Branch 0.2 is fine and the code coverage page will be available once a merge request is launched. But the main branch can't demonstrate it. We should check what's going wrong.

image

How to reproduce

Please see above.

Additional context

No response

hiirrxnn commented 9 months ago

I would like to work on this . Please tell me precisely what is to be done here.

justinmclean commented 9 months ago

Unless you have experience with the tool and the GitHub plugin it might be best to select another issue to work on.

yuqi1129 commented 9 months ago

@hiirrxnn Hi, are you still interested in this issue?

yijhenlin commented 7 months ago

It seems like the file integration-test-common/build/reports/jacoco/test/jacocoTestReport.xml is incorrect. (There was a similar issue in Madrapps/jacoco-report) I'm not sure if we need the report of integration-test-common, since I saw the workflow use the arg -PskipITs. If not, maybe this issue can be resolved by ignoring integration-test-common when generating reports. (or skipITs should skip this module?) Snipaste_2024-03-29_20-43-08

justinmclean commented 3 months ago

@yuqi1129 Are you still working on this?

yuqi1129 commented 3 months ago

@yuqi1129 Are you still working on this?

Yeah, I'm communicating with the ASF community and seeking their help.