Was about to tackle and #46 and was overwhelmed with how large snapshot.py had become. I took the dive and split it up into several files to give us some room to grow.
Beyond copying and pasting code I made the following changes:
Created the write_json_file within serializers/utils.py. This was to remove all json module things from the snapshot module. Updated tests for this change.
Added a test for trying to snapshot when not within a context. This was a TODO that never got done and took the opportunity.
Created module-level imports so that none of these changes are backwards incompatible.
Was about to tackle and #46 and was overwhelmed with how large
snapshot.py
had become. I took the dive and split it up into several files to give us some room to grow.Beyond copying and pasting code I made the following changes:
write_json_file
withinserializers/utils.py
. This was to remove alljson
module things from thesnapshot
module. Updated tests for this change.TODO
that never got done and took the opportunity.