enapps-enorman commented on PR #10:
URL: 
https://github.com/apache/sling-org-apache-sling-discovery-impl/pull/10#issuecomment-2427296489

   > Strange, I specifically added a test for ClusterViewChangeListener, but 
SonarCloud still reports no lines covered. I can see 90% line coverage after 
running the new test locally.
   
   @mreutegg It looks to me like the jacoco arguments are not being passed to 
the maven-surefire-plugin. This seems to be due to this pom overriding the 
argLine configuration of the maven-surefire-plugin which overwrites what the 
parent pom was doing to pass the jacoco arguments to the test runner.
   
   Changing it to something like below to merge what was in the parent with the 
local values should work:
   `<argLine>-Xmx2048m ${jacoco.ut.command}</argLine>`
   
   If you run maven locally with the jacoco-report profile enabled, you should 
see the reports generated under target/site
   `mvn clean install -P jacoco-report`
   


-- 
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