EliAndrewC/sideboard

Test for sorted dict encoding, presumably one that uses OrderedDict

Closed this issue · 1 comments

EliAndrewC commented

Our SortedDictJsonEncoder was broken for a long time, and our tests happened to pass because our dicts happened to serialize the same. We should use some OrderedDict instances to ensure that this does NOT happen by default, which will give us more confidence in the future.

This is pretty low priority, but worth listing as an issue since we specifically had this bite us.