intermediate changes

ref:614ed510ddd3cdf86a8c5dbf19afd113397e0172

1 files changed, 8 insertions, 7 deletions

diff --git a/contrib/python/pytest/py3/_pytest/nose.py b/contrib/python/pytest/py3/_pytest/nose.py
index d6f3c2b224..bb8f99772a 100644
--- a/contrib/python/pytest/py3/_pytest/nose.py
+++ b/contrib/python/pytest/py3/_pytest/nose.py
@@ -1,16 +1,17 @@
-""" run test suites written for nose. """
+"""Run testsuites written for nose."""
 from _pytest import python
 from _pytest import unittest
 from _pytest.config import hookimpl
+from _pytest.nodes import Item
 
 
 @hookimpl(trylast=True)
 def pytest_runtest_setup(item):
     if is_potential_nosetest(item):
         if not call_optional(item.obj, "setup"):
-            # call module level setup if there is no object level one
+            # Call module level setup if there is no object level one.
             call_optional(item.parent.obj, "setup")
-        # XXX this implies we only call teardown when setup worked
+        # XXX This implies we only call teardown when setup worked.
         item.session._setupstate.addfinalizer((lambda: teardown_nose(item)), item)
 
 
@@ -20,9 +21,9 @@ def teardown_nose(item):
             call_optional(item.parent.obj, "teardown")
 
 
-def is_potential_nosetest(item):
-    # extra check needed since we do not do nose style setup/teardown
-    # on direct unittest style classes
+def is_potential_nosetest(item: Item) -> bool:
+    # Extra check needed since we do not do nose style setup/teardown
+    # on direct unittest style classes.
     return isinstance(item, python.Function) and not isinstance(
         item, unittest.TestCaseFunction
     )
@@ -33,6 +34,6 @@ def call_optional(obj, name):
     isfixture = hasattr(method, "_pytestfixturefunction")
     if method is not None and not isfixture and callable(method):
         # If there's any problems allow the exception to raise rather than
-        # silently ignoring them
+        # silently ignoring them.
         method()
         return True