Merge pull request #7 from kuimono/schema-additionalProperties-type-fix

Allow `Schema` type `additionalProperties` field to be boolean

Author: kuimono

CHANGELOG.md

@@ -1,5 +1,11 @@
 # Change Log
 
+## v1.2.2 - Unreleased
+
+### Fixed
+- [4](https://github.com/kuimono/openapi-schema-pydantic/issues/4) Allow `Schema` type `additionalProperties` field to be boolean
+
+
 ## v1.2.1 - 2021-09-10
 
 ### Added

openapi_schema_pydantic/v3/v3_1_0/schema.py

@@ -243,7 +243,7 @@ class Schema(BaseModel):
     object.
     """
 
-    additionalProperties: Optional[Union[Reference, "Schema"]] = None
+    additionalProperties: Optional[Union[Reference, "Schema", bool]] = None
     """
     The value of "additionalProperties" MUST be a valid JSON Schema.
 

tests/test_schema_class.py

@@ -1,5 +1,8 @@
 import logging
 
+from pydantic import BaseModel, Extra
+from pydantic.schema import schema
+
 from openapi_schema_pydantic import Schema, Reference
 
 
@@ -16,3 +19,30 @@ def test_schema():
     assert isinstance(schema.allOf, list)
     assert isinstance(schema.allOf[0], Reference)
     assert schema.allOf[0].ref == "#/definitions/TestType"
+
+
+def test_issue_4():
+    """https://github.com/kuimono/openapi-schema-pydantic/issues/4"""
+
+    class TestModel(BaseModel):
+        test_field: str
+
+        class Config:
+            extra = Extra.forbid
+
+    schema_definition = schema([TestModel])
+    assert schema_definition == {
+        "definitions": {
+            "TestModel": {
+                "title": "TestModel",
+                "type": "object",
+                "properties": {"test_field": {"title": "Test Field", "type": "string"}},
+                "required": ["test_field"],
+                "additionalProperties": False,
+            }
+        }
+    }
+
+    # allow "additionalProperties" to have boolean value
+    result = Schema.parse_obj(schema_definition["definitions"]["TestModel"])
+    assert result.additionalProperties is False