follow-up ruff upgrade

Author: csernazs

pyproject.toml

@@ -116,6 +116,7 @@ lint.ignore = [
   "C408",
   "C901",
   "COM812",
+  "CPY001",
   "EM101",
   "EM103",
   "FBT002",
@@ -148,3 +149,6 @@ lint.ignore = [
 line-length = 120
 target-version = "py310"
 exclude = ["doc", "example*.py", "tests/examples/*.py"]
+
+[tool.ruff.lint.pylint]
+max-positional-args = 10

pytest_httpserver/blocking_httpserver.py

@@ -76,7 +76,7 @@ def assert_request(
         data: str | bytes | None = None,
         data_encoding: str = "utf-8",
         headers: Mapping[str, str] | None = None,
-        query_string: None | QueryMatcher | str | bytes | Mapping[str, str] = None,
+        query_string: QueryMatcher | str | bytes | Mapping[str, str] | None = None,
         header_value_matcher: HeaderValueMatcher | None = None,
         json: Any = UNDEFINED,
         timeout: int = 30,

pytest_httpserver/httpserver.py

@@ -265,7 +265,7 @@ def get_comparing_values(self, request_query_string: bytes) -> tuple[bool, bool]
             return (True, False)
 
 
-def _create_query_matcher(query_string: None | QueryMatcher | str | bytes | Mapping[str, str]) -> QueryMatcher:
+def _create_query_matcher(query_string: QueryMatcher | str | bytes | Mapping[str, str] | None) -> QueryMatcher:
     if isinstance(query_string, QueryMatcher):
         return query_string
 
@@ -300,7 +300,7 @@ class RequestMatcherKwargs(TypedDict, total=False):
     data: str | bytes | None
     data_encoding: str
     headers: Mapping[str, str] | None
-    query_string: None | QueryMatcher | str | bytes | Mapping[str, str]
+    query_string: QueryMatcher | str | bytes | Mapping[str, str] | None
     header_value_matcher: HVMATCHER_T | None
     json: Any
 
@@ -341,7 +341,7 @@ def __init__(
         data: str | bytes | None = None,
         data_encoding: str = "utf-8",
         headers: Mapping[str, str] | None = None,
-        query_string: None | QueryMatcher | str | bytes | Mapping[str, str] = None,
+        query_string: QueryMatcher | str | bytes | Mapping[str, str] | None = None,
         header_value_matcher: HVMATCHER_T | None = None,
         json: Any = UNDEFINED,
     ) -> None:
@@ -1068,7 +1068,7 @@ def expect_request(
         data: str | bytes | None = None,
         data_encoding: str = "utf-8",
         headers: Mapping[str, str] | None = None,
-        query_string: None | QueryMatcher | str | bytes | Mapping[str, str] = None,
+        query_string: QueryMatcher | str | bytes | Mapping[str, str] | None = None,
         header_value_matcher: HVMATCHER_T | None = None,
         handler_type: HandlerType = HandlerType.PERMANENT,
         json: Any = UNDEFINED,
@@ -1151,7 +1151,7 @@ def expect_oneshot_request(
         data: str | bytes | None = None,
         data_encoding: str = "utf-8",
         headers: Mapping[str, str] | None = None,
-        query_string: None | QueryMatcher | str | bytes | Mapping[str, str] = None,
+        query_string: QueryMatcher | str | bytes | Mapping[str, str] | None = None,
         header_value_matcher: HVMATCHER_T | None = None,
         json: Any = UNDEFINED,
     ) -> RequestHandler:
@@ -1206,7 +1206,7 @@ def expect_ordered_request(
         data: str | bytes | None = None,
         data_encoding: str = "utf-8",
         headers: Mapping[str, str] | None = None,
-        query_string: None | QueryMatcher | str | bytes | Mapping[str, str] = None,
+        query_string: QueryMatcher | str | bytes | Mapping[str, str] | None = None,
         header_value_matcher: HVMATCHER_T | None = None,
         json: Any = UNDEFINED,
     ) -> RequestHandler:

tests/test_log_querying.py

@@ -40,8 +40,10 @@ def test_verify_assert_msg(httpserver: HTTPServer):
 
     expected_lines = [
         "Matching request found 0 times but expected 1 times.",
-        "Expected request: <RequestMatcher uri='/foo' method='POST' query_string=None headers={} data"
-        "=None json={'foo': 'bar'}>",
+        (
+            "Expected request: <RequestMatcher uri='/foo' method='POST' query_string=None headers={} data"
+            "=None json={'foo': 'bar'}>"
+        ),
         "Found 1 similar request(s):",
         "--- Similar Request Start",
         "Path: /foo",
@@ -70,8 +72,10 @@ def test_verify_assert_msg_no_similar_requests(httpserver: HTTPServer):
 
     expected_lines = [
         "Matching request found 0 times but expected 1 times.",
-        "Expected request: <RequestMatcher uri='/foo' method='POST' query_string=None headers={} data"
-        "=None json={'foo': 'bar'}>",
+        (
+            "Expected request: <RequestMatcher uri='/foo' method='POST' query_string=None headers={} data"
+            "=None json={'foo': 'bar'}>"
+        ),
         "No similar requests found.",
     ]