Formats raise error for other types fix

Author: p1c2u

openapi_core/unmarshalling/schemas/unmarshallers.py

@@ -273,6 +273,9 @@ def unmarshal(self, value: Any) -> Any:
         schema_format = self.find_format(value)
         if schema_format is None:
             return typed
+        # ignore incompatible formats
+        if not isinstance(value, str):
+            return typed
         return self.formats_unmarshaller.unmarshal(schema_format, typed)
 
     def get_type_unmarshaller(

tests/integration/unmarshalling/test_unmarshallers.py

@@ -374,23 +374,17 @@ def test_string_uuid_invalid(self, unmarshallers_factory):
         assert len(exc_info.value.schema_errors) == 1
         assert f"is not a 'uuid'" in exc_info.value.schema_errors[0].message
 
-    @pytest.mark.xfail(
-        reason=(
-            "Formats raise error for other types. "
-            "See https://github.com/python-openapi/openapi-schema-validator/issues/66"
-        )
-    )
     @pytest.mark.parametrize(
         "type,format,value,expected",
         [
             ("string", "float", "test", "test"),
             ("string", "double", "test", "test"),
             ("string", "byte", "test", "test"),
-            ("integer", "date", "10", 10),
-            ("integer", "date-time", "10", 10),
+            ("integer", "date", 10, 10),
+            ("integer", "date-time", 10, 10),
             ("string", "int32", "test", "test"),
             ("string", "int64", "test", "test"),
-            ("integer", "password", "10", 10),
+            ("integer", "password", 10, 10),
         ],
     )
     def test_formats_ignored(
@@ -1728,7 +1722,8 @@ def test_basic_type_oas30_formats_invalid(
         reason=(
             "OAS 3.0 string type checker allows byte. "
             "See https://github.com/python-openapi/openapi-schema-validator/issues/64"
-        )
+        ),
+        strict=True,
     )
     def test_string_format_binary_invalid(self, unmarshallers_factory):
         schema = {
@@ -1748,7 +1743,8 @@ def test_string_format_binary_invalid(self, unmarshallers_factory):
         reason=(
             "Rraises TypeError not SchemaError. "
             "See ttps://github.com/python-openapi/openapi-schema-validator/issues/65"
-        )
+        ),
+        strict=True,
     )
     @pytest.mark.parametrize(
         "types,value",
@@ -1928,7 +1924,8 @@ def unmarshallers_factory(self):
         reason=(
             "OpenAPI 3.1 schema validator uses OpenAPI 3.0 format checker."
             "See https://github.com/python-openapi/openapi-core/issues/506"
-        )
+        ),
+        strict=True,
     )
     @pytest.mark.parametrize(
         "type,format",

tests/unit/templating/test_paths_finders.py

@@ -173,7 +173,10 @@ class BaseTestServerNotFound:
     def servers(self):
         return []
 
-    @pytest.mark.xfail(reason="returns default server")
+    @pytest.mark.xfail(
+        reason="returns default server",
+        strict=True,
+    )
     def test_raises(self, finder):
         method = "get"
         full_url = "http://petstore.swagger.io/resource"

tests/unit/unmarshalling/test_schema_unmarshallers.py

@@ -197,7 +197,8 @@ def custom_format_validator(value):
         reason=(
             "Not registered format raises FormatterNotFoundError"
             "See https://github.com/python-openapi/openapi-core/issues/515"
-        )
+        ),
+        strict=True,
     )
     def test_schema_format_validator_format_invalid(
         self, schema_unmarshaller_factory, unmarshaller_factory