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

Javascript - Ignore configured expressions #2084

Conversation

itsmeraji
Copy link

Please check this PR. This is an attempt to configure the statements to be removed from getting mutated.

@itsmeraji
Copy link
Author

@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.

@simondel
Copy link
Member

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 simondel closed this Mar 13, 2020
@itsmeraji
Copy link
Author

@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.

@simondel
Copy link
Member

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.

@itsmeraji
Copy link
Author

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.

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