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

migrate from external mock package to stdlib unittest.mock #470

Merged
merged 2 commits into from
Dec 5, 2023

Conversation

eli-schwartz
Copy link
Contributor

It was moved to the stdlib in python 3.3, and the PyPI version is a backport for python versions less than 3.3.

Note: it was also never a dependendency so there was no guarantee it was installed to be used.

@Suor
Copy link
Owner

Suor commented Dec 5, 2023

And yet it somehow broken PyPy 3.9

@eli-schwartz
Copy link
Contributor Author

eli-schwartz commented Dec 5, 2023

And yet it somehow broken PyPy 3.9

Considering that PyPy 3.9 has segfaulted in the tests, I would very much like to know how my patch resulted in PyPy segfaulting.

I do not acknowledge that I broke anything. :) I don't know what happened, but I'm positive that it's not related.

Whatever is going on here, it is probably a bug in the PyPy interpreter though. Segfaults usually are.

@Suor
Copy link
Owner

Suor commented Dec 5, 2023

It might be their bug but it's my problem :) If I merge it I will need to fix it.

@Suor
Copy link
Owner

Suor commented Dec 5, 2023

Saying that, if changing from PyPy3.9 to 3.10 fixes it then it is fine with me.

It was moved to the stdlib in python 3.3, and the PyPI version is a
backport for python versions less than 3.3.
@eli-schwartz
Copy link
Contributor Author

That appears to have magically fixed it.

@Suor Suor merged commit a97c521 into Suor:master Dec 5, 2023
8 checks passed
@Suor
Copy link
Owner

Suor commented Dec 5, 2023

Thanks, merged

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