pitest / pitclipse

Mutation testing for Java in Eclipse IDE. Based on PIT (Pitest).
https://pitest.org
Apache License 2.0
59 stars 17 forks source link

Target class selection #186

Closed JKutscha closed 2 years ago

JKutscha commented 2 years ago

Issue #164. Follow up of #173.

JKutscha commented 2 years ago

@LorenzoBettini would you have a look at these changes and maybe open a companion PR with sonar?

Most of the changes in this PR were already reviewed in #173.

LorenzoBettini commented 2 years ago

@LorenzoBettini would you have a look at these changes and maybe open a companion PR with sonar?

Most of the changes in this PR were already reviewed in #173.

@JKutscha sorry for the long delay I took some time to review this and SonarCloud found a bug: a possible stack overflow in a regular expression; I fixed this by using possessive quantifiers, https://github.com/pitest/pitclipse/pull/197/commits/4a179136da3c5c13b93e1328024185cbb3c527e4

I'm going to merge this soon.

Thanks again!