[pre-commit.ci] pre-commit autoupdate (#3752)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Thiago Bellini Ribeiro <[email protected]>

.pre-commit-config.yaml

@@ -1,6 +1,6 @@
 repos:
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.8.6
+    rev: v0.9.1
     hooks:
       - id: ruff-format
         exclude: ^tests/\w+/snapshots/

pyproject.toml

@@ -257,6 +257,7 @@ ignore = [
     "A001",
     "A002",
     "A003",
+    "A005",
 
     # Unused arguments
     "ARG001",

strawberry/codegen/plugins/python.py

@@ -65,7 +65,7 @@ def generate_code(
 
     def _print_imports(self) -> str:
         imports = [
-            f'from {import_} import {", ".join(sorted(types))}'
+            f"from {import_} import {', '.join(sorted(types))}"
             for import_, types in self.imports.items()
         ]
 
@@ -187,9 +187,9 @@ def _print_scalar_type(self, type_: GraphQLScalar) -> str:
         if type_.name in self.SCALARS_TO_PYTHON_TYPES:
             return ""
 
-        assert (
-            type_.python_type is not None
-        ), f"Scalar type must have a python type: {type_.name}"
+        assert type_.python_type is not None, (
+            f"Scalar type must have a python type: {type_.name}"
+        )
 
         return f'{type_.name} = NewType("{type_.name}", {type_.python_type.__name__})'
 

strawberry/codegen/query_codegen.py

@@ -347,9 +347,9 @@ def _populate_fragment_types(self, ast: DocumentNode) -> None:
         )
         for fd in fragment_definitions:
             query_type = self.schema.get_type_by_name(fd.type_condition.name.value)
-            assert isinstance(
-                query_type, StrawberryObjectDefinition
-            ), f"{fd.type_condition.name.value!r} is not a type in the graphql schema!"
+            assert isinstance(query_type, StrawberryObjectDefinition), (
+                f"{fd.type_condition.name.value!r} is not a type in the graphql schema!"
+            )
 
             typename = fd.type_condition.name.value
             graph_ql_object_type_factory = partial(
@@ -695,9 +695,9 @@ def _field_from_selection_set(
             selection.name.value, parent_type_name
         )
 
-        assert (
-            selected_field
-        ), f"Couldn't find {parent_type_name}.{selection.name.value}"
+        assert selected_field, (
+            f"Couldn't find {parent_type_name}.{selection.name.value}"
+        )
 
         selected_field_type, wrapper = self._unwrap_type(selected_field.type)
         name = capitalize_first(to_camel_case(selection.name.value))

strawberry/exceptions/missing_return_annotation.py

@@ -27,7 +27,7 @@ def __init__(
             "did you forget to add it?"
         )
         self.rich_message = (
-            "[bold red]Missing annotation for field " f"`[underline]{resolver.name}[/]`"
+            f"[bold red]Missing annotation for field `[underline]{resolver.name}[/]`"
         )
 
         self.suggestion = (

strawberry/exceptions/permission_fail_silently_requires_optional.py

@@ -16,7 +16,7 @@ class PermissionFailSilentlyRequiresOptionalError(StrawberryException):
     def __init__(self, field: StrawberryField) -> None:
         self.field = field
         self.message = (
-            "Cannot use fail_silently=True with a non-optional " "or non-list field"
+            "Cannot use fail_silently=True with a non-optional or non-list field"
         )
         self.rich_message = (
             "fail_silently permissions can only be used with fields of type "

strawberry/printer/ast_from_value.py

@@ -56,8 +56,7 @@ def ast_from_leaf_type(
         return IntValueNode(value=str(serialized))
     if isinstance(serialized, float) and isfinite(serialized):
         value = str(serialized)
-        if value.endswith(".0"):
-            value = value[:-2]
+        value = value.removesuffix(".0")
         return FloatValueNode(value=value)
 
     if isinstance(serialized, str):

strawberry/schema/schema_converter.py

@@ -518,12 +518,10 @@ def from_interface(
             assert isinstance(graphql_interface, GraphQLInterfaceType)  # For mypy
             return graphql_interface
 
-        def _get_resolve_type() -> (
-            Callable[
-                [Any, GraphQLResolveInfo, GraphQLAbstractType],
-                Union[Awaitable[Optional[str]], str, None],
-            ]
-        ):
+        def _get_resolve_type() -> Callable[
+            [Any, GraphQLResolveInfo, GraphQLAbstractType],
+            Union[Awaitable[Optional[str]], str, None],
+        ]:
             if interface.resolve_type:
                 return interface.resolve_type
 

strawberry/schema/types/scalar.py

@@ -68,7 +68,7 @@ def _get_scalar_definition(scalar: type) -> ScalarDefinition:
             GlobalID,
             name="GlobalID",
             description=GraphQLID.description,
-            parse_literal=lambda v, vars=None: GlobalID.from_id(
+            parse_literal=lambda v, vars=None: GlobalID.from_id(  # noqa: A006
                 GraphQLID.parse_literal(v, vars)
             ),
             parse_value=GlobalID.from_id,

tests/litestar/test_response_headers.py

@@ -59,6 +59,5 @@ def abc(self, info: strawberry.Info) -> str:
     assert response.json() == {"data": {"abc": "abc"}}
 
     assert response.headers["set-cookie"] == (
-        "strawberry=rocks; Path=/; SameSite=lax, "
-        "Litestar=rocks; Path=/; SameSite=lax"
+        "strawberry=rocks; Path=/; SameSite=lax, Litestar=rocks; Path=/; SameSite=lax"
     )

tests/schema/test_basic.py

@@ -476,9 +476,9 @@ class Query:
     }
     """
     assert str(schema) == textwrap.dedent(expected).strip()
-    assert "<strawberry.schema.schema.Schema object" in repr(
-        schema
-    ), "Repr should not be affected"
+    assert "<strawberry.schema.schema.Schema object" in repr(schema), (
+        "Repr should not be affected"
+    )
 
 
 def test_field_with_default():

tests/schema/test_one_of.py

@@ -90,7 +90,7 @@ def test_must_specify_at_least_one_key_literal(value: str, variables: dict[str,
         variables_definitions.append("$input: ExampleInputTagged!")
 
     variables_definition_str = (
-        f'({", ".join(variables_definitions)})' if variables_definitions else ""
+        f"({', '.join(variables_definitions)})" if variables_definitions else ""
     )
 
     query = f"""
@@ -184,7 +184,7 @@ def test_works(value: str, variables: dict[str, Any], expected: dict[str, Any]):
         variables_definitions.append("$input: ExampleInputTagged!")
 
     variables_definition_str = (
-        f'({", ".join(variables_definitions)})' if variables_definitions else ""
+        f"({', '.join(variables_definitions)})" if variables_definitions else ""
     )
 
     field = next(iter(expected.keys()))

tests/schema/types/test_datetime.py

@@ -144,7 +144,7 @@ def datetime_input(
         "20120411T03:30-25:40",
         "20120411T03:30+00:60",
         "20120411T03:30+00:61",
-        "20120411T033030.123456012:00" "2014-03-12T12:30:14",
+        "20120411T033030.123456012:002014-03-12T12:30:14",
         "2014-04-21T24:00:01",
     ],
 )

tests/test_dataloaders.py

@@ -123,8 +123,7 @@ async def idx(keys: list[int]) -> list[int]:
     with pytest.raises(
         WrongNumberOfResultsReturned,
         match=(
-            "Received wrong number of results in dataloader, "
-            "expected: 1, received: 2"
+            "Received wrong number of results in dataloader, expected: 1, received: 2"
         ),
     ):
         await loader.load(1)