4588653b24
ruff is faster and handle everything we had prior. isort configuration done based on the indication from https://github.com/astral-sh/ruff/issues/4670, previousely based on reorder-python-import (#11896) flake8-docstrings was a wrapper around pydocstyle (now archived) that explicitly asks to use ruff in https://github.com/PyCQA/pydocstyle/pull/658. flake8-typing-import is useful mainly for project that support python 3.7 and the one useful check will be implemented in https://github.com/astral-sh/ruff/issues/2302 We need to keep blacken-doc because ruff does not handle detection of python code inside .md and .rst. The direct link to the repo is now used to avoid a redirection. Manual fixes: - Lines that became too long - % formatting that was not done automatically - type: ignore that were moved around - noqa of hard to fix issues (UP031 generally) - fmt: off and fmt: on that is not really identical between black and ruff - autofix re-order in pre-commit from faster to slower Co-authored-by: Ran Benita <ran@unusedvar.com>
69 lines
1.9 KiB
Python
69 lines
1.9 KiB
Python
"""Module containing a parametrized tests testing cross-python serialization
|
|
via the pickle module."""
|
|
|
|
import shutil
|
|
import subprocess
|
|
import textwrap
|
|
|
|
import pytest
|
|
|
|
|
|
pythonlist = ["python3.9", "python3.10", "python3.11"]
|
|
|
|
|
|
@pytest.fixture(params=pythonlist)
|
|
def python1(request, tmp_path):
|
|
picklefile = tmp_path / "data.pickle"
|
|
return Python(request.param, picklefile)
|
|
|
|
|
|
@pytest.fixture(params=pythonlist)
|
|
def python2(request, python1):
|
|
return Python(request.param, python1.picklefile)
|
|
|
|
|
|
class Python:
|
|
def __init__(self, version, picklefile):
|
|
self.pythonpath = shutil.which(version)
|
|
if not self.pythonpath:
|
|
pytest.skip(f"{version!r} not found")
|
|
self.picklefile = picklefile
|
|
|
|
def dumps(self, obj):
|
|
dumpfile = self.picklefile.with_name("dump.py")
|
|
dumpfile.write_text(
|
|
textwrap.dedent(
|
|
rf"""
|
|
import pickle
|
|
f = open({str(self.picklefile)!r}, 'wb')
|
|
s = pickle.dump({obj!r}, f, protocol=2)
|
|
f.close()
|
|
"""
|
|
)
|
|
)
|
|
subprocess.run((self.pythonpath, str(dumpfile)), check=True)
|
|
|
|
def load_and_is_true(self, expression):
|
|
loadfile = self.picklefile.with_name("load.py")
|
|
loadfile.write_text(
|
|
textwrap.dedent(
|
|
rf"""
|
|
import pickle
|
|
f = open({str(self.picklefile)!r}, 'rb')
|
|
obj = pickle.load(f)
|
|
f.close()
|
|
res = eval({expression!r})
|
|
if not res:
|
|
raise SystemExit(1)
|
|
"""
|
|
)
|
|
)
|
|
print(loadfile)
|
|
subprocess.run((self.pythonpath, str(loadfile)), check=True)
|
|
|
|
|
|
@pytest.mark.parametrize("obj", [42, {}, {1: 3}])
|
|
def test_basic_objects(python1, python2, obj):
|
|
python1.dumps(obj)
|
|
python2.load_and_is_true(f"obj == {obj}")
|