unit-api icon indicating copy to clipboard operation
unit-api copied to clipboard

Published 20 hours ago • verified publisher icon unitsofmeasurement

Improve code coverage of ServiceProvider

Open keilw opened this issue 4 years ago • 1 comments

At the moment ServiceProvider has a lower code coverage which affects the overall metrics for the project. Should be above 80% for Sonatype. While the new annotations or also existing javax ones can be tested via extension modules or the RI there are no tests here. Either ServiceProvider should be excluded from JacoCo or we get a test in here that also tests those annotations.

keilw avatar Nov 17 '20 15:11 keilw

Haven't been here for a while, but after some playing around and reviewing I've a strategy to improve coverage, so there will be a PR soon for discussion.

Oliver-Loeffler avatar Jan 21 '22 21:01 Oliver-Loeffler

Note: pull request #242 (if accepted) should probably be merged before to work on test coverage.

desruisseaux avatar Oct 02 '22 15:10 desruisseaux

With the latest PRs up to #251 the coverage has gone up to nearly 90%. We may never get to 100, but that is a pretty good improvement and also marked in green, hence closing this ticket.

keilw avatar May 19 '23 21:05 keilw