Skip to content

Commit

Permalink
Remove typeguard
Browse files Browse the repository at this point in the history
  • Loading branch information
Paul Prescod committed Apr 12, 2023
1 parent 8e41f17 commit 87c6278
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 10 deletions.
2 changes: 1 addition & 1 deletion pytest.ini
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
[pytest]
testpaths = tests
addopts = --typeguard-packages=snowfakery -p pytest_snowfakery.salesforce_cci_pytest_plugin
addopts = -p pytest_snowfakery.salesforce_cci_pytest_plugin
1 change: 0 additions & 1 deletion requirements/dev.in
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ pre-commit
pyright
pytest
pytest-cov
typeguard
faker-microservice
tox
tox-gh-actions # needed for CI only
Expand Down
4 changes: 0 additions & 4 deletions requirements/dev.txt
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,6 @@ importlib-metadata==6.3.0
# via
# markdown
# mkdocs
# typeguard
iniconfig==2.0.0
# via pytest
jinja2==3.1.2
Expand Down Expand Up @@ -191,16 +190,13 @@ tox==4.4.11
# tox-gh-actions
tox-gh-actions==3.1.0
# via -r requirements/dev.in
typeguard==3.0.2
# via -r requirements/dev.in
types-pyyaml==6.0.12.9
# via responses
typing-extensions==4.5.0
# via
# -r requirements/prod.txt
# black
# pydantic
# typeguard
urllib3==1.26.15
# via
# -r requirements/prod.txt
Expand Down
7 changes: 3 additions & 4 deletions tests/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -63,10 +63,9 @@ def table_values(tablename, index=None, field=None):

@pytest.fixture(scope="function")
def disable_typeguard():
with patch("typeguard.check_argument_types", lambda *args, **kwargs: ...), patch(
"typeguard.check_return_type", lambda *args, **kwargs: ...
):
yield
# doesn't really do anything. at some point we can remove it if
# typeguard is really gone for good.
yield


@pytest.fixture(scope="function")
Expand Down

0 comments on commit 87c6278

Please sign in to comment.