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.
EliAndrewC
commented
8 years ago
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.