recap-build/recap

`to_dict` loses StringType fields with no `bytes_` defined

criccomini opened this issue · 1 comments

criccomini commented

Adding this:

(
    StructType(
        fields=[
            StringType(),
        ]
    ),
    {
        "type": "struct",
        "fields": [
            {"type": "string"},
        ],
    },
    True,
    False,
),

To the test_to_dict param list causes the test to fail. I believe this should work since bytes_ are no longer a required field for string types.

criccomini commented

Closed by #340