Refactor DVCS loader tests to actually run the DVCSLoader store_data method
This will allow us to run integration tests on the behavior of that method. This also caught a tiny bug in the store_data method.
Test Plan
tox
Migrated from D3200 (view on Phabricator)