Merge pull request #645 from nose-devs/update-precommit-hooks

Update pre-commit config + apply changes

Author: sirosen

.pre-commit-config.yaml

@@ -1,40 +1,40 @@
 repos:
 - repo: https://github.com/pre-commit/pre-commit-hooks.git
-  rev: v5.0.0
+  rev: v6.0.0
   hooks:
     - id: check-merge-conflict
     - id: check-toml
     - id: check-yaml
     - id: end-of-file-fixer
     - id: trailing-whitespace
 - repo: https://github.com/python-jsonschema/check-jsonschema
-  rev: 0.33.0
+  rev: 0.36.1
   hooks:
     - id: check-github-workflows
     - id: check-dependabot
     - id: check-readthedocs
 - repo: https://github.com/psf/black
-  rev: 25.1.0
+  rev: 26.1.0
   hooks:
   - id: black
     language_version: python3
 - repo: https://github.com/PyCQA/isort
-  rev: 6.0.1
+  rev: 7.0.0
   hooks:
     - id: isort
 - repo: https://github.com/asottile/pyupgrade
-  rev: v3.19.1
+  rev: v3.21.2
   hooks:
     - id: pyupgrade
       args: ["--py37-plus"]
 - repo: https://github.com/PyCQA/flake8
-  rev: 7.2.0
+  rev: 7.3.0
   hooks:
     - id: flake8
       additional_dependencies:
-        - 'flake8-bugbear==24.12.12'
-        - 'flake8-comprehensions==3.16.0'
-        - 'flake8-typing-as-t==1.0.0'
+        - 'flake8-bugbear==25.11.29'
+        - 'flake8-comprehensions==3.17.0'
+        - 'flake8-typing-as-t==1.1.0'
 - repo: https://github.com/sirosen/slyp
   rev: 0.8.2
   hooks:
@@ -49,7 +49,8 @@ repos:
         - "-L"
         - "assertIn"
 - repo: https://github.com/tox-dev/pyproject-fmt
-  rev: v2.5.1
+  rev: v2.16.0
   hooks:
     - id: pyproject-fmt
+      args: ["--max-supported-python", "3.13"]
       additional_dependencies: ["tox>=4.9"]

nose2/plugins/loader/parameters.py

@@ -169,7 +169,7 @@ def _method(self, method=method, argSet=argSet):
                     return method(self, *argSet)
 
                 _method = functools.update_wrapper(_method, method)
-                delattr(_method, "paramList")
+                del _method.paramList
                 setattr(testCaseClass, method_name, _method)
             names.append(method_name)
         return names
@@ -188,7 +188,7 @@ def func(argSet=argSet, obj=obj):
                 return obj(*argSet)
 
             func = functools.update_wrapper(func, obj)
-            delattr(func, "paramList")
+            del func.paramList
             name = f"{obj.__module__}.{obj.__name__}"
             func_name = util.name_from_args(name, index, argSet)
             yield util.transplant_class(ParamsFunctionCase, obj.__module__)(

nose2/plugins/prettyassert.py

@@ -137,9 +137,9 @@ def _get_inspection_info(trace):
     - statement which failed (which can be garbage -- don't trust it)
     - can_tokenize: a bool indicating that the lines of source can be parsed
     """
-    (frame, fname, lineno, funcname, context, ctx_index) = inspect.getinnerframes(
-        trace
-    )[-1]
+    frame, fname, lineno, funcname, context, ctx_index = inspect.getinnerframes(trace)[
+        -1
+    ]
     original_source_lines, firstlineno = inspect.getsourcelines(frame)
 
     # truncate to the code in this frame to remove anything after current

nose2/suite.py

@@ -105,7 +105,7 @@ def tearDownTest(self, test):
 
         event = events.StopLayerTeardownTestEvent(self.layer, test)
         self.session.hooks.stopLayerTeardownTest(event)
-        delattr(test, "_layer_wasSetUp")
+        del test._layer_wasSetUp
 
     def tearDown(self):
         if self.layer is None:

nose2/tests/functional/support/such/test_such.py

@@ -124,8 +124,8 @@ def test_setup(case):
 
             @it.has_test_teardown
             def test_teardown(case):
-                delattr(it, "is_funny")
-                delattr(case, "is_funny")
+                del it.is_funny
+                del case.is_funny
 
             @it.should("do something else")  # type: ignore[no-redef]
             def test(case):  # noqa: F811

nose2/tests/functional/support/toml/a/pyproject.toml

@@ -1,5 +1,3 @@
-[tool.nose2.a]
-a = 1
-
-[tool.nose2.unittest]
-plugins = "plugin_a"
+[tool.nose2]
+a.a = 1
+unittest.plugins = "plugin_a"

nose2/tests/functional/support/toml/b/pyproject.toml

@@ -1,5 +1,5 @@
-[tool.nose2.b]
-b = """
+[tool.nose2]
+b.b = """
 4
 5
 """

nose2/tests/functional/test_collect_plugin.py

@@ -17,8 +17,6 @@ def test_collect_tests_in_package(self):
 
 
 # expectations
-EXPECT_LAYOUT1 = re.compile(
-    r"""Ran 25 tests in \d.\d+s
+EXPECT_LAYOUT1 = re.compile(r"""Ran 25 tests in \d.\d+s
 
-OK"""
-)
+OK""")

nose2/tests/functional/test_mp_plugin.py

@@ -82,20 +82,20 @@ def test_flatten_respects_class_fixtures(self):
 
     def test_conn_prep(self):
         self.plugin.bind_host = None
-        (parent_conn, child_conn) = self.plugin._prepConns()
-        (parent_pipe, child_pipe) = multiprocessing.Pipe()
+        parent_conn, child_conn = self.plugin._prepConns()
+        parent_pipe, child_pipe = multiprocessing.Pipe()
         self.assertIsInstance(parent_conn, type(parent_pipe))
         self.assertIsInstance(child_conn, type(child_pipe))
 
         self.plugin.bind_host = "127.0.0.1"
         self.plugin.bind_port = 0
-        (parent_conn, child_conn) = self.plugin._prepConns()
+        parent_conn, child_conn = self.plugin._prepConns()
         self.assertIsInstance(parent_conn, connection.Listener)
         self.assertIsInstance(child_conn, tuple)
         self.assertEqual(parent_conn.address, child_conn[:2])
 
     def test_conn_accept(self):
-        (parent_conn, child_conn) = multiprocessing.Pipe()
+        parent_conn, child_conn = multiprocessing.Pipe()
         self.assertEqual(self.plugin._acceptConns(parent_conn), parent_conn)
 
         listener = connection.Listener(("127.0.0.1", 0))
@@ -161,27 +161,23 @@ def test_dispatch_tests_receive_events(self):
                         {
                             "outcome": "failed",
                             "expected": False,
-                            "metadata": {
-                                "stdout": """\
+                            "metadata": {"stdout": """\
 -------------------- >> begin captured stdout << ---------------------
 Hello stdout
 
---------------------- >> end captured stdout << ----------------------"""
-                            },
+--------------------- >> end captured stdout << ----------------------"""},
                         },
                     ),
                     (
                         "testOutcome",
                         {
                             "outcome": "failed",
                             "expected": False,
-                            "metadata": {
-                                "stdout": """\
+                            "metadata": {"stdout": """\
 -------------------- >> begin captured stdout << ---------------------
 Hello stdout
 
---------------------- >> end captured stdout << ----------------------"""
-                            },
+--------------------- >> end captured stdout << ----------------------"""},
                         },
                     ),
                     ("stopTest", {}),

nose2/tests/unit/test_collector.py

@@ -12,14 +12,10 @@ class TestCollector(TestCase):
 
     def test_collector_completes_with_no_tests(self):
         with open("unittest.cfg", "w") as ut_file:
-            ut_file.write(
-                dedent(
-                    """
+            ut_file.write(dedent("""
                 [unittest]
                 quiet = true
-            """
-                )
-            )
+            """))
         test = collector.collector()
         with RedirectStdStreams() as redir:
             self.assertRaises(SystemExit, test.run, None)