diff --git a/fastapi/routing.py b/fastapi/routing.py index ac8da2e1e2..efde24e0a7 100644 --- a/fastapi/routing.py +++ b/fastapi/routing.py @@ -1410,32 +1410,32 @@ class APIRouter(routing.Router): current_generate_unique_id = get_value_or_default( generate_unique_id_function, self.generate_unique_id_function ) - route_kwargs: dict[str, Any] = dict( - response_model=response_model, - status_code=status_code, - tags=current_tags, - dependencies=current_dependencies, - summary=summary, - description=description, - response_description=response_description, - responses=combined_responses, - deprecated=deprecated or self.deprecated, - methods=methods, - operation_id=operation_id, - response_model_include=response_model_include, - response_model_exclude=response_model_exclude, - response_model_by_alias=response_model_by_alias, - response_model_exclude_unset=response_model_exclude_unset, - response_model_exclude_defaults=response_model_exclude_defaults, - response_model_exclude_none=response_model_exclude_none, - include_in_schema=include_in_schema and self.include_in_schema, - response_class=current_response_class, - name=name, - dependency_overrides_provider=self.dependency_overrides_provider, - callbacks=current_callbacks, - openapi_extra=openapi_extra, - generate_unique_id_function=current_generate_unique_id, - ) + route_kwargs: dict[str, Any] = { + "response_model": response_model, + "status_code": status_code, + "tags": current_tags, + "dependencies": current_dependencies, + "summary": summary, + "description": description, + "response_description": response_description, + "responses": combined_responses, + "deprecated": deprecated or self.deprecated, + "methods": methods, + "operation_id": operation_id, + "response_model_include": response_model_include, + "response_model_exclude": response_model_exclude, + "response_model_by_alias": response_model_by_alias, + "response_model_exclude_unset": response_model_exclude_unset, + "response_model_exclude_defaults": response_model_exclude_defaults, + "response_model_exclude_none": response_model_exclude_none, + "include_in_schema": include_in_schema and self.include_in_schema, + "response_class": current_response_class, + "name": name, + "dependency_overrides_provider": self.dependency_overrides_provider, + "callbacks": current_callbacks, + "openapi_extra": openapi_extra, + "generate_unique_id_function": current_generate_unique_id, + } if _accepts_kwarg(route_class, "strict_content_type"): route_kwargs["strict_content_type"] = get_value_or_default( strict_content_type, self.strict_content_type diff --git a/tests/test_custom_route_class.py b/tests/test_custom_route_class.py index 31c1db9080..f954d82340 100644 --- a/tests/test_custom_route_class.py +++ b/tests/test_custom_route_class.py @@ -241,7 +241,7 @@ def test_accepts_kwarg_signature_error(): """Cover the fallback when inspect.signature raises (line 826-827).""" from unittest.mock import patch - from fastapi.routing import _accepts_kwarg, _ACCEPTS_KWARG_CACHE + from fastapi.routing import _ACCEPTS_KWARG_CACHE, _accepts_kwarg class SimpleRoute: def __init__(self, x):