amazon-ion / ion-python

A Python implementation of Amazon Ion.
https://amazon-ion.github.io/ion-docs/
Apache License 2.0
253 stars 50 forks source link

Build Std Dict without Lists #331

Closed rmarrowstone closed 6 months ago

rmarrowstone commented 6 months ago

The previous STRUCT_AS_STD_DICT commit still builds a "hashmap of lists" structure for "std" dicts, which was not intended. It was a failure to get my WIP/POC code into that PR, which this change remedies.

I added a new test to actually verify the behavior. I kept but did not add to the existing test_value_model_flags because that has a role in covering the breadth of the IonPyValueFlags and I didn't want to overcomplicate that for this Struct specific behavior.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.