Anders Aaberg

Now that OpenClover seems dead and is incompatible with Groovy 3 + Grails 5, then we desperately need to fix Jacoco coverage for Grails in order to upgrade to Grails...

Many thanks for the fixes @aulea - with Grails 5.1.2, coverage has improved to 66% instruction coverage and 69% branch coverage! However, it seems that many getters and setters are...

@aulea : Many thanks, I think that you are right that the fix needs to be made in Groovy. I tried to add `@Generated` on the private fields in `RequestForwarder`,...

@aulea : gr8, the instruction coverage increased: from 66% to 81%. However, there is still some work needed. I have a Controller, which takes a command object: ``` class TestController...

@aulea remember to sign the CLA so we can get this nice fix applied

@aulea : sorry for the late reply. It sounds very strange that the CLA does not update after your signature. I've tried it before, where I accidentally made the commit...

Please include this in Grails 5, as we cannot use OpenClover in Grails 5 and Jacoco coverage is currently very bad compared to OpenClover in Grails 4.

Candidate fix: https://github.com/grails/gorm-hibernate5/compare/7.2.x...andersaaberg:patch-1

Please include this in Grails 5, as we cannot use OpenClover in Grails 5 and Jacoco coverage is currently very bad compared to OpenClover in Grails 4.

@puneetbehl : then we need to set that configuration for all of our projects - that does not really seem to adhere to "convention over configuration". How come can the...