search

STAMP-project / test-runner

test driver to run JUnit tests isolated in a new JVM
GNU General Public License v3.0
16 stars 15 forks source link

Option for intrumenting and computing the coverage of test classes #96

Open andre15silva opened 3 years ago

andre15silva commented 3 years ago

Right now, only source classes are instrumented. We should have an option for enabling test instrumentation too.

martinezmatias commented 3 years ago

Hi @andre15silva

Right now, only source classes are instrumented. We should have an option for enabling test instrumentation too.

I think we have it (as far I remember I implemented):

The boolean field that indicates whether we would cover or not code from tests is here: https://github.com/SpoonLabs/flacoco/blob/master/src/main/java/fr/spoonlabs/flacoco/core/config/FlacocoConfig.java#L32

andre15silva commented 3 years ago

Hi @martinezmatias ,

yes, but you instrumented the test classes in CoverageRunner, which now isn't an option since we use test-runner through the EntryPoint.

I'm just finishing a PR to test-runner which enables an options for covering the tests.

martinezmatias commented 3 years ago

Thanks @andre15silva

yes, but you instrumented the test classes in CoverageRunner, which now isn't an option since we use test-runner through the EntryPoint.

Perfect. I forgot the new way we interact with 'test-runner'