Skip to content

Commit

Permalink
Ignore some unused arguments in tests
Browse files Browse the repository at this point in the history
  • Loading branch information
santisoler committed Dec 6, 2024
1 parent 2e2d316 commit 442d912
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion pooch/tests/test_core.py
Original file line number Diff line number Diff line change
Expand Up @@ -519,7 +519,7 @@ def test_check_availability_on_ftp(ftpserver):
def test_check_availability_invalid_downloader():
"Should raise an exception if the downloader doesn't support this"

def downloader(url, output, pooch): # pylint: disable=unused-argument
def downloader(url, output, pooch): # noqa: ARG001
"A downloader that doesn't support check_only"
return None

Expand Down
4 changes: 2 additions & 2 deletions pooch/tests/test_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,7 @@ def mockmakedirs(path, exist_ok=False): # pylint: disable=unused-argument
def test_local_storage_makedirs_permissionerror(monkeypatch):
"Should warn the user when can't create the local data dir"

def mockmakedirs(path, exist_ok=False): # pylint: disable=unused-argument
def mockmakedirs(path, exist_ok=False): # noqa: ARG001
"Raise an exception to mimic permission issues"
raise PermissionError("Fake error")

Expand All @@ -104,7 +104,7 @@ def test_local_storage_newfile_permissionerror(monkeypatch):
# This is a separate function because there should be a warning if the data
# dir already exists but we can't write to it.

def mocktempfile(**kwargs): # pylint: disable=unused-argument
def mocktempfile(**kwargs): # noqa: ARG001
"Raise an exception to mimic permission issues"
raise PermissionError("Fake error")

Expand Down

0 comments on commit 442d912

Please sign in to comment.