-
Notifications
You must be signed in to change notification settings - Fork 250
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
Javascript - Ignore configured expressions #2084
Javascript - Ignore configured expressions #2084
Conversation
@nicojs / @kmdrGroch , we've been waiting for ignoring propTypes and such from mutation in Stryker. This PR has the relevant changes. Please check and let me know if this will hold good. |
We want to introduce a parser API to support an easy and safe way to make the exclusions. This issue will lay the groundwork for it: #1893. Before this is done, we will not support this option. We will keep your work here in mind as a reference ♥ |
@simondel, Thanks for the response. When can we expect that useful parser API ? Our team wants to implement mutation testing but we re waiting for this for a long time. |
We'll be working on this or the coming months. What's keeping you from using mutation testing without ignoring statements? It will result in a lower score, but that will simply be your baseline. |
We have a lot of applications and the developers feel that their testing score is less because there are a lot of things that shouldn't be really tested (like propTypes, PropTypes.oneof, logger.debug, etc), but mutation is done for everything. So we wanted to ignore the unwanted mutations before widely practicing it. |
Please check this PR. This is an attempt to configure the statements to be removed from getting mutated.