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(

openapi_core/unmarshalling/schemas/util.py

@@ -17,8 +17,8 @@ def format_uuid(value: Any) -> UUID:
     return UUID(value)
 
 
-def format_byte(value: str, encoding: str = "utf8") -> str:
-    return str(b64decode(value), encoding)
+def format_byte(value: str) -> bytes:
+    return b64decode(value)
 
 
 def format_number(value: str) -> Union[int, float]:

tests/integration/unmarshalling/test_unmarshallers.py

@@ -240,6 +240,30 @@ def test_basic_type_formats(
 
         assert result == unmarshalled
 
+    @pytest.mark.parametrize(
+        "type,format,value",
+        [
+            ("string", "float", "test"),
+            ("string", "double", "test"),
+            ("number", "date", 3),
+            ("number", "date-time", 3),
+            ("number", "uuid", 3),
+        ],
+    )
+    def test_basic_type_formats_ignored(
+        self, unmarshallers_factory, type, format, value
+    ):
+        schema = {
+            "type": type,
+            "format": format,
+        }
+        spec = Spec.from_dict(schema, validator=None)
+        unmarshaller = unmarshallers_factory.create(spec)
+
+        result = unmarshaller.unmarshal(value)
+
+        assert result == value
+
     @pytest.mark.parametrize(
         "type,format,value",
         [
@@ -268,7 +292,8 @@ def test_basic_type_formats_invalid(
     @pytest.mark.parametrize(
         "value,expected",
         [
-            ("dGVzdA==", "test"),
+            ("dGVzdA==", b"test"),
+            ("test", b"\xb5\xeb-"),
         ],
     )
     def test_string_byte(self, unmarshallers_factory, value, expected):
@@ -374,23 +399,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", "byte", 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(
@@ -1679,7 +1698,7 @@ def test_not_nullable(self, unmarshallers_factory, type):
     @pytest.mark.parametrize(
         "type,format,value,unmarshalled",
         [
-            ("string", "byte", "dGVzdA==", "test"),
+            ("string", "byte", "dGVzdA==", b"test"),
             ("string", "binary", b"test", b"test"),
         ],
     )
@@ -1728,7 +1747,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 +1768,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 +1949,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