Fix constructing an OpenAPI with both extracted and explicit schemas.

Until now, the explicit list of schemas replaced rather than
expanded the extracted schemas because we were calling
_handle_pydantic_schema() twice. The first time, it was mutating
the schema and leaving no info that the second call would find.
Solution: call _handle_pydantic_schema() only once, which is
more efficient anyway.

Author: hathawsh

openapi_pydantic/util.py

@@ -71,9 +71,7 @@ def construct_open_api_with_schema_class(
     if scan_for_pydantic_schema_reference:
         extracted_schema_classes = _handle_pydantic_schema(new_open_api)
         if schema_classes:
-            schema_classes = list(
-                {*schema_classes, *_handle_pydantic_schema(new_open_api)}
-            )
+            schema_classes = list({*schema_classes, *extracted_schema_classes})
         else:
             schema_classes = extracted_schema_classes
 

openapi_pydantic/v3/v3_0_3/util.py

@@ -142,9 +142,7 @@ def construct_open_api_with_schema_class(
     if scan_for_pydantic_schema_reference:
         extracted_schema_classes = _handle_pydantic_schema(new_open_api)
         if schema_classes:
-            schema_classes = list(
-                {*schema_classes, *_handle_pydantic_schema(new_open_api)}
-            )
+            schema_classes = list({*schema_classes, *extracted_schema_classes})
         else:
             schema_classes = extracted_schema_classes