Fix line length violations and remove unnecessary PLW0717 suppression from test files

Replace # noqa: LN001 suppressions with properly split strings wrapped
in parentheses to satisfy ISC004. Remove PLW0717 from the test file
ignore list — test files have no violations so the suppression was
unnecessary.

Author: julianz-

.ruff.toml

@@ -219,7 +219,6 @@ testing = [
   "ARG004",  # unused-static-method-argument (hit in WSGI test apps)
   "PLC2701",  # Allow importing internal files needed for testing
   # "PLR6301",  # Allow 'self' parameter in method definitions (required for test stubs)
-  "PLW0717",  # too-many-statements-in-try-clause  # FIXME
   "S101",  # Allow use of `assert` in test files
   "S404",  # Allow importing 'subprocess' module to testing call external tools needed by these hooks
   "S603",  # subprocess-without-shell-equals-true

cheroot/test/test_conn.py

@@ -191,7 +191,8 @@ def testing_server(raw_testing_server, monkeypatch):
             continue
 
         assert c_msg in raw_testing_server.error_log.ignored_msgs, (
-            f"Found error in the error log: message = '{c_msg}', level = '{c_level}'\n{c_traceback}"  # noqa: LN001
+            f"Found error in the error log: message = '{c_msg}', "
+            f"level = '{c_level}'\n{c_traceback}"
         )
 
 
@@ -788,11 +789,21 @@ def _read_request_line(self):
         (logging.WARNING, r'^socket\.error 666$'),
         (
             logging.INFO,
-            r'^Got a connection error while handling a connection from .*:\d{1,5} \(666\)',  # noqa: LN001
+            (
+                r'^Got a connection error while handling a connection '
+                r'from .*:\d{1,5} \(666\)'
+            ),
         ),
         (
             logging.CRITICAL,
-            r'A fatal exception happened\. Setting the server interrupt flag to ConnectionResetError\(666,?\) and giving up\.\n\nPlease, report this on the Cheroot tracker at <https://github\.com/cherrypy/cheroot/issues/new/choose>, providing a full reproducer with as much context and details as possible\.$',  # noqa: LN001
+            (
+                r'A fatal exception happened\. Setting the server interrupt flag '
+                r'to ConnectionResetError\(666,?\) and giving up\.\n\nPlease, '
+                r'report this on the Cheroot tracker at '
+                r'<https://github\.com/cherrypy/cheroot/issues/new/choose>, '
+                r'providing a full reproducer with as much context and details '
+                r'as possible\.$'
+            ),
         ),
     )
 
@@ -832,11 +843,17 @@ def _trigger_kb_intr(_req, _resp):
     expected_log_entries = (
         (
             logging.DEBUG,
-            r'^Got a server shutdown request while handling a connection from .*:\d{1,5} \(simulated test handler keyboard interrupt\)$',  # noqa: LN001
+            (
+                r'^Got a server shutdown request while handling a connection '
+                r'from .*:\d{1,5} \(simulated test handler keyboard interrupt\)$'
+            ),
         ),
         (
             logging.DEBUG,
-            r"^Setting the server interrupt flag to KeyboardInterrupt\('simulated test handler keyboard interrupt',?\)$",  # noqa: LN001
+            (
+                r'^Setting the server interrupt flag to KeyboardInterrupt'
+                r"\('simulated test handler keyboard interrupt',?\)$"
+            ),
         ),
         (
             logging.INFO,
@@ -903,7 +920,10 @@ def _trigger_scary_exc(_req, _resp):
     expected_log_entries = (
         (
             logging.ERROR,
-            r"^Unhandled error while processing an incoming connection SillyMistake\('simulated unhandled exception 💣 in test handler',?\)$",  # noqa: LN001
+            (
+                r'^Unhandled error while processing an incoming connection '
+                r"SillyMistake\('simulated unhandled exception 💣 in test handler',?\)$"
+            ),
         ),
         (
             logging.INFO,
@@ -985,7 +1005,10 @@ def _read_request_line(self):
     expected_log_entries = (
         (
             logging.ERROR,
-            r'^Unhandled error while processing an incoming connection ScaryCrash\(666,?\)$',  # noqa: LN001
+            (
+                r'^Unhandled error while processing an incoming connection '
+                r'ScaryCrash\(666,?\)$'
+            ),
         ),
         (
             logging.INFO,

docs/changelog-fragments.d/826.contrib.rst

@@ -7,6 +7,6 @@ Bumped Ruff to v0.15.18, which introduces new rules:
 
 Three further new rules are suppressed pending fixes: too many statements
 in a try clause (PLW0717), non-empty ``__init__`` modules (RUF067), and
-starting a process with a partial executable path (S607)
+starting a process with a partial executable path (S607).
 
--- by :user:`julianz-`.
+-- by :user:`julianz-`

docs/conf.py

@@ -107,8 +107,14 @@
     r'https://github\.com/cherrypy/cherrypy/tree',
     # Has an ephemeral anchor (line-range) but actual HTML has separate per-
     # line anchors.
-    r'https://github\.com/python/cpython/blob/c39b52f/Lib/poplib\.py#L297-L302',  # noqa: LN001
-    r'https://github\.com/python/cpython/blob/c39b52f/Lib/poplib\.py#user-content-L297-L302',  # noqa: LN001
+    (
+        r'https://github\.com'
+        r'/python/cpython/blob/c39b52f/Lib/poplib\.py#L297-L302'
+    ),
+    (
+        r'https://github\.com'
+        r'/python/cpython/blob/c39b52f/Lib/poplib\.py#user-content-L297-L302'
+    ),
     r'^https://img\.shields\.io/matrix',  # these are rate-limited
     r'^https://matrix\.to/#',  # these render fully on front-end from anchors
     r'^https://stackoverflow\.com/',  # these generate HTTP 403 Forbidden