Page MenuHomeSoftware Heritage

add tests to git loader
Open, HighPublic

Description

as per title: the Git loader is a key component of our in-production toolchain, we should really have tests around at least its basic ingestion logic

Event Timeline

zack created this task.Oct 1 2018, 7:40 PM
zack triaged this task as High priority.
vlorentz claimed this task.Oct 30 2018, 3:14 PM
vlorentz closed this task as Resolved.Nov 9 2018, 1:45 PM

Resolved by D622.

zack added a comment.Nov 9 2018, 1:48 PM

Resolved by D622.

Based on https://jenkins.softwareheritage.org/view/swh%20master/job/DLDG/job/tests/5/cobertura/loader_git/ I'd argue that tests has been added to the loader part of git-loader, but not to the updater part of it, which is what is used mostly in production (yes, it's an historical misnomer, and I really wish that the two could be merged, but that's unrelated to T1219…).

So either we keep this task open as WIP status, or we add a new dedicated one to keep track that the git "updater" needs tests.

vlorentz reopened this task as Open.Nov 9 2018, 5:49 PM