Hi Petr,

On Thu, Jun 25, 2015 at 9:40 AM, Petr Shypila <ikrump...@gmail.com> wrote:
> ...I also got a look on other modules under ./bundles and it looks like engine
> is a very good one, because it's coverage is only 20%(e.g. resourceresolver
> coverage is 70%)....

Both engine and resourceresolver are good candidates, as they are
critical core bundles.

I suspect that our launchpad integration tests also cover quite a bit
of those modules, it might be nice to have a look at the aggregate
coverage of (for example) the bundles/engine module based on both its
own tests and on the launchpad/testing tests.

I did experiment with generating that aggregate launchpad + local
coverage in SLING-1803 (the integration-deploys-jacoco-exec.patch) in
addition to SLING-4760. Once you're mostly done with the contentloader
tests it might be good to look at this - if we can easily get that
aggregate coverage that might help focus your efforts better. That
patch is probably stale but we could reuse the general idea - having
launchpad/testing deploy its coverage data to Maven so that other
modules can merge it with their own coverage.

> ...I see that most tests are written with
> JMock. Could be there some other useful information for me how to test this
> module in better way?...

I've been moving to Mockito lately, the code is more readable than
JMock IMO. If a module has few tests so far you might switch it to
Mockito for consistency. Maybe JMock has also improved in the
meantime.

-Bertrand

Reply via email to