ansible-core 2.18.5rc1__py3-none-any.whl → 2.19.0b2__py3-none-any.whl

ansible_test/_util/controller/sanity/pylint/config/ansible-test-target.cfg

@@ -1,6 +1,12 @@
-[MESSAGES CONTROL]
+[pylint]
+
+max-line-length=160
+
+load-plugins=
+    pylint.extensions.docstyle,
 
 disable=
+    docstring-first-line-empty,
     consider-using-f-string,  # Python 2.x support still required
     cyclic-import,  # consistent results require running with --jobs 1 and testing all files
     deprecated-argument,  # results vary by Python version
@@ -28,8 +34,6 @@ disable=
     use-dict-literal,  # ignoring as a common style issue
     useless-return,  # complains about returning None when the return type is optional
 
-[BASIC]
-
 bad-names=
     _,
     bar,
@@ -52,8 +56,6 @@ attr-rgx=[a-z_][a-z0-9_]{1,40}$
 method-rgx=[a-z_][a-z0-9_]{1,40}$
 function-rgx=[a-z_][a-z0-9_]{1,40}$
 
-[IMPORTS]
-
 preferred-modules =
     distutils.version:ansible.module_utils.compat.version,
 

ansible_test/_util/controller/sanity/pylint/config/ansible-test.cfg

@@ -1,6 +1,12 @@
-[MESSAGES CONTROL]
+[pylint]
+
+max-line-length=160
+
+load-plugins=
+    pylint.extensions.docstyle,
 
 disable=
+    docstring-first-line-empty,
     consider-using-f-string,  # many occurrences
     cyclic-import,  # consistent results require running with --jobs 1 and testing all files
     deprecated-argument,  # results vary by Python version
@@ -27,8 +33,6 @@ disable=
     unspecified-encoding,  # always run with UTF-8 encoding enforced
     useless-return,  # complains about returning None when the return type is optional
 
-[BASIC]
-
 bad-names=
     _,
     bar,
@@ -55,8 +59,6 @@ function-rgx=[a-z_][a-z0-9_]{1,40}$
 # See: https://github.com/PyCQA/pylint/pull/7322
 typevar-rgx=^_{0,2}(?:[^\W\da-z_]+|(?:[^\W\da-z_]+[^\WA-Z_]+)+T?(?<!Type))(?:_co(?:ntra)?)?$
 
-[IMPORTS]
-
 preferred-modules =
     distutils.version:ansible.module_utils.compat.version,
 

ansible_test/_util/controller/sanity/pylint/config/code-smell.cfg

@@ -1,6 +1,12 @@
-[MESSAGES CONTROL]
+[pylint]
+
+max-line-length=160
+
+load-plugins=
+    pylint.extensions.docstyle,
 
 disable=
+    docstring-first-line-empty,
     consider-using-f-string,  # many occurrences
     cyclic-import,  # consistent results require running with --jobs 1 and testing all files
     deprecated-argument,  # results vary by Python version
@@ -26,8 +32,6 @@ disable=
     unspecified-encoding,  # always run with UTF-8 encoding enforced
     useless-return,  # complains about returning None when the return type is optional
 
-[BASIC]
-
 bad-names=
     _,
     bar,
@@ -51,8 +55,6 @@ method-rgx=[a-z_][a-z0-9_]{1,40}$
 function-rgx=[a-z_][a-z0-9_]{1,40}$
 module-rgx=[a-z_][a-z0-9_-]{2,40}$
 
-[IMPORTS]
-
 preferred-modules =
     distutils.version:ansible.module_utils.compat.version,
 

ansible_test/_util/controller/sanity/pylint/config/collection.cfg

@@ -1,4 +1,6 @@
-[MESSAGES CONTROL]
+[pylint]
+
+max-line-length=160
 
 disable=
     abstract-method,
@@ -127,8 +129,6 @@ disable=
     wrong-import-order,
     wrong-import-position,
 
-[BASIC]
-
 bad-names=
     _,
     bar,
@@ -145,7 +145,5 @@ good-names=
     k,
     Run,
 
-[TYPECHECK]
-
 ignored-modules=
     _MovedItems,

ansible_test/_util/controller/sanity/pylint/config/default.cfg

@@ -1,6 +1,12 @@
-[MESSAGES CONTROL]
+[pylint]
+
+max-line-length=160
+
+load-plugins=
+    pylint.extensions.docstyle,
 
 disable=
+    docstring-first-line-empty,
     import-outside-toplevel,  # common pattern in ansible related code
     abstract-method,
     access-member-before-definition,
@@ -118,8 +124,6 @@ disable=
     wrong-import-order,
     wrong-import-position,
 
-[BASIC]
-
 bad-names=
     _,
     bar,
@@ -136,12 +140,8 @@ good-names=
     k,
     Run,
 
-[TYPECHECK]
-
 ignored-modules=
     _MovedItems,
 
-[IMPORTS]
-
 preferred-modules =
     distutils.version:ansible.module_utils.compat.version,

ansible_test/_util/controller/sanity/pylint/plugins/deprecated.py

@@ -1,4 +1,5 @@
 """Ansible specific plyint plugin for checking deprecations."""
+
 # (c) 2018, Matt Martz <matt@sivel.net>
 # GNU General Public License v3.0+ (see COPYING or https://www.gnu.org/licenses/gpl-3.0.txt)
 # -*- coding: utf-8 -*-
@@ -12,16 +13,6 @@ from tokenize import COMMENT, TokenInfo
 
 import astroid
 
-# support pylint 2.x and 3.x -- remove when supporting only 3.x
-try:
-    from pylint.interfaces import IAstroidChecker, ITokenChecker
-except ImportError:
-    class IAstroidChecker:
-        """Backwards compatibility for 2.x / 3.x support."""
-
-    class ITokenChecker:
-        """Backwards compatibility for 2.x / 3.x support."""
-
 try:
     from pylint.checkers.utils import check_messages
 except ImportError:
@@ -151,7 +142,6 @@ class AnsibleDeprecatedChecker(BaseChecker):
     has not passed or met the time for removal
     """
 
-    __implements__ = (IAstroidChecker,)
     name = 'deprecated'
     msgs = MSGS
 
@@ -296,8 +286,6 @@ class AnsibleDeprecatedCommentChecker(BaseTokenChecker):
     has not passed or met the time for removal
     """
 
-    __implements__ = (ITokenChecker,)
-
     name = 'deprecated-comment'
     msgs = {
         'E9601': ("Deprecated core version (%r) found: %s",

ansible_test/_util/controller/sanity/pylint/plugins/hide_unraisable.py

@@ -1,4 +1,5 @@
 """Temporary plugin to prevent stdout noise pollution from finalization of abandoned generators."""
+
 from __future__ import annotations
 
 import sys

ansible_test/_util/controller/sanity/pylint/plugins/string_format.py

@@ -1,4 +1,5 @@
 """Ansible specific pylint plugin for checking format string usage."""
+
 # (c) 2018, Matt Martz <matt@sivel.net>
 # GNU General Public License v3.0+ (see COPYING or https://www.gnu.org/licenses/gpl-3.0.txt)
 # -*- coding: utf-8 -*-
@@ -6,13 +7,6 @@ from __future__ import annotations
 
 import astroid
 
-# support pylint 2.x and 3.x -- remove when supporting only 3.x
-try:
-    from pylint.interfaces import IAstroidChecker
-except ImportError:
-    class IAstroidChecker:
-        """Backwards compatibility for 2.x / 3.x support."""
-
 try:
     from pylint.checkers.utils import check_messages
 except ImportError:
@@ -38,7 +32,6 @@ class AnsibleStringFormatChecker(BaseChecker):
     is valid and the arguments match the format string.
     """
 
-    __implements__ = (IAstroidChecker,)
     name = 'string'
     msgs = MSGS
 

ansible_test/_util/controller/sanity/pylint/plugins/unwanted.py

@@ -1,4 +1,5 @@
 """A plugin for pylint to identify imports and functions which should not be used."""
+
 from __future__ import annotations
 
 import os
@@ -6,13 +7,6 @@ import typing as t
 
 import astroid
 
-# support pylint 2.x and 3.x -- remove when supporting only 3.x
-try:
-    from pylint.interfaces import IAstroidChecker
-except ImportError:
-    class IAstroidChecker:
-        """Backwards compatibility for 2.x / 3.x support."""
-
 from pylint.checkers import BaseChecker
 
 ANSIBLE_TEST_MODULES_PATH = os.environ['ANSIBLE_TEST_MODULES_PATH']
@@ -63,7 +57,6 @@ def is_module_path(path):  # type: (str) -> bool
 
 class AnsibleUnwantedChecker(BaseChecker):
     """Checker for unwanted imports and functions."""
-    __implements__ = (IAstroidChecker,)
 
     name = 'unwanted'
 

ansible_test/_util/controller/sanity/validate-modules/validate_modules/main.py

@@ -65,7 +65,7 @@ def setup_collection_loader():
 setup_collection_loader()
 
 from ansible import __version__ as ansible_version
-from ansible.executor.module_common import REPLACER_WINDOWS, NEW_STYLE_PYTHON_MODULE_RE
+from ansible.executor.module_common import REPLACER_WINDOWS as _REPLACER_WINDOWS, NEW_STYLE_PYTHON_MODULE_RE
 from ansible.module_utils.common.collections import is_iterable
 from ansible.module_utils.common.parameters import DEFAULT_TYPE_VALIDATORS
 from ansible.module_utils.compat.version import StrictVersion, LooseVersion
@@ -90,7 +90,7 @@ from .utils import CaptureStd, NoArgsAnsibleModule, compare_unordered_lists, par
 TRY_EXCEPT = ast.Try
 # REPLACER_WINDOWS from ansible.executor.module_common is byte
 # string but we need unicode for Python 3
-REPLACER_WINDOWS = REPLACER_WINDOWS.decode('utf-8')
+REPLACER_WINDOWS = _REPLACER_WINDOWS.decode('utf-8')
 
 REJECTLIST_DIRS = frozenset(('.git', 'test', '.github', '.idea'))
 INDENT_REGEX = re.compile(r'([\t]*)')
@@ -311,8 +311,8 @@ class ModuleValidator(Validator):
 
         self.analyze_arg_spec = analyze_arg_spec and plugin_type == 'module'
 
-        self._Version = LooseVersion
-        self._StrictVersion = StrictVersion
+        self._Version: type[LooseVersion | SemanticVersion] = LooseVersion
+        self._StrictVersion: type[StrictVersion | SemanticVersion] = StrictVersion
 
         self.collection = collection
         self.collection_name = 'ansible.builtin'
@@ -687,30 +687,57 @@ class ModuleValidator(Validator):
         # get module list for each
         # check "shape" of each module name
 
-        module_requires = r'(?im)^#\s*requires\s+\-module(?:s?)\s*(Ansible\.ModuleUtils\..+)'
-        csharp_requires = r'(?im)^#\s*ansiblerequires\s+\-csharputil\s*(Ansible\..+)'
+        legacy_ps_requires = r'(?im)^#\s*Requires\s+\-Module(?:s?)\s+(Ansible\.ModuleUtils\..+)'
+        ps_requires = r"""(?imx)
+            ^\#\s*AnsibleRequires\s+-PowerShell\s+
+            (
+                # Builtin PowerShell module
+                (Ansible\.ModuleUtils\.[\w\.]+)
+                |
+                # Fully qualified collection PowerShell module
+                (ansible_collections\.\w+\.\w+\.plugins\.module_utils\.[\w\.]+)
+                |
+                # Relative collection PowerShell module
+                (\.[\w\.]+)
+            )
+            (\s+-Optional)?"""
+        csharp_requires = r"""(?imx)
+            ^\#\s*AnsibleRequires\s+-CSharpUtil\s+
+            (
+                # Builtin C# util
+                (Ansible\.[\w\.]+)
+                |
+                # Fully qualified collection C# util
+                (ansible_collections\.\w+\.\w+\.plugins\.module_utils\.[\w\.]+)
+                |
+                # Relative collection C# util
+                (\.[\w\.]+)
+            )
+            (\s+-Optional)?"""
+
         found_requires = False
 
-        for req_stmt in re.finditer(module_requires, self.text):
-            found_requires = True
-            # this will bomb on dictionary format - "don't do that"
-            module_list = [x.strip() for x in req_stmt.group(1).split(',')]
-            if len(module_list) > 1:
-                self.reporter.error(
-                    path=self.object_path,
-                    code='multiple-utils-per-requires',
-                    msg='Ansible.ModuleUtils requirements do not support multiple modules per statement: "%s"' % req_stmt.group(0)
-                )
-                continue
+        for pattern, required_type in [(legacy_ps_requires, "Requires"), (ps_requires, "AnsibleRequires")]:
+            for req_stmt in re.finditer(pattern, self.text):
+                found_requires = True
+                # this will bomb on dictionary format - "don't do that"
+                module_list = [x.strip() for x in req_stmt.group(1).split(',')]
+                if len(module_list) > 1:
+                    self.reporter.error(
+                        path=self.object_path,
+                        code='multiple-utils-per-requires',
+                        msg='Ansible.ModuleUtils requirements do not support multiple modules per statement: "%s"' % req_stmt.group(0)
+                    )
+                    continue
 
-            module_name = module_list[0]
+                module_name = module_list[0]
 
-            if module_name.lower().endswith('.psm1'):
-                self.reporter.error(
-                    path=self.object_path,
-                    code='invalid-requires-extension',
-                    msg='Module #Requires should not end in .psm1: "%s"' % module_name
-                )
+                if module_name.lower().endswith('.psm1'):
+                    self.reporter.error(
+                        path=self.object_path,
+                        code='invalid-requires-extension',
+                        msg='Module #%s should not end in .psm1: "%s"' % (required_type, module_name)
+                    )
 
         for req_stmt in re.finditer(csharp_requires, self.text):
             found_requires = True
@@ -979,7 +1006,7 @@ class ModuleValidator(Validator):
             missing_fragment = False
             with CaptureStd():
                 try:
-                    get_docstring(self.path, fragment_loader=fragment_loader,
+                    get_docstring(os.path.abspath(self.path), fragment_loader=fragment_loader,
                                   verbose=True,
                                   collection_name=self.collection_name,
                                   plugin_type=self.plugin_type)
@@ -1003,7 +1030,7 @@ class ModuleValidator(Validator):
                     )
 
             if not missing_fragment:
-                add_fragments(doc, self.object_path, fragment_loader=fragment_loader,
+                add_fragments(doc, os.path.abspath(self.object_path), fragment_loader=fragment_loader,
                               is_module=self.plugin_type == 'module')
 
             if 'options' in doc and doc['options'] is None:
@@ -1589,7 +1616,7 @@ class ModuleValidator(Validator):
 
         try:
             if not context:
-                add_fragments(docs, self.object_path, fragment_loader=fragment_loader,
+                add_fragments(docs, os.path.abspath(self.object_path), fragment_loader=fragment_loader,
                               is_module=self.plugin_type == 'module')
         except Exception:
             # Cannot merge fragments
@@ -2186,7 +2213,7 @@ class ModuleValidator(Validator):
         with CaptureStd():
             try:
                 existing_doc, dummy_examples, dummy_return, existing_metadata = get_docstring(
-                    self.base_module, fragment_loader, verbose=True, collection_name=self.collection_name,
+                    os.path.abspath(self.base_module), fragment_loader, verbose=True, collection_name=self.collection_name,
                     is_module=self.plugin_type == 'module')
                 existing_options = existing_doc.get('options', {}) or {}
             except AssertionError:

ansible_test/_util/controller/sanity/validate-modules/validate_modules/module_args.py

@@ -110,15 +110,22 @@ def get_ps_argument_spec(filename, collection):
         b_module_data = module_fd.read()
 
     ps_dep_finder = PSModuleDepFinder()
-    ps_dep_finder.scan_module(b_module_data, fqn=fqc_name)
+    module_deps = ps_dep_finder.scan_module(b_module_data, fqn=fqc_name)
+    ansible_basic = ''
+    ps_utils = {}
+    for dep in module_deps:
+        dep_info = ps_dep_finder.scripts[dep]
 
-    # For ps_argspec.ps1 to compile Ansible.Basic it also needs the AddType module_util.
-    ps_dep_finder._add_module(name=b"Ansible.ModuleUtils.AddType", ext=".psm1", fqn=None, optional=False, wrapper=False)
+        if dep == 'Ansible.Basic.cs':
+            ansible_basic = dep_info.path
+
+        elif dep.endswith('.psm1'):
+            ps_utils[dep] = dep_info.path
 
     util_manifest = json.dumps({
         'module_path': to_text(module_path, errors='surrogate_or_strict'),
-        'ansible_basic': ps_dep_finder.cs_utils_module["Ansible.Basic"]['path'],
-        'ps_utils': {name: info['path'] for name, info in ps_dep_finder.ps_modules.items()}
+        'ansible_basic': ansible_basic,
+        'ps_utils': ps_utils,
     })
 
     script_path = os.path.join(os.path.dirname(os.path.realpath(__file__)), 'ps_argspec.ps1')

ansible_test/_util/controller/sanity/validate-modules/validate_modules/schema.py

@@ -67,6 +67,17 @@ def collection_name(v, error_code=None):
     return v
 
 
+def fqcn(v, error_code=None):
+    if not isinstance(v, string_types):
+        raise _add_ansible_error_code(
+            Invalid('Module/plugin name must be a string'), error_code or 'invalid-documentation')
+    m = FULLY_QUALIFIED_COLLECTION_RESOURCE_RE.match(v)
+    if not m:
+        raise _add_ansible_error_code(
+            Invalid('Module/plugin name must be of format `<namespace>.<collection>.<name>(.<subname>)*`'), error_code or 'invalid-documentation')
+    return v
+
+
 def deprecation_versions():
     """Create a list of valid version for deprecation entries, current+4"""
     major, minor = [int(version) for version in __version__.split('.')[0:2]]
@@ -196,11 +207,11 @@ seealso_schema = Schema(
     [
         Any(
             {
-                Required('module'): Any(*string_types),
+                Required('module'): fqcn,
                 'description': doc_string,
             },
             {
-                Required('plugin'): Any(*string_types),
+                Required('plugin'): fqcn,
                 Required('plugin_type'): Any(*DOCUMENTABLE_PLUGINS),
                 'description': doc_string,
             },

ansible_test/_util/controller/sanity/validate-modules/validate_modules/utils.py

@@ -199,6 +199,7 @@ class NoArgsAnsibleModule(AnsibleModule):
     """AnsibleModule that does not actually load params. This is used to get access to the
     methods within AnsibleModule without having to fake a bunch of data
     """
+
     def _load_params(self):
         self.params = {'_ansible_selinux_special_fs': [], '_ansible_remote_tmp': '/tmp', '_ansible_keep_remote_files': False, '_ansible_check_mode': False}
 

ansible_test/_util/controller/sanity/yamllint/yamllinter.py

@@ -1,7 +1,9 @@
 """Wrapper around yamllint that supports YAML embedded in Ansible modules."""
+
 from __future__ import annotations
 
 import ast
+import copy
 import json
 import os
 import re
@@ -9,10 +11,10 @@ import sys
 import typing as t
 
 import yaml
-from yaml.resolver import Resolver
-from yaml.constructor import SafeConstructor
+from yaml.resolver import Resolver, BaseResolver
+from yaml.constructor import SafeConstructor, ConstructorError
 from yaml.error import MarkedYAMLError
-from yaml.cyaml import CParser
+from yaml.cyaml import CParser  # type: ignore[attr-defined]
 
 from yamllint import linter
 from yamllint.config import YamlLintConfig
@@ -29,37 +31,49 @@ def main():
 
 class TestConstructor(SafeConstructor):
     """Yaml Safe Constructor that knows about Ansible tags."""
+
     def construct_yaml_unsafe(self, node):
         """Construct an unsafe tag."""
-        try:
-            constructor = getattr(node, 'id', 'object')
-            if constructor is not None:
-                constructor = getattr(self, 'construct_%s' % constructor)
-        except AttributeError:
-            constructor = self.construct_object
+        return self._resolve_and_construct_object(node)
+
+    def construct_yaml_vault(self, node):
+        """Construct a vault tag."""
+        ciphertext = self._resolve_and_construct_object(node)
+
+        if not isinstance(ciphertext, str):
+            raise ConstructorError(problem=f"the {node.tag!r} tag requires a string value", problem_mark=node.start_mark)
 
-        value = constructor(node)
+        return ciphertext
 
-        return value
+    def _resolve_and_construct_object(self, node):
+        # use a copied node to avoid mutating existing node and tripping the recursion check in construct_object
+        copied_node = copy.copy(node)
+        # repeat implicit resolution process to determine the proper tag for the value in the unsafe node
+        copied_node.tag = t.cast(BaseResolver, self).resolve(type(node), node.value, (True, False))  # pylint: disable=no-member
+
+        # re-entrant call using the correct tag
+        # non-deferred construction of hierarchical nodes so the result is a fully realized object, and so our stateful unsafe propagation behavior works
+        return self.construct_object(copied_node, deep=True)
 
 
 TestConstructor.add_constructor(
     '!unsafe',
-    TestConstructor.construct_yaml_unsafe)
+    TestConstructor.construct_yaml_unsafe)  # type: ignore[type-var]
 
 
 TestConstructor.add_constructor(
     '!vault',
-    TestConstructor.construct_yaml_str)
+    TestConstructor.construct_yaml_vault)  # type: ignore[type-var]
 
 
 TestConstructor.add_constructor(
     '!vault-encrypted',
-    TestConstructor.construct_yaml_str)
+    TestConstructor.construct_yaml_vault)  # type: ignore[type-var]
 
 
 class TestLoader(CParser, TestConstructor, Resolver):
     """Custom YAML loader that recognizes custom Ansible tags."""
+
     def __init__(self, stream):
         CParser.__init__(self, stream)
         TestConstructor.__init__(self)
@@ -68,6 +82,7 @@ class TestLoader(CParser, TestConstructor, Resolver):
 
 class YamlChecker:
     """Wrapper around yamllint that supports YAML embedded in Ansible modules."""
+
     def __init__(self):
         self.messages = []
 
@@ -135,11 +150,11 @@ class YamlChecker:
 
             self.messages += [self.result_to_message(r, path, lineno - 1, key) for r in messages]
 
-    def check_parsable(self, path, contents, lineno=1, allow_multiple=False, prefix=""):  # type: (str, str, int, bool) -> None
+    def check_parsable(self, path: str, contents: str, lineno: int = 1, allow_multiple: bool = False, prefix: str = "") -> None:
         """Check the given contents to verify they can be parsed as YAML."""
         prefix = f"{prefix}: " if prefix else ""
         try:
-            documents = len(list(yaml.load_all(contents, Loader=TestLoader)))
+            documents = len(list(yaml.load_all(contents, Loader=TestLoader)))  # type: ignore[arg-type]
             if documents > 1 and not allow_multiple:
                 self.messages += [{'code': 'multiple-yaml-documents',
                                    'message': f'{prefix}expected a single document in the stream',
@@ -149,8 +164,11 @@ class YamlChecker:
                                    'level': 'error',
                                    }]
         except MarkedYAMLError as ex:
+            message = ' '.join(filter(None, (ex.context, ex.problem, ex.note)))
+            message = message.strip()
+
             self.messages += [{'code': 'unparsable-with-libyaml',
-                               'message': f'{prefix}{ex.args[0]} - {ex.args[2]}',
+                               'message': f'{prefix}{message}',
                                'path': path,
                                'line': ex.problem_mark.line + lineno,
                                'column': ex.problem_mark.column + 1,

ansible_test/_util/controller/tools/collection_detail.py

@@ -1,4 +1,5 @@
 """Retrieve collection detail."""
+
 from __future__ import annotations
 
 import json

ansible_test/_util/controller/tools/yaml_to_json.py

@@ -1,4 +1,5 @@
 """Read YAML from stdin and write JSON to stdout."""
+
 from __future__ import annotations
 
 import datetime
@@ -10,7 +11,7 @@ from yaml import load
 try:
     from yaml import CSafeLoader as SafeLoader
 except ImportError:
-    from yaml import SafeLoader
+    from yaml import SafeLoader  # type: ignore[assignment]
 
 # unique ISO date marker matching the one present in importer.py
 ISO_DATE_MARKER = 'isodate:f23983df-f3df-453c-9904-bcd08af468cc:'

ansible_test/_util/target/pytest/plugins/ansible_forked.py

@@ -1,4 +1,5 @@
 """Run each test in its own fork. PYTEST_DONT_REWRITE"""
+
 # MIT License (see licenses/MIT-license.txt or https://opensource.org/licenses/MIT)
 # Based on code originally from:
 # https://github.com/pytest-dev/pytest-forked
@@ -63,6 +64,8 @@ def run_parent(item, pid, result_path):  # type: (Item, int, str) -> list[TestRe
     """Wait for the child process to exit and return the test reports. Called in the parent process."""
     exit_code = waitstatus_to_exitcode(os.waitpid(pid, 0)[1])
 
+    reports: list[TestReport]
+
     if exit_code:
         reason = "Test CRASHED with exit code {}.".format(exit_code)
         report = TestReport(item.nodeid, item.location, {x: 1 for x in item.keywords}, "failed", reason, "call", user_properties=item.user_properties)
@@ -73,8 +76,10 @@ def run_parent(item, pid, result_path):  # type: (Item, int, str) -> list[TestRe
 
         reports = [report]
     else:
+        captured_warnings: list[warnings.WarningMessage]
+
         with open(result_path, "rb") as result_file:
-            reports, captured_warnings = pickle.load(result_file)  # type: list[TestReport], list[warnings.WarningMessage]
+            reports, captured_warnings = pickle.load(result_file)
 
         for warning in captured_warnings:
             warnings.warn_explicit(warning.message, warning.category, warning.filename, warning.lineno)

ansible_test/_util/target/pytest/plugins/ansible_pytest_collections.py

@@ -1,4 +1,5 @@
 """Enable unit testing of Ansible collections. PYTEST_DONT_REWRITE"""
+
 from __future__ import annotations
 
 import os
@@ -69,7 +70,7 @@ def enable_assertion_rewriting_hook():  # type: () -> None
     # noinspection PyProtectedMember
     from ansible.utils.collection_loader._collection_finder import _AnsibleCollectionPkgLoaderBase
 
-    _AnsibleCollectionPkgLoaderBase.exec_module = exec_module
+    _AnsibleCollectionPkgLoaderBase.exec_module = exec_module  # type: ignore[method-assign]
 
 
 def pytest_configure():

ansible_test/_util/target/pytest/plugins/ansible_pytest_coverage.py

@@ -1,4 +1,5 @@
 """Monkey patch os._exit when running under coverage so we don't lose coverage data in forks, such as with `pytest --boxed`. PYTEST_DONT_REWRITE"""
+
 from __future__ import annotations
 
 

ansible_test/_util/target/sanity/compile/compile.py

@@ -1,4 +1,5 @@
 """Python syntax checker with lint friendly output."""
+
 from __future__ import annotations
 
 import sys