[pre-commit.ci] pre-commit autoupdate (#509)

* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/astral-sh/ruff-pre-commit: v0.12.11 → v0.13.3](astral-sh/ruff-pre-commit@v0.12.11...v0.13.3)
- https://github.com/psf/black → https://github.com/psf/black-pre-commit-mirror
- [github.com/psf/black-pre-commit-mirror: 25.1.0 → 25.9.0](psf/black-pre-commit-mirror@25.1.0...25.9.0)
- [github.com/tox-dev/pyproject-fmt: v2.6.0 → v2.7.0](tox-dev/pyproject-fmt@v2.6.0...v2.7.0)

* ruff check --select=RUF059 --fix --unsafe-fixes

* Add Python 3.14 and 3.14t to the testing

* Ruff v0.14.0

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Christian Clauss <cclauss@me.com>

Author: pre-commit-ci[bot]

.github/workflows/tox.yml

@@ -19,13 +19,14 @@ jobs:
       fail-fast: false
       max-parallel: 5
       matrix:
-        python: ["3.9", "3.10", "3.11", "3.12", "3.13"]
+        python: ["3.9", "3.10", "3.11", "3.12", "3.13", "3.14", "3.14t"]
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v5
       - uses: actions/setup-python@v6
         with:
           python-version: ${{ matrix.python }}
+          allow-prereleases: true
       - run: pip install --upgrade pip
       - run: pip install tox
       - run: tox -e py

.pre-commit-config.yaml

@@ -36,17 +36,17 @@ repos:
                 - tomli
 
     - repo: https://github.com/astral-sh/ruff-pre-commit
-      rev: v0.12.11
+      rev: v0.14.0
       hooks:
-          - id: ruff
+          - id: ruff-check
 
-    - repo: https://github.com/psf/black
-      rev: 25.1.0
+    - repo: https://github.com/psf/black-pre-commit-mirror
+      rev: 25.9.0
       hooks:
           - id: black
 
     - repo: https://github.com/tox-dev/pyproject-fmt
-      rev: v2.6.0
+      rev: v2.7.0
       hooks:
           - id: pyproject-fmt
 

pyproject.toml

@@ -2,37 +2,39 @@
 target-version = "py37"
 
 lint.select = [
-  "A",   # flake8-builtins
-  "B",   # flake8-bugbear
-  "BLE", # flake8-blind-except
-  "C4",  # flake8-comprehensions
-  "C90", # McCabe cyclomatic complexity
-  "E",   # pycodestyle
-  "ERA", # eradicate
-  "EXE", # flake8-executable
-  "F",   # Pyflakes
-  "G",   # flake8-logging-format
-  "I",   # isort
-  "ICN", # flake8-import-conventions
-  "INP", # flake8-no-pep420
-  "INT", # flake8-gettext
-  "ISC", # flake8-implicit-str-concat
-  "N",   # pep8-naming
-  "PGH", # pygrep-hooks
-  "PIE", # flake8-pie
-  "PL",  # Pylint
-  "PT",  # flake8-pytest-style
-  "PTH", # flake8-use-pathlib
-  "PYI", # flake8-pyi
-  "RET", # flake8-return
-  "RSE", # flake8-raise
-  "RUF", # Ruff-specific rules
-  "S",   # flake8-bandit
-  "T10", # flake8-debugger
-  "TCH", # flake8-type-checking
-  "TID", # flake8-tidy-imports
-  "W",   # pycodestyle
-  "YTT", # flake8-2020
+  "A",     # flake8-builtins
+  "AIR",   # Airflow
+  "ASYNC", # flake8-async
+  "B",     # flake8-bugbear
+  "BLE",   # flake8-blind-except
+  "C4",    # flake8-comprehensions
+  "C90",   # McCabe cyclomatic complexity
+  "E",     # pycodestyle
+  "ERA",   # eradicate
+  "EXE",   # flake8-executable
+  "F",     # Pyflakes
+  "G",     # flake8-logging-format
+  "I",     # isort
+  "ICN",   # flake8-import-conventions
+  "INP",   # flake8-no-pep420
+  "INT",   # flake8-gettext
+  "ISC",   # flake8-implicit-str-concat
+  "N",     # pep8-naming
+  "PGH",   # pygrep-hooks
+  "PIE",   # flake8-pie
+  "PL",    # Pylint
+  "PT",    # flake8-pytest-style
+  "PTH",   # flake8-use-pathlib
+  "PYI",   # flake8-pyi
+  "RET",   # flake8-return
+  "RSE",   # flake8-raise
+  "RUF",   # Ruff-specific rules
+  "S",     # flake8-bandit
+  "T10",   # flake8-debugger
+  "TCH",   # flake8-type-checking
+  "TID",   # flake8-tidy-imports
+  "W",     # pycodestyle
+  "YTT",   # flake8-2020
   # "ANN",  # flake8-annotations
   # "ARG",  # flake8-unused-arguments
   # "COM",  # flake8-commas
@@ -67,7 +69,6 @@ lint.ignore = [
   "RET506",
   "RUF100",
   "S113",
-  "S311",
   "S314",
   "S603",
 ]

pysolr.py

@@ -1665,7 +1665,7 @@ def getRandomURL(self, collname, only_leader=False):
         hosts = self.getHosts(collname, only_leader=only_leader)
         if not hosts:
             raise SolrError("ZooKeeper returned no active shards!")
-        return "%s/%s" % (random.choice(hosts), collname)  # NOQA: B311
+        return "%s/%s" % (random.choice(hosts), collname)  # NOQA: S311
 
     def getLeaderURL(self, collname):
         return self.getRandomURL(collname, only_leader=True)

tests/test_client.py

@@ -565,7 +565,7 @@ def test_search(self):
         results = self.solr.search("doc")
         self.assertEqual(len(results), 3)
         # search should default to 'select' handler
-        args, kwargs = self.solr._send_request.call_args
+        args, _kwargs = self.solr._send_request.call_args
         self.assertTrue(args[1].startswith("select/?"))
 
         results = self.solr.search("example")
@@ -619,20 +619,20 @@ def test_multiple_search_handlers(self):
         # search should support custom handlers
         with self.assertRaises(SolrError):
             self.solr.search("doc", search_handler="fakehandler")
-        args, kwargs = self.solr._send_request.call_args
+        args, _kwargs = self.solr._send_request.call_args
         self.assertTrue(args[1].startswith("fakehandler"))
 
     def test_more_like_this(self):
         results = self.solr.more_like_this("id:doc_1", "text")
         self.assertEqual(len(results), 0)
         # more_like_this should default to 'mlt' handler
-        args, kwargs = self.solr._send_request.call_args
+        args, _kwargs = self.solr._send_request.call_args
         self.assertTrue(args[1].startswith("mlt/?"))
 
         # more_like_this should support custom handlers
         with self.assertRaises(SolrError):
             self.solr.more_like_this("id:doc_1", "text", handler="fakehandler")
-        args, kwargs = self.solr._send_request.call_args
+        args, _kwargs = self.solr._send_request.call_args
         self.assertTrue(args[1].startswith("fakehandler"))
 
     def test_suggest_terms(self):
@@ -654,13 +654,13 @@ def test_suggest_terms(self):
             },
         )
         # suggest_terms should default to 'mlt' handler
-        args, kwargs = self.solr._send_request.call_args
+        args, _kwargs = self.solr._send_request.call_args
         self.assertTrue(args[1].startswith("terms/?"))
 
         # suggest_terms should support custom handlers
         with self.assertRaises(SolrError):
             self.solr.suggest_terms("title", "", handler="fakehandler")
-        args, kwargs = self.solr._send_request.call_args
+        args, _kwargs = self.solr._send_request.call_args
         self.assertTrue(args[1].startswith("fakehandler"))
 
     def test__build_xml_doc(self):
@@ -768,17 +768,17 @@ def test_build_json_doc_matches_xml(self):
 
     def test__build_docs_plain(self):
         docs = [{"id": "doc_1", "title": "", "price": 12.59, "popularity": 10}]
-        solrapi, m, len_message = self.solr._build_docs(docs)
+        solrapi, _m, _len_message = self.solr._build_docs(docs)
         self.assertEqual(solrapi, "JSON")
 
     def test__build_docs_boost(self):
         docs = [{"id": "doc_1", "title": "", "price": 12.59, "popularity": 10}]
-        solrapi, m, len_message = self.solr._build_docs(docs, boost={"title": 10.0})
+        solrapi, _m, _len_message = self.solr._build_docs(docs, boost={"title": 10.0})
         self.assertEqual(solrapi, "XML")
 
     def test__build_docs_field_updates(self):
         docs = [{"id": "doc_1", "popularity": 10}]
-        solrapi, m, len_message = self.solr._build_docs(
+        solrapi, _m, _len_message = self.solr._build_docs(
             docs, fieldUpdates={"popularity": "inc"}
         )
         self.assertEqual(solrapi, "JSON")
@@ -795,7 +795,7 @@ def test_add(self):
             commit=True,
         )
         # add should default to 'update' handler
-        args, kwargs = self.solr._send_request.call_args
+        args, _kwargs = self.solr._send_request.call_args
         self.assertTrue(args[1].startswith("update/?"))
 
         self.assertEqual(len(self.solr.search("doc")), 5)
@@ -804,7 +804,7 @@ def test_add(self):
         # add should support custom handlers
         with self.assertRaises(SolrError):
             self.solr.add([], handler="fakehandler", commit=True)
-        args, kwargs = self.solr._send_request.call_args
+        args, _kwargs = self.solr._send_request.call_args
         self.assertTrue(args[1].startswith("fakehandler"))
 
     def test_add_with_boost(self):
@@ -927,7 +927,7 @@ def test_delete(self):
         self.assertEqual(len(self.solr.search("doc")), 3)
         self.solr.delete(id="doc_1", commit=True)
         # delete should default to 'update' handler
-        args, kwargs = self.solr._send_request.call_args
+        args, _kwargs = self.solr._send_request.call_args
         self.assertTrue(args[1].startswith("update/?"))
 
         self.assertEqual(len(self.solr.search("doc")), 2)
@@ -971,7 +971,7 @@ def leaf_doc(doc):
         self.assertEqual(len(self.solr.search(leaf_q)), len(to_delete_docs))
         # Extract a random doc from the list, to later check it wasn't deleted.
         graced_doc_id = to_delete_ids.pop(
-            random.randint(0, len(to_delete_ids) - 1)  # NOQA: B311
+            random.randint(0, len(to_delete_ids) - 1)  # NOQA: S311
         )
         self.solr.delete(id=to_delete_ids, commit=True)
         # There should be only one left, our graced id
@@ -993,7 +993,7 @@ def leaf_doc(doc):
         # delete should support custom handlers
         with self.assertRaises(SolrError):
             self.solr.delete(id="doc_1", handler="fakehandler", commit=True)
-        args, kwargs = self.solr._send_request.call_args
+        args, _kwargs = self.solr._send_request.call_args
         self.assertTrue(args[1].startswith("fakehandler"))
 
     def test_commit(self):
@@ -1002,7 +1002,7 @@ def test_commit(self):
         self.assertEqual(len(self.solr.search("doc")), 3)
         self.solr.commit()
         # commit should default to 'update' handler
-        args, kwargs = self.solr._send_request.call_args
+        args, _kwargs = self.solr._send_request.call_args
         self.assertTrue(args[1].startswith("update/?"))
         self.assertEqual(len(self.solr.search("doc")), 4)
 
@@ -1031,7 +1031,7 @@ def test_overwrite(self):
         # commit should support custom handlers
         with self.assertRaises(SolrError):
             self.solr.commit(handler="fakehandler")
-        args, kwargs = self.solr._send_request.call_args
+        args, _kwargs = self.solr._send_request.call_args
         self.assertTrue(args[1].startswith("fakehandler"))
 
     def test_optimize(self):
@@ -1041,14 +1041,14 @@ def test_optimize(self):
         self.assertEqual(len(self.solr.search("doc")), 3)
         self.solr.optimize()
         # optimize should default to 'update' handler
-        args, kwargs = self.solr._send_request.call_args
+        args, _kwargs = self.solr._send_request.call_args
         self.assertTrue(args[1].startswith("update/?"))
         self.assertEqual(len(self.solr.search("doc")), 4)
 
         # optimize should support custom handlers
         with self.assertRaises(SolrError):
             self.solr.optimize(handler="fakehandler")
-        args, kwargs = self.solr._send_request.call_args
+        args, _kwargs = self.solr._send_request.call_args
         self.assertTrue(args[1].startswith("fakehandler"))
 
     def test_extract(self):
@@ -1067,13 +1067,13 @@ def test_extract(self):
         fake_f.name = "test.html"
         extracted = self.solr.extract(fake_f)
         # extract should default to 'update/extract' handler
-        args, kwargs = self.solr._send_request.call_args
+        args, _kwargs = self.solr._send_request.call_args
         self.assertTrue(args[1].startswith("update/extract"))
 
         # extract should support custom handlers
         with self.assertRaises(SolrError):
             self.solr.extract(fake_f, handler="fakehandler")
-        args, kwargs = self.solr._send_request.call_args
+        args, _kwargs = self.solr._send_request.call_args
         self.assertTrue(args[1].startswith("fakehandler"))
 
         # Verify documented response structure:
@@ -1110,13 +1110,13 @@ def test_extract_special_char_in_filename(self):
         fake_f.name = "test☃.html"
         extracted = self.solr.extract(fake_f)
         # extract should default to 'update/extract' handler
-        args, kwargs = self.solr._send_request.call_args
+        args, _kwargs = self.solr._send_request.call_args
         self.assertTrue(args[1].startswith("update/extract"))
 
         # extract should support custom handlers
         with self.assertRaises(SolrError):
             self.solr.extract(fake_f, handler="fakehandler")
-        args, kwargs = self.solr._send_request.call_args
+        args, _kwargs = self.solr._send_request.call_args
         self.assertTrue(args[1].startswith("fakehandler"))
 
         # Verify documented response structure:
@@ -1151,26 +1151,26 @@ def test_request_handler(self):
         self.solr.use_qt_param = True
 
         self.solr.search("my query")
-        args, kwargs = self.solr._send_request.call_args
+        args, _kwargs = self.solr._send_request.call_args
         self.assertTrue(args[1].startswith("select"))
 
         self.solr.search("my", search_handler="/autocomplete")
-        args, kwargs = self.solr._send_request.call_args
+        args, _kwargs = self.solr._send_request.call_args
         self.assertTrue(args[1].startswith("select"))
         self.assertGreaterEqual(args[1].find("qt=%2Fautocomplete"), 0)
 
         self.solr.search_handler = "/autocomplete"
 
         self.solr.search("my")
-        args, kwargs = self.solr._send_request.call_args
+        args, _kwargs = self.solr._send_request.call_args
         self.assertTrue(args[1].startswith("select"))
         self.assertGreaterEqual(args[1].find("qt=%2Fautocomplete"), 0)
 
         self.solr.use_qt_param = False
         # will change the path, so expect a 404
         with self.assertRaises(SolrError):
             self.solr.search("my")
-        args, kwargs = self.solr._send_request.call_args
+        args, _kwargs = self.solr._send_request.call_args
         self.assertTrue(args[1].startswith("/autocomplete"))
         self.assertLess(args[1].find("qt=%2Fautocomplete"), 0)
 

tox.ini

@@ -1,5 +1,5 @@
 [tox]
-envlist = py39,py310,py311,py312,py313,pypy3
+envlist = py39,py310,py311,py312,py313,py314,py314t,pypy3
 
 [base]
 deps =