Skip to content

Commit

Permalink
Move test skip decorator
Browse files Browse the repository at this point in the history
Accidentally was skipping the wrong test, moved the decorator onto the correct test
  • Loading branch information
ethanglide committed Dec 9, 2024
1 parent 9d8e15a commit 2def62d
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions libs/core/tests/unit_tests/test_tools.py
Original file line number Diff line number Diff line change
Expand Up @@ -2177,10 +2177,6 @@ def foo(self, b: str) -> str:
assert a.foo.invoke("b") == "ab"


@pytest.mark.skipif(
sys.version_info >= (3, 13),
reason="Class methods wrapping properties was removed with Python 3.13",
)
def test_method_tool_toolcall_invoke() -> None:
"""Test that a method tool can be invoked with a ToolCall."""

Expand Down Expand Up @@ -2226,6 +2222,10 @@ def foo(cls, a: int, b: int) -> int:
assert A.foo.invoke({"a": 1, "b": 2}) == 8


@pytest.mark.skipif(
sys.version_info >= (3, 13),
reason="Class methods wrapping properties was removed with Python 3.13",
)
def test_method_tool_classmethod_args() -> None:
"""Test that a classmethod tool's args do not include cls."""

Expand Down

0 comments on commit 2def62d

Please sign in to comment.