Support for excluding mutators

[christianhujer]

Feature Request:

I would like to be able to specify the mutators like this:

pitest {
    // …
    mutators = ['All'] - ['UOI']
    // …
}

or, if that is not possible, with something equivalent, like this:

pitest {
    // …
    mutators = ['All']
    excludeMutators = ['UOI']
    // …
}

Use case: instead of having a long list of all mutators to run, be able to specify a large set of mutators, like 'ALL', and then just exclude some from the list, like ['UOI'], when they produce too many false positives.

[npathai]

I will work on it and raise a PR

[szpak]

Just remember to test it thoroughly. What possible in unit (or unit-like) tests.

[npathai]

@szpak Yep will do that. Will look at all existing tests and add required tests accordingly 👍 Will ask if I have any questions

[npathai]

@szpak From what I found out, there is no support for excluding mutators in PIT project. What do you think should be the next step?

[szpak]

I didn't need it before, but it seems there is no easy way to exclude mutators (instead of provide just the list of selected mutators to use). It could be implemented in the Gradle plugin alone, but most likely it would be good to have it available in pit-core. Therefore, I propose to report that suggestion in the PIT issue tracker to see what Henry thinks about that.

[npathai]

Apologies for late reply. I will do that.

[kamesh95]

Any update here? I wanted to change mutators group to ALL with pitest gradle plugin but I cannot find any property to do that in here. Can you help on this? Thank you

[szpak]

@kamesh95 I haven't seen a related issue in the PIT issue tracker. You might want to create one.

[npathai]

@szpak I have raised the feature request in PIT issue tracker.

[szpak]

@npathai Could you check it with PIT 1.7.3?

pitest {
    pitestVersion = '1.7.3'
    mutators = ['ALL', '-UOI', ...]
    ...
}

[npathai]

@szpak Sure let me do that. I will raise PR when ready.