Page MenuHomeSoftware Heritage

Make sure loading a svn repo properly sets the snapshot id in the visit
ClosedPublic

Authored by olasd on Feb 19 2020, 7:31 PM.

Details

Summary

Since the refactoring of swh.loader.core 0.0.74, so called BufferedLoaders don't
set the snapshot in the visit automatically. This loader would load data
properly, but fail to record the loaded snapshot into the visit.

Test Plan

running the new tests without the loader change would fail

Diff Detail

Repository
rDLDSVN Subversion (SVN) loader
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 10703
Build 16054: tox-on-jenkinsJenkins
Build 16053: arc lint + arc unit

Event Timeline

anlambert added a subscriber: anlambert.

Apart a docstring that needs to be updated, looks good !

swh/loader/svn/loader.py
574–577

Docstring should be updated to indicate the generated snapshot is now returned

This revision is now accepted and ready to land.Feb 19 2020, 7:49 PM
This revision was landed with ongoing or failed builds.Feb 19 2020, 7:59 PM
This revision was automatically updated to reflect the committed changes.