Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

config: expanded pitest mutators to defaults #6270

Merged
merged 1 commit into from
Dec 12, 2018

Conversation

rnveach
Copy link
Member

@rnveach rnveach commented Dec 10, 2018

This is in support of #6264 .

Defaults are not added by default when you start adding new mutators. Just adding the new one without the defaults results in only the new mutator running. This is what I used to generate the report in the first post of the issue.

@romani
Copy link
Member

romani commented Dec 12, 2018

Please confirm, that no new mutations were added and all referenced are defaults, is there a link to that list at pitest web site ?

@rnveach
Copy link
Member Author

rnveach commented Dec 12, 2018

that no new mutations were added

Isn't this what what our pitest.sh script does in circleCI?
CI passed so percentages remained the same, and script didn't find any new survived mutations. Only package we don't really test is gui.

You can see old pitest report at http://rveach.no-ip.org/checkstyle/regression/pitest-reports/18/ (commit is in the report ).

all referenced are defaults, is there a link to that list at pitest web site ?

I forgot to include the link.
I am using this list as the list of defaults: https://github.com/hcoles/pitest/blob/6101377de641c57a17ff82545682b9b7fbc3c061/pitest/src/main/java/org/pitest/mutationtest/engine/gregor/config/Mutator.java#L209-L215

You can also see defaults in their document at http://pitest.org/quickstart/mutators/ . See activated by default on top.

@romani romani merged commit d3e5b0e into checkstyle:master Dec 12, 2018
@rnveach rnveach deleted the issue_6264_2 branch December 12, 2018 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants