Add support for marshmallow.fields.Number.as_string

Author: rockTA

AUTHORS.rst

@@ -75,3 +75,4 @@ Contributors (chronological)
 - Stefan van der Walt `<https://github.com/stefanv>`_
 - `<https://github.com/kasium>`_
 - Edwin Erdmanis `@vorticity <https://github.com/vorticity>`_
+- Timotheus Roeck `@rockTA <https://github.com/rockTA>`

src/apispec/ext/marshmallow/field_converter.py

@@ -187,6 +187,7 @@ def field2type_and_format(
         # hierarchy until we find something that does.
         for field_class in type(field).__mro__:
             if field_class in self.field_mapping:
+                field_class = get_diverging_field_class_if_required(field, field_class)
                 type_, fmt = self.field_mapping[field_class]
                 break
         else:
@@ -546,3 +547,20 @@ def make_min_max_attributes(validators, min_attr, max_attr) -> dict:
     if max_list:
         attributes[max_attr] = min(max_list)
     return attributes
+
+
+def get_diverging_field_class_if_required(
+    field: marshmallow.fields.Field, field_class: typing.Type
+) -> typing.Type:
+    """Return a field class that diverges from the origin class.
+
+    This is currently only required for Number fields, because some applications serialize decimal
+    numbers as strings, as binary representation of floats can't be precise. However, if more fields
+    would allow diverging serializer fields in the future, this function could be extended.
+    """
+    if (
+        issubclass(field_class, marshmallow.fields.Number)
+        and getattr(field, "as_string", False) is True
+    ):
+        return marshmallow.fields.String
+    return field_class

tests/test_ext_marshmallow_field.py

@@ -439,3 +439,12 @@ class _DesertSentinel:
     field.metadata[_DesertSentinel()] = "to be ignored"
     result = spec_fixture.openapi.field2property(field)
     assert result == {"description": "A description", "type": "boolean"}
+
+
+def test_number_as_string_is_converted_as_expected(spec_fixture):
+    field = fields.Number(
+        as_string=True,
+        metadata={"description": "A number field that is serialized as a string"},
+    )
+    result = spec_fixture.openapi.field2property(field)
+    assert result["type"] == "string"