Skip to content

Commit

Permalink
Merge pull request #10 from RazerM/feature/remove-unused-skipif
Browse files Browse the repository at this point in the history
Remove unused skipif markers
  • Loading branch information
RazerM authored Jan 23, 2024
2 parents 73fcca9 + 52521c1 commit 042ceca
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 15 deletions.
18 changes: 5 additions & 13 deletions tests/test_autorepr.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
import sys
from textwrap import dedent
from unittest.mock import Mock

Expand Down Expand Up @@ -89,19 +88,13 @@ def __init__(self, a, b):
pass


@pytest.mark.skipif(sys.version_info < (3,), reason="Requires Python 3")
def test_kwonly():
code = dedent(
"""
with pytest.raises(ValueError):
@autorepr(positional='a')
class A:
def __init__(self, *, a):
pass
"""
)
with pytest.raises(ValueError):

exec(code)
@autorepr(positional="a")
class A:
def __init__(self, *, a):
pass


def test_exceptions():
Expand Down Expand Up @@ -161,7 +154,6 @@ def __init__(self, a, b):
assert repr(a) == "A(a=1, b=2)"


@pytest.mark.skipif(sys.version_info < (3, 2), reason="Requires Python 3.2+")
def test_recursive_repr():
"""Test that autorepr applies the :func:`reprlib.recursive_repr` decorator."""

Expand Down
2 changes: 0 additions & 2 deletions tests/test_helper.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
import sys
import textwrap

import pytest
Expand Down Expand Up @@ -205,7 +204,6 @@ def _repr_helper_(self, r):
assert pretty(ce) == textwrap.dedent(prettystr).lstrip()


@pytest.mark.skipif(sys.version_info < (3, 2), reason="Requires Python 3.2+")
def test_helper_mixin_recursive():
"""Test that the mixin applies the :func:`reprlib.recursive_repr` decorator."""

Expand Down

0 comments on commit 042ceca

Please sign in to comment.