Clean up other Python files (#11379)

* Address issues found by ruff check.

* Make mypy happy; remove some Python 2 compat code.

* Also declare port1.

tests/integration/targets/callback/filter_plugins/helper.py

@@ -12,7 +12,6 @@ def callback_results_extractor(outputs_results):
         expected_output = result["test"]["expected_output"]
         stdout_lines = result["stdout_lines"]
         for i in range(max(len(expected_output), len(stdout_lines))):
-            line = "line_%s" % (i + 1)
             test_line = stdout_lines[i] if i < len(stdout_lines) else None
             expected_lines = expected_output[i] if i < len(expected_output) else None
             if not isinstance(expected_lines, str) and expected_lines is not None:

tests/integration/targets/cmd_runner/action_plugins/_unsafe_assert.py

@@ -9,7 +9,7 @@ from ansible.playbook.conditional import Conditional
 from ansible.plugins.action import ActionBase
 
 try:
-    from ansible.utils.datatag import trust_value as _trust_value
+    from ansible.utils.datatag import trust_value as _trust_value  # type: ignore[import-not-found]
 except ImportError:
 
     def _trust_value(input):

tests/integration/targets/cmd_runner/library/cmd_echo.py

@@ -51,11 +51,10 @@ def main():
         with runner.context(p["arg_order"], check_mode_skip=p["check_mode_skip"]) as ctx:
             result = ctx.run(**p["arg_values"])
             info = ctx.run_info
-        check = "check"
         rc, out, err = result if result is not None else (None, None, None)
 
         module.exit_json(rc=rc, out=out, err=err, info=info)
-    except Exception as exc:
+    except Exception:
         module.fail_json(rc=1, module_stderr=traceback.format_exc(), msg="Module crashed with exception")
 
 

tests/integration/targets/flatpak/files/serve.py

@@ -9,13 +9,8 @@ import os
 import posixpath
 import sys
 
-try:
-    from http.server import SimpleHTTPRequestHandler, HTTPServer
-    from urllib.parse import unquote
-except ImportError:
-    from SimpleHTTPServer import SimpleHTTPRequestHandler
-    from BaseHTTPServer import HTTPServer
-    from urllib import unquote
+from http.server import SimpleHTTPRequestHandler, HTTPServer
+from urllib.parse import unquote
 
 
 # Argument parsing
@@ -23,8 +18,8 @@ if len(sys.argv) != 4:
     print(f"Syntax: {sys.argv[0]} <bind> <port> <path>")
     sys.exit(-1)
 
-HOST, PORT, PATH = sys.argv[1:4]
-PORT = int(PORT)
+HOST, PORT_str, PATH = sys.argv[1:4]
+PORT = int(PORT_str)
 
 
 # The HTTP request handler
@@ -40,7 +35,7 @@ class Handler(SimpleHTTPRequestHandler):
         trailing_slash = path.rstrip().endswith("/")
         try:
             path = unquote(path, errors="surrogatepass")
-        except (UnicodeDecodeError, TypeError) as exc:
+        except (UnicodeDecodeError, TypeError):
             path = unquote(path)
         path = posixpath.normpath(path)
         words = path.split("/")

tests/integration/targets/java_cert/files/setupSSLServer.py

@@ -10,15 +10,11 @@ import sys
 root_dir = sys.argv[1]
 port = int(sys.argv[2])
 
-try:
-    from BaseHTTPServer import HTTPServer
-    from SimpleHTTPServer import SimpleHTTPRequestHandler
-except ModuleNotFoundError:
-    from http.server import HTTPServer, SimpleHTTPRequestHandler
+from http.server import HTTPServer, SimpleHTTPRequestHandler
 
 httpd = HTTPServer(("localhost", port), SimpleHTTPRequestHandler)
 try:
-    httpd.socket = ssl.wrap_socket(
+    httpd.socket = ssl.wrap_socket(  # type: ignore[attr-defined]
         httpd.socket,
         server_side=True,
         certfile=os.path.join(root_dir, "cert.pem"),

tests/integration/targets/mail/files/smtpserver.py

@@ -27,6 +27,8 @@ port = "25:465"
 if len(sys.argv) > 1:
     port = sys.argv[1]
 ports = port.split(":")
+port1: int
+port2: int | None
 if len(ports) > 1:
     port1, port2 = int(ports[0]), int(ports[1])
 else:
@@ -58,6 +60,6 @@ else:
     print("Start SMTP server on port", port1)
     smtp_server1 = smtpd.DebuggingServer(("127.0.0.1", port1), None)  # pylint: disable=used-before-assignment
     if port2:
-        print("WARNING: TLS is NOT supported on this system, not listening on port %s." % port2)
+        print(f"WARNING: TLS is NOT supported on this system, not listening on port {port2}.")
 
 asyncore.loop()

tests/integration/targets/module_helper/library/mdepfail.py

@@ -32,7 +32,7 @@ from ansible_collections.community.general.plugins.module_utils import deps
 from ansible_collections.community.general.plugins.module_utils.module_helper import ModuleHelper
 
 with deps.declare("nopackagewiththisname"):
-    import nopackagewiththisname  # noqa: F401, pylint: disable=unused-import
+    import nopackagewiththisname  # noqa: F401, pylint: disable=unused-import # type: ignore[import-not-found]
 
 
 class MSimple(ModuleHelper):

tests/integration/targets/monit/files/httpd_echo.py

@@ -6,16 +6,11 @@ from __future__ import annotations
 
 import daemon
 
-try:
-    from BaseHTTPServer import BaseHTTPRequestHandler, HTTPServer
+from http.server import BaseHTTPRequestHandler, HTTPServer
 
-    def write_to_output(stream, content):
-        stream.write(content)
-except ImportError:
-    from http.server import BaseHTTPRequestHandler, HTTPServer
 
-    def write_to_output(stream, content):
-        stream.write(bytes(content, "utf-8"))
+def write_to_output(stream, content):
+    stream.write(bytes(content, "utf-8"))
 
 
 hostname = "localhost"

tests/integration/targets/supervisorctl/files/sendProcessStdin.py

@@ -22,4 +22,4 @@ else:
     url = "http://127.0.0.1:9001/RPC2"
 
 server = ServerProxy(url, verbose=True)
-server.supervisor.sendProcessStdin(proc, "import sys; print(%s); sys.stdout.flush();\n" % value)
+server.supervisor.sendProcessStdin(proc, f"import sys; print({value}); sys.stdout.flush();\n")

tests/sanity/extra/botmeta.py

@@ -56,7 +56,7 @@ IGNORE_NO_MAINTAINERS = [
 
 
 class BotmetaCheck:
-    def __init__(self):
+    def __init__(self) -> None:
         self.errors: list[str] = []
         self.botmeta_filename = ".github/BOTMETA.yml"
         self.list_entries = frozenset(("supershipit", "maintainers", "labels", "keywords", "notify", "ignore"))
@@ -121,8 +121,8 @@ class BotmetaCheck:
             )
         ):
             maintainers = self.read_authors(filename)
-            for maintainer in maintainers:
-                maintainer = self.extract_author_name(maintainer)
+            for maintainer_str in maintainers:
+                maintainer = self.extract_author_name(maintainer_str)
                 if maintainer is not None and maintainer not in all_maintainers:
                     others = ", ".join(all_maintainers)
                     msg = f"Author {maintainer} not mentioned as active or inactive maintainer for {filename} (mentioned are: {others})"