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

Feat: add option to set detail of coverage information + method level instruction coverage #88

Closed lacinoire closed 3 years ago

lacinoire commented 3 years ago

(includes / dependent on #87)

Good afternoon 😊

For Test Cube I needed more detailed coverage information, specifically the instruction coverage reported per method. I adapted my modifications to fit @martinezmatias new coverage transformers and added an option to specify which coverage transformer should be used.

Included a test for the new method-detailed coverage and a more general test to help me understand the comparison of two coverages.

martinezmatias commented 3 years ago

Hi @lacinoire

The PR looks great to me and can be merged. (I don't have rights to merge) Regards Matias

danglotb commented 3 years ago

Looks like there are common commits with #87 Would be possible to remove them in order to split the fix/feat into two distinct pull-requests? If not, we might close #87 and merge this one. Thank you! :smile:

lacinoire commented 3 years ago

I cherry picked the last commit and now there are no overlaps with #87 anymore 🙂