-
-
Notifications
You must be signed in to change notification settings - Fork 12
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
time-machine test dependency is problematic for PyPy3 #132
Comments
the time-machine problem is not so big. The PyPy3 support is a bigger problem. This could take a while. |
Oh, I've assumed that the lack of If that's any help, my patch was: diff --git a/tests/conftest.py b/tests/conftest.py
index 5ef37fc..8a69dc2 100644
--- a/tests/conftest.py
+++ b/tests/conftest.py
@@ -268,8 +268,8 @@ import datetime
import pytest
@pytest.fixture(autouse=True)
-def set_time(time_machine):
- time_machine.move_to(datetime.datetime(2024, 3, 14, 0, 0, 0, 0),tick=False)
+def set_time(freezer):
+ freezer.move_to(datetime.datetime(2024, 3, 14, 0, 0, 0, 0))
yield
"""
) |
and I also know that I have a |
0.14.0 official supports pypy. But I removed the PYPY classifier because the code rewriting does not currently work. |
Thanks! |
I'm sorry to be bothering you but the
time-machine
test dependency that is present right now makes it impossible to run over 200 tests on PyPy3.time-machine
relies on patching internals of CPython's_datetime
implementation, and as such is not installable on PyPy3 at all.Could you please consider using e.g.
pytest-freezer
instead? This one's portable and works across all Python implementations.Unfortunately, PyPy3 support became important right now because
pydantic-core
started usinginline-snapshot
, and its consumers do support PyPy3.The text was updated successfully, but these errors were encountered: