Assignment not mutated away #3201
-
|
Was a little surprised when I did not have a mutation in the following code on line 31
I would expect that a simple assignment like that would be mutated to just be replaced with a ; - just like the method call in line 30 does - but apparently it does not. And reading the docs for supported mutations, I can't find anything that would do that to a simple assignment. I tried to search for the answer here, but to no avail - what am I missing here? |
Beta Was this translation helpful? Give feedback.
Replies: 3 comments 5 replies
-
|
Our docs are generally up to date with the implementation. If it's not listed as a supported mutation, it is not a mutation we perform. As you can see here: https://stryker-mutator.io/docs/stryker-net/mutations/#removal-mutators-statement-block assignment statements are not listed, so you're not missing anything. It is not currently a mutation we perform. |
Beta Was this translation helpful? Give feedback.
-
|
It was discussed here: #870 It was not implemented at the time but we could probably revisit that since the conclusion was that it would be useful. |
Beta Was this translation helpful? Give feedback.
-
|
And thanks for the reference to the feature request - I clearly failed to find that while searching! |
Beta Was this translation helpful? Give feedback.
-
|
Super - it just seemed like it would be an obvious thing to mutate, so I'm wondering if it had been tried but there are problems with it I can't see. If not, it might be something, if I got the time, that might be useful to look at and create a PR for it? |
Beta Was this translation helpful? Give feedback.
-
|
It will potentially create a lot of mutants, which will have a big impact on total runtime. I think that's about it. |
Beta Was this translation helpful? Give feedback.
-
|
well, @skovsende, what kind of mutation you have in mind ? besides assignment to default value (which does not exist) and removal (which does exist but is disabled for assignments). |
Beta Was this translation helpful? Give feedback.
-
|
In this particular case it would have been nice that there had been a mutator that just removed it - because with the test suite that we currently have there are no test failures if I delete the line - I had hoped stryker would have catched that and told me that the line wasn't actually tested. |
Beta Was this translation helpful? Give feedback.
-
|
Oh, and thanks for taking the time to answer! :) |
Beta Was this translation helpful? Give feedback.
Our docs are generally up to date with the implementation. If it's not listed as a supported mutation, it is not a mutation we perform. As you can see here: https://stryker-mutator.io/docs/stryker-net/mutations/#removal-mutators-statement-block assignment statements are not listed, so you're not missing anything. It is not currently a mutation we perform.