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

Support --ignoreStatic #3282

Closed
nicojs opened this issue Nov 26, 2021 · 2 comments · Fixed by #3284
Closed

Support --ignoreStatic #3282

nicojs opened this issue Nov 26, 2021 · 2 comments · Fixed by #3284
Labels
🚀 Feature request New feature request

Comments

@nicojs
Copy link
Member

nicojs commented Nov 26, 2021

We want StrykerJS to be able to ignore static mutants. This is a requirement for supporting native ES modules, as well as a performance booster for others.

See #2922

@nicojs nicojs added the 🚀 Feature request New feature request label Nov 26, 2021
@nicojs
Copy link
Member Author

nicojs commented Nov 26, 2021

Work on --ignoreStatic has started in the feat/ignore-static branch.

The preliminary results are great! Tests with running Stryker on @stryker-mutator/core:

Without ignoring static:

INFO MutationTestExecutor Done in 8 minutes 50 seconds.

With --ignoreStatic

INFO MutationTestExecutor Done in 3 minutes 56 seconds.

Note: I've disabled the typescript-checker for this test run

The difference is 189 mutants out of a total of 3115. So ~6% of all mutants cost more than 50% of the execution time. Absolutely mindblowing 🤯

The resulting mutation score is different (as to be expected).

@nicojs
Copy link
Member Author

nicojs commented Mar 2, 2022

Closed with #3284

@nicojs nicojs closed this as completed Mar 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚀 Feature request New feature request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant