adds changes for complete annotation of functions.

Author: bot2x

testing/test_warnings.py

@@ -39,7 +39,7 @@ def foo():
 
 
 @pytest.mark.filterwarnings("default")
-def test_normal_flow(pytester: Pytester, pyfile_with_warnings):
+def test_normal_flow(pytester: Pytester, pyfile_with_warnings) -> None:
     """Check that the warnings section is displayed."""
     result = pytester.runpytest(pyfile_with_warnings)
     result.stdout.fnmatch_lines(
@@ -56,7 +56,7 @@ def test_normal_flow(pytester: Pytester, pyfile_with_warnings):
 
 
 @pytest.mark.filterwarnings("always")
-def test_setup_teardown_warnings(pytester: Pytester):
+def test_setup_teardown_warnings(pytester: Pytester) -> None:
     pytester.makepyfile(
         """
         import warnings
@@ -86,7 +86,7 @@ def test_func(fix):
 
 
 @pytest.mark.parametrize("method", ["cmdline", "ini"])
-def test_as_errors(pytester: Pytester, pyfile_with_warnings, method):
+def test_as_errors(pytester: Pytester, pyfile_with_warnings, method) -> None:
     args = ("-W", "error") if method == "cmdline" else ()
     if method == "ini":
         pytester.makeini(
@@ -108,7 +108,7 @@ def test_as_errors(pytester: Pytester, pyfile_with_warnings, method):
 
 
 @pytest.mark.parametrize("method", ["cmdline", "ini"])
-def test_ignore(pytester: Pytester, pyfile_with_warnings, method):
+def test_ignore(pytester: Pytester, pyfile_with_warnings, method) -> None:
     args = ("-W", "ignore") if method == "cmdline" else ()
     if method == "ini":
         pytester.makeini(
@@ -124,7 +124,7 @@ def test_ignore(pytester: Pytester, pyfile_with_warnings, method):
 
 
 @pytest.mark.filterwarnings("always")
-def test_unicode(pytester: Pytester):
+def test_unicode(pytester: Pytester) -> None:
     pytester.makepyfile(
         """
         import warnings
@@ -150,7 +150,7 @@ def test_func(fix):
     )
 
 
-def test_works_with_filterwarnings(pytester: Pytester):
+def test_works_with_filterwarnings(pytester: Pytester) -> None:
     """Ensure our warnings capture does not mess with pre-installed filters (#2430)."""
     pytester.makepyfile(
         """
@@ -175,7 +175,7 @@ def test_my_warning(self):
 
 
 @pytest.mark.parametrize("default_config", ["ini", "cmdline"])
-def test_filterwarnings_mark(pytester: Pytester, default_config):
+def test_filterwarnings_mark(pytester: Pytester, default_config) -> None:
     """Test ``filterwarnings`` mark works and takes precedence over command
     line and ini options."""
     if default_config == "ini":
@@ -206,7 +206,7 @@ def test_show_warning():
     result.stdout.fnmatch_lines(["*= 1 failed, 2 passed, 1 warning in *"])
 
 
-def test_non_string_warning_argument(pytester: Pytester):
+def test_non_string_warning_argument(pytester: Pytester) -> None:
     """Non-str argument passed to warning breaks pytest (#2956)"""
     pytester.makepyfile(
         """\
@@ -221,7 +221,7 @@ def test():
     result.stdout.fnmatch_lines(["*= 1 passed, 1 warning in *"])
 
 
-def test_filterwarnings_mark_registration(pytester: Pytester):
+def test_filterwarnings_mark_registration(pytester: Pytester) -> None:
     """Ensure filterwarnings mark is registered"""
     pytester.makepyfile(
         """
@@ -237,7 +237,7 @@ def test_func():
 
 
 @pytest.mark.filterwarnings("always")
-def test_warning_captured_hook(pytester: Pytester):
+def test_warning_captured_hook(pytester: Pytester) -> None:
     pytester.makeconftest(
         """
         def pytest_configure(config):
@@ -298,7 +298,7 @@ def pytest_warning_recorded(self, warning_message, when, nodeid, location):
 
 
 @pytest.mark.filterwarnings("always")
-def test_collection_warnings(pytester: Pytester):
+def test_collection_warnings(pytester: Pytester) -> None:
     """Check that we also capture warnings issued during test collection (#3251)."""
     pytester.makepyfile(
         """
@@ -322,7 +322,7 @@ def test_foo():
 
 
 @pytest.mark.filterwarnings("always")
-def test_mark_regex_escape(pytester: Pytester):
+def test_mark_regex_escape(pytester: Pytester) -> None:
     """@pytest.mark.filterwarnings should not try to escape regex characters (#3936)"""
     pytester.makepyfile(
         r"""
@@ -339,7 +339,7 @@ def test_foo():
 
 @pytest.mark.filterwarnings("default")
 @pytest.mark.parametrize("ignore_pytest_warnings", ["no", "ini", "cmdline"])
-def test_hide_pytest_internal_warnings(pytester: Pytester, ignore_pytest_warnings):
+def test_hide_pytest_internal_warnings(pytester: Pytester, ignore_pytest_warnings) -> None:
     """Make sure we can ignore internal pytest warnings using a warnings filter."""
     pytester.makepyfile(
         """
@@ -378,7 +378,7 @@ def test_bar():
 
 
 @pytest.mark.parametrize("ignore_on_cmdline", [True, False])
-def test_option_precedence_cmdline_over_ini(pytester: Pytester, ignore_on_cmdline):
+def test_option_precedence_cmdline_over_ini(pytester: Pytester, ignore_on_cmdline) -> None:
     """Filters defined in the command-line should take precedence over filters in ini files (#3946)."""
     pytester.makeini(
         """
@@ -401,7 +401,7 @@ def test():
         result.stdout.fnmatch_lines(["* 1 failed in*"])
 
 
-def test_option_precedence_mark(pytester: Pytester):
+def test_option_precedence_mark(pytester: Pytester) -> None:
     """Filters defined by marks should always take precedence (#3946)."""
     pytester.makeini(
         """
@@ -427,7 +427,7 @@ class TestDeprecationWarningsByDefault:
     from pytest's own test suite
     """
 
-    def create_file(self, pytester: Pytester, mark=""):
+    def create_file(self, pytester: Pytester, mark="") -> None:
         pytester.makepyfile(
             """
             import pytest, warnings
@@ -443,7 +443,7 @@ def test_foo():
         )
 
     @pytest.mark.parametrize("customize_filters", [True, False])
-    def test_shown_by_default(self, pytester: Pytester, customize_filters):
+    def test_shown_by_default(self, pytester: Pytester, customize_filters) -> None:
         """Show deprecation warnings by default, even if user has customized the warnings filters (#4013)."""
         self.create_file(pytester)
         if customize_filters:
@@ -464,7 +464,7 @@ def test_shown_by_default(self, pytester: Pytester, customize_filters):
             ]
         )
 
-    def test_hidden_by_ini(self, pytester: Pytester):
+    def test_hidden_by_ini(self, pytester: Pytester) -> None:
         self.create_file(pytester)
         pytester.makeini(
             """
@@ -477,7 +477,7 @@ def test_hidden_by_ini(self, pytester: Pytester):
         result = pytester.runpytest_subprocess()
         assert WARNINGS_SUMMARY_HEADER not in result.stdout.str()
 
-    def test_hidden_by_mark(self, pytester: Pytester):
+    def test_hidden_by_mark(self, pytester: Pytester) -> None:
         """Should hide the deprecation warning from the function, but the warning during collection should
         be displayed normally.
         """
@@ -494,7 +494,7 @@ def test_hidden_by_mark(self, pytester: Pytester):
             ]
         )
 
-    def test_hidden_by_cmdline(self, pytester: Pytester):
+    def test_hidden_by_cmdline(self, pytester: Pytester) -> None:
         self.create_file(pytester)
         result = pytester.runpytest_subprocess(
             "-W",
@@ -504,7 +504,7 @@ def test_hidden_by_cmdline(self, pytester: Pytester):
         )
         assert WARNINGS_SUMMARY_HEADER not in result.stdout.str()
 
-    def test_hidden_by_system(self, pytester: Pytester, monkeypatch):
+    def test_hidden_by_system(self, pytester: Pytester, monkeypatch) -> None:
         self.create_file(pytester)
         monkeypatch.setenv("PYTHONWARNINGS", "once::UserWarning")
         result = pytester.runpytest_subprocess()
@@ -515,7 +515,7 @@ def test_hidden_by_system(self, pytester: Pytester, monkeypatch):
 @pytest.mark.skip(
     reason="This test should be enabled again before pytest 7.0 is released"
 )
-def test_deprecation_warning_as_error(pytester: Pytester, change_default):
+def test_deprecation_warning_as_error(pytester: Pytester, change_default) -> None:
     """This ensures that PytestDeprecationWarnings raised by pytest are turned into errors.
 
     This test should be enabled as part of each major release, and skipped again afterwards
@@ -552,10 +552,10 @@ def test():
 
 class TestAssertionWarnings:
     @staticmethod
-    def assert_result_warns(result, msg):
+    def assert_result_warns(result, msg) -> None:
         result.stdout.fnmatch_lines(["*PytestAssertRewriteWarning: %s*" % msg])
 
-    def test_tuple_warning(self, pytester: Pytester):
+    def test_tuple_warning(self, pytester: Pytester) -> None:
         pytester.makepyfile(
             """\
             def test_foo():
@@ -568,7 +568,7 @@ def test_foo():
         )
 
 
-def test_warnings_checker_twice():
+def test_warnings_checker_twice() -> None:
     """Issue #4617"""
     expectation = pytest.warns(UserWarning)
     with expectation:
@@ -579,7 +579,7 @@ def test_warnings_checker_twice():
 
 @pytest.mark.filterwarnings("ignore::pytest.PytestExperimentalApiWarning")
 @pytest.mark.filterwarnings("always")
-def test_group_warnings_by_message(pytester: Pytester):
+def test_group_warnings_by_message(pytester: Pytester) -> None:
     pytester.copy_example("warnings/test_group_warnings_by_message.py")
     result = pytester.runpytest()
     result.stdout.fnmatch_lines(
@@ -611,7 +611,7 @@ def test_group_warnings_by_message(pytester: Pytester):
 
 @pytest.mark.filterwarnings("ignore::pytest.PytestExperimentalApiWarning")
 @pytest.mark.filterwarnings("always")
-def test_group_warnings_by_message_summary(pytester: Pytester):
+def test_group_warnings_by_message_summary(pytester: Pytester) -> None:
     pytester.copy_example("warnings/test_group_warnings_by_message_summary")
     pytester.syspathinsert()
     result = pytester.runpytest()
@@ -634,7 +634,7 @@ def test_group_warnings_by_message_summary(pytester: Pytester):
     )
 
 
-def test_pytest_configure_warning(pytester: Pytester, recwarn):
+def test_pytest_configure_warning(pytester: Pytester, recwarn) -> None:
     """Issue 5115."""
     pytester.makeconftest(
         """
@@ -668,7 +668,7 @@ def pytest_warning_recorded(cls, warning_message, when, nodeid, location):
 
         return CapturedWarnings
 
-    def test_issue4445_rewrite(self, pytester: Pytester, capwarn):
+    def test_issue4445_rewrite(self, pytester: Pytester, capwarn) -> None:
         """#4445: Make sure the warning points to a reasonable location
         See origin of _issue_warning_captured at: _pytest.assertion.rewrite.py:241
         """
@@ -694,7 +694,7 @@ def test_issue4445_rewrite(self, pytester: Pytester, capwarn):
         assert func == "<module>"  # the above conftest.py
         assert lineno == 4
 
-    def test_issue4445_preparse(self, pytester: Pytester, capwarn):
+    def test_issue4445_preparse(self, pytester: Pytester, capwarn) -> None:
         """#4445: Make sure the warning points to a reasonable location
         See origin of _issue_warning_captured at: _pytest.config.__init__.py:910
         """
@@ -729,7 +729,7 @@ def test_conftest_warning_captured(self, pytester: Pytester) -> None:
             ["conftest.py:2", "*UserWarning: my custom warning*"]
         )
 
-    def test_issue4445_import_plugin(self, pytester: Pytester, capwarn):
+    def test_issue4445_import_plugin(self, pytester: Pytester, capwarn) -> None:
         """#4445: Make sure the warning points to a reasonable location"""
         pytester.makepyfile(
             some_plugin="""
@@ -751,7 +751,7 @@ def test_issue4445_import_plugin(self, pytester: Pytester, capwarn):
         assert f"config{os.sep}__init__.py" in file
         assert func == "_warn_about_skipped_plugins"
 
-    def test_issue4445_issue5928_mark_generator(self, pytester: Pytester):
+    def test_issue4445_issue5928_mark_generator(self, pytester: Pytester) -> None:
         """#4445 and #5928: Make sure the warning from an unknown mark points to
         the test file where this mark is used.
         """