Problem
Through #1092, much of the code was unraveled and decoupled.
However, some of the intermediary functions in mid-level modules as well as their tests, were left in. We need to remove that code, unnest the modules and rewrite any tests that aren't redundant to call and test the higher level functions in the Sync module itself.
Subtasks
Notes
This PR might be large in scope, so as an alternative, It may be prudent to consider just the "identification" step part of this PR, and instead of doing the actual coding work here, create issues for each identified function/untested area.
Problem
Through #1092, much of the code was unraveled and decoupled.
However, some of the intermediary functions in mid-level modules as well as their tests, were left in. We need to remove that code, unnest the modules and rewrite any tests that aren't redundant to call and test the higher level functions in the
Syncmodule itself.Subtasks
Syncmodule directly, remove those testsSyncmoduleNotes
This PR might be large in scope, so as an alternative, It may be prudent to consider just the "identification" step part of this PR, and instead of doing the actual coding work here, create issues for each identified function/untested area.