ianhomer / odin

Sequencer to enhance live musical performances, jamming and compositions
Other
5 stars 0 forks source link

Poor code coverage on lombok beans #4

Open ianhomer opened 7 years ago

ianhomer commented 7 years ago

Beans annotated with @ Data from lombok, e.g. PersistableMetronome have poor test coverage. This is because these auto generated methods are not covered.

This will be solved soon by lombo, jacoco & sonar

Lombok provides the foundation in v1.16.14 to generate a @lombok.Generated annotation.

Jacoco will use this in 0.7.10 to update the coverage data

And then - as per https://github.com/jacoco/jacoco/pull/513#issuecomment-293176354 - sonar will need to do something to filter out this data.

For now ... no need to do anything in the odin project ... just follow this integration being kindly provided :)

When this all provided we may need to add the following to the lombok.config depending on what the default behaviour is

lombok.addLombokGeneratedAnnotation = true