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

string mutation causes UnicodeDecodeError #253

Closed
golankai opened this issue Jul 27, 2022 · 4 comments
Closed

string mutation causes UnicodeDecodeError #253

golankai opened this issue Jul 27, 2022 · 4 comments

Comments

@golankai
Copy link

I am working on a project that includes text normalization and using mutmut for mutation testing.
mutmut is mutating line 35 to be normalized = 'XXXX' instead of normalized = ''
image

I then get the following error:
image

when I run my tests regularly they work, and when I apply this change manually and run my tests, they work too.

Have any Idea what can happen that causes this failure?

@boxed
Copy link
Owner

boxed commented Jul 27, 2022

Are you sure this is the mutation that fails? Is that what you get with mutmut show 0?

"position 35" in the error would be the column 35, not the line 35 of the file.

@golankai
Copy link
Author

the position is indeed 35, but coincidently also the row I referred to :) when reporting the error, the position changes, as it is using a random string from Hypothesis testing.
I am sure.

  1. Running the tests themselves produces no error.
  2. Applying the mutation manually and running the tests produces an assertion error (this is good) but not a fatal Error of 'utf-8'.
  3. running mutmut run crashes as I showed above, and then also by running mutmut run 4 (4 is the id of this mutant).

This is quite weird. I have seen before in our project those errors that comes from normalization, but I don't get how and why this mutant causes them, only when running through mutmut.

@boxed
Copy link
Owner

boxed commented Jul 28, 2022

Weird. You could add a print(repr(line)) in the mutmut code.

@boxed
Copy link
Owner

boxed commented Oct 20, 2024

I just released mutmut 3, which is a big rewrite. I believe this issue no longer applies anymore. Feel free to reopen it if it still exists.

@boxed boxed closed this as completed Oct 20, 2024
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

No branches or pull requests

2 participants