-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
Resolve Pitest suppression for ParenPad #12282
Comments
@rnveach We have currently this line of code which is mutated checkstyle/src/main/java/com/puppycrawl/tools/checkstyle/checks/whitespace/ParenPadCheck.java Line 257 in ad27aa0
and the source code for this mutated code is just to remove it |
The mutation is not to remove it.
Wiki saying remove it is not correct, but we change case and move it to default. |
@rnveach yes we don't have to remove the switch case according to wiki I have also done like that before. and know it is clear to me when we are making the changes like that I am getting errors in three input file i am sharing one with you Lines 122 to 126 in ad27aa0
here the violation is missing in line 125 at 2 places according to the current scenario checkstyle don't generate the violation for any case for QUESTION I just want to know it is expected a violation in such condition like line no 126 in same file according to its config comment msg
|
If you make a change to mimic a mutation and our test fails, then either the mutation you applied isn't correct or there is more going on with pitest than the mutation alone (caching, etc...). Normally tests do not fail with the mutation applied. Please provide me the step by step we request. Provide me the mutated branch pushed to the server and paste the URL here. Let me know if tests fail with the mutation applied. |
Please send PR with mutation to let us see all details. Please try to remove all other "case" one by one. |
Fix was merged, complex understanding of pitest is at #12318 (comment) and simpler understanding at #12318 (comment) |
Related to #7797
kill mutation
checkstyle/.ci/pitest-suppressions/pitest-whitespace-suppressions.xml
Lines 93 to 100 in fa68569
The text was updated successfully, but these errors were encountered: