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

Test fixes for changes in recent upstream Python (#581) #582

Merged
merged 3 commits into from
Sep 27, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 5 additions & 2 deletions toolz/tests/test_functoolz.py
Original file line number Diff line number Diff line change
Expand Up @@ -738,10 +738,13 @@ def f(a, b):
def test_excepts():
# These are descriptors, make sure this works correctly.
assert excepts.__name__ == 'excepts'
# in Python < 3.13 the second line is indented, in 3.13+
# it is not, strip all lines to fudge it
testlines = "\n".join((line.strip() for line in excepts.__doc__.splitlines()))
assert (
'A wrapper around a function to catch exceptions and\n'
' dispatch to a handler.\n'
) in excepts.__doc__
'dispatch to a handler.\n'
) in testlines

def idx(a):
"""idx docstring
Expand Down
21 changes: 19 additions & 2 deletions toolz/tests/test_inspect_args.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
import inspect
import itertools
import operator
import sys
import toolz
from toolz.functoolz import (curry, is_valid_args, is_partial_args, is_arity,
num_required_args, has_varargs, has_keywords)
Expand Down Expand Up @@ -482,6 +483,22 @@ def __wrapped__(self):
wrapped = Wrapped(func)
assert inspect.signature(func) == inspect.signature(wrapped)

assert num_required_args(Wrapped) is None
_sigs.signatures[Wrapped] = (_sigs.expand_sig((0, lambda func: None)),)
# inspect.signature did not used to work properly on wrappers,
# but it was fixed in Python 3.11.9, Python 3.12.3 and Python
# 3.13+
inspectbroken = True
if sys.version_info.major > 3:
inspectbroken = False
if sys.version_info.major == 3:
if sys.version_info.minor == 11 and sys.version_info.micro > 8:
inspectbroken = False
if sys.version_info.minor == 12 and sys.version_info.micro > 2:
inspectbroken = False
if sys.version_info.minor > 12:
inspectbroken = False

if inspectbroken:
assert num_required_args(Wrapped) is None
_sigs.signatures[Wrapped] = (_sigs.expand_sig((0, lambda func: None)),)

assert num_required_args(Wrapped) == 1
Loading