pre-commit autoupdate 2023_05_19

.pre-commit-config.yaml

@@ -21,7 +21,7 @@ repos:
       - id: trailing-whitespace
 
   - repo: https://github.com/charliermarsh/ruff-pre-commit
-    rev: v0.0.261
+    rev: v0.0.269
 
     hooks:
       - id: ruff
@@ -42,7 +42,7 @@ repos:
       - id: codespell  # See setup.cfg for args
 
   - repo: https://github.com/pre-commit/mirrors-mypy
-    rev: v1.1.1
+    rev: v1.3.0
     hooks:
       - id: mypy
         additional_dependencies:

pyproject.toml

@@ -80,11 +80,7 @@ max-statements = 124
 
 [tool.ruff.per-file-ignores]
 "__init__.py" = ["E402"]
-"tests/*" = [
-  "PT017",
-  "S101",
-]
-"tests/conftest.py" = ["B018", "F811"]
+"tests/*" = ["PT017", "S101"]
 "tests/cli/test_ia_list.py" = ["E741"]
 "tests/test_api.py" = ["E712"]
 "tests/test_config.py" = ["PT011"]

tests/conftest.py

@@ -12,17 +12,6 @@
 from internetarchive.cli import ia
 from internetarchive.utils import json
 
-try:
-    FileNotFoundError
-except NameError:
-    FileNotFoundError = IOError
-
-try:
-    WindowsError  # type: ignore[used-before-def]
-except NameError:
-    class WindowsError(Exception):
-        pass
-
 PROTOCOL = 'https:'
 BASE_URL = 'https://archive.org/'
 METADATA_URL = f'{BASE_URL}metadata/'
@@ -74,7 +63,7 @@ def load_test_data_file(filename):
 
 
 def call_cmd(cmd, expected_exit_code=0):
-    proc = Popen(cmd, shell=True, stdout=PIPE, stderr=PIPE)
+    proc = Popen(cmd, shell=True, stdout=PIPE, stderr=PIPE)  # noqa: S602
     stdout, stderr = proc.communicate()
     stdout = stdout.decode('utf-8').strip()
     stderr = stderr.decode('utf-8').strip()
@@ -134,5 +123,5 @@ def nasa_metadata():
 
 # TODO: Why is this function defined twice in this file?  See issue #505
 @pytest.fixture()  # type: ignore
-def nasa_item(nasa_mocker):
+def nasa_item(nasa_mocker):  # noqa: F811
     return get_item('nasa')