100% test coverage

diff --git a/fastapi/routing.py b/fastapi/routing.py
index 187a8faf614df3858f6d781bc87c35492db9340d..2f53e0b08cb659617f817d6edb2c7bc0b43297ff 100644 (file)
--- a/fastapi/routing.py
+++ b/fastapi/routing.py
@@ -147,36 +147,19 @@ class APIRoute(routing.Route):
             ), f"(Duplicated Status Code): Response with status code [{add_response.status_code}] already defined!"
             existed_codes.append(add_response.status_code)
             response_models: List[Any] = [m for m in add_response.models]
-            valid_response_models = True
-            try:
-                valid_response_models = all(
-                    [issubclass(m, BaseModel) for m in response_models]
-                )
-            except TypeError:
-                valid_response_models = False
-            if not valid_response_models:
-                raise ValueError(
-                    "All response models must be "
-                    "a subclass of `pydantic.BaseModel` "
-                    "model."
-                )
+            schema_field = None
             if add_response.content_type == "application/json" or lenient_issubclass(
                 content_type, JSONResponse
-            ):
-                if len(response_models):
-                    schema_field = Field(
-                        name=f"Additional_response_{add_response.status_code}",
-                        type_=Union[tuple(response_models)],
-                        class_validators=[],
-                        default=None,
-                        required=False,
-                        model_config=UnconstrainedConfig,
-                        schema=Schema(None),
-                    )
-                else:
-                    schema_field = None
-            else:
-                schema_field = None
+            ) and len(response_models):
+                schema_field = Field(
+                    name=f"Additional_response_{add_response.status_code}",
+                    type_=Union[tuple(response_models)],
+                    class_validators=[],
+                    default=None,
+                    required=False,
+                    model_config=UnconstrainedConfig,
+                    schema=Schema(None),
+                )
             add_resp_description = AdditionalResponseDescription(
                 description=add_response.description,
                 content_type=add_response.content_type,

diff --git a/tests/test_additional_responses.py b/tests/test_additional_responses.py
index aa18dfbc683ca159e4d0beed7bacfd0959123da9..aa150b5381a67745115d4fefe6ababe02ee4f975 100644 (file)
--- a/tests/test_additional_responses.py
+++ b/tests/test_additional_responses.py
@@ -2,6 +2,7 @@ import pytest
 from fastapi import FastAPI
 from fastapi.openapi.models import AdditionalResponse
 from pydantic import BaseModel
+from pydantic.error_wrappers import ValidationError
 from starlette.responses import JSONResponse
 from starlette.testclient import TestClient
 
@@ -419,7 +420,7 @@ def test_uncompatible_response_model_undecorated():
     response_403 = AdditionalResponse(
         status_code=403, description="Forbidden", models=[NotBaseModel]
     )
-    with pytest.raises(ValueError):
+    with pytest.raises(RuntimeError):
         app.add_api_route("/", get_not_decorated, additional_responses=[response_403])