✅ Refactor 2 tests, for consistency and simplification (#9504)

✅ Refactor tests, for consistency and simplification

tests/test_request_body_parameters_media_type.py

@@ -33,36 +33,146 @@ async def create_shop(
     pass  # pragma: no cover
 
 
-create_product_request_body = {
+client = TestClient(app)
+
+
+def test_openapi_schema():
+    response = client.get("/openapi.json")
+    assert response.status_code == 200, response.text
+    # insert_assert(response.json())
+    assert response.json() == {
+        "openapi": "3.0.2",
+        "info": {"title": "FastAPI", "version": "0.1.0"},
+        "paths": {
+            "/products": {
+                "post": {
+                    "summary": "Create Product",
+                    "operationId": "create_product_products_post",
+                    "requestBody": {
                         "content": {
                             "application/vnd.api+json": {
-            "schema": {"$ref": "#/components/schemas/Body_create_product_products_post"}
+                                "schema": {
+                                    "$ref": "#/components/schemas/Body_create_product_products_post"
+                                }
                             }
                         },
                         "required": True,
+                    },
+                    "responses": {
+                        "200": {
+                            "description": "Successful Response",
+                            "content": {"application/json": {"schema": {}}},
+                        },
+                        "422": {
+                            "description": "Validation Error",
+                            "content": {
+                                "application/json": {
+                                    "schema": {
+                                        "$ref": "#/components/schemas/HTTPValidationError"
                                     }
-
+                                }
-create_shop_request_body = {
+                            },
+                        },
+                    },
+                }
+            },
+            "/shops": {
+                "post": {
+                    "summary": "Create Shop",
+                    "operationId": "create_shop_shops_post",
+                    "requestBody": {
                         "content": {
                             "application/vnd.api+json": {
-            "schema": {"$ref": "#/components/schemas/Body_create_shop_shops_post"}
+                                "schema": {
+                                    "$ref": "#/components/schemas/Body_create_shop_shops_post"
+                                }
                             }
                         },
                         "required": True,
+                    },
+                    "responses": {
+                        "200": {
+                            "description": "Successful Response",
+                            "content": {"application/json": {"schema": {}}},
+                        },
+                        "422": {
+                            "description": "Validation Error",
+                            "content": {
+                                "application/json": {
+                                    "schema": {
+                                        "$ref": "#/components/schemas/HTTPValidationError"
+                                    }
+                                }
+                            },
+                        },
+                    },
+                }
+            },
+        },
+        "components": {
+            "schemas": {
+                "Body_create_product_products_post": {
+                    "title": "Body_create_product_products_post",
+                    "required": ["data"],
+                    "type": "object",
+                    "properties": {"data": {"$ref": "#/components/schemas/Product"}},
+                },
+                "Body_create_shop_shops_post": {
+                    "title": "Body_create_shop_shops_post",
+                    "required": ["data"],
+                    "type": "object",
+                    "properties": {
+                        "data": {"$ref": "#/components/schemas/Shop"},
+                        "included": {
+                            "title": "Included",
+                            "type": "array",
+                            "items": {"$ref": "#/components/schemas/Product"},
+                            "default": [],
+                        },
+                    },
+                },
+                "HTTPValidationError": {
+                    "title": "HTTPValidationError",
+                    "type": "object",
+                    "properties": {
+                        "detail": {
+                            "title": "Detail",
+                            "type": "array",
+                            "items": {"$ref": "#/components/schemas/ValidationError"},
+                        }
+                    },
+                },
+                "Product": {
+                    "title": "Product",
+                    "required": ["name", "price"],
+                    "type": "object",
+                    "properties": {
+                        "name": {"title": "Name", "type": "string"},
+                        "price": {"title": "Price", "type": "number"},
+                    },
+                },
+                "Shop": {
+                    "title": "Shop",
+                    "required": ["name"],
+                    "type": "object",
+                    "properties": {"name": {"title": "Name", "type": "string"}},
+                },
+                "ValidationError": {
+                    "title": "ValidationError",
+                    "required": ["loc", "msg", "type"],
+                    "type": "object",
+                    "properties": {
+                        "loc": {
+                            "title": "Location",
+                            "type": "array",
+                            "items": {
+                                "anyOf": [{"type": "string"}, {"type": "integer"}]
+                            },
+                        },
+                        "msg": {"title": "Message", "type": "string"},
+                        "type": {"title": "Error Type", "type": "string"},
+                    },
+                },
+            }
+        },
     }
-
-client = TestClient(app)
-
-
-def test_openapi_schema():
-    response = client.get("/openapi.json")
-    assert response.status_code == 200, response.text
-    openapi_schema = response.json()
-    assert (
-        openapi_schema["paths"]["/products"]["post"]["requestBody"]
-        == create_product_request_body
-    )
-    assert (
-        openapi_schema["paths"]["/shops"]["post"]["requestBody"]
-        == create_shop_request_body
-    )

tests/test_tutorial/test_dataclasses/test_tutorial002.py

@@ -1,12 +1,27 @@
-from copy import deepcopy
-
 from fastapi.testclient import TestClient
 
 from docs_src.dataclasses.tutorial002 import app
 
 client = TestClient(app)
 
-openapi_schema = {
+
+def test_get_item():
+    response = client.get("/items/next")
+    assert response.status_code == 200
+    assert response.json() == {
+        "name": "Island In The Moon",
+        "price": 12.99,
+        "description": "A place to be be playin' and havin' fun",
+        "tags": ["breater"],
+        "tax": None,
+    }
+
+
+def test_openapi_schema():
+    response = client.get("/openapi.json")
+    assert response.status_code == 200
+    data = response.json()
+    assert data == {
         "openapi": "3.0.2",
         "info": {"title": "FastAPI", "version": "0.1.0"},
         "paths": {
@@ -48,32 +63,3 @@ openapi_schema = {
             }
         },
     }
-
-
-def test_openapi_schema():
-    response = client.get("/openapi.json")
-    assert response.status_code == 200
-    # TODO: remove this once Pydantic 1.9 is released
-    # Ref: https://github.com/pydantic/pydantic/pull/2557
-    data = response.json()
-    alternative_data1 = deepcopy(data)
-    alternative_data2 = deepcopy(data)
-    alternative_data1["components"]["schemas"]["Item"]["required"] = ["name", "price"]
-    alternative_data2["components"]["schemas"]["Item"]["required"] = [
-        "name",
-        "price",
-        "tags",
-    ]
-    assert alternative_data1 == openapi_schema or alternative_data2 == openapi_schema
-
-
-def test_get_item():
-    response = client.get("/items/next")
-    assert response.status_code == 200
-    assert response.json() == {
-        "name": "Island In The Moon",
-        "price": 12.99,
-        "description": "A place to be be playin' and havin' fun",
-        "tags": ["breater"],
-        "tax": None,
-    }