Page MenuHomeSoftware Heritage

add tests to git loader
Closed, MigratedEdits Locked


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 triaged this task as High priority.Oct 1 2018, 7:40 PM
zack created this task.

Resolved by D622.

Based on 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.

ardumont changed the task status from Open to Work in Progress.Dec 17 2018, 12:01 PM
ardumont claimed this task.
ardumont added a project: Sprint 2018 12.
ardumont moved this task from Backlog to in progress on the Sprint 2018 12 board.
ardumont added a subscriber: vlorentz.
ardumont raised the priority of this task from High to Needs Triage.Dec 17 2018, 1:56 PM
ardumont moved this task from in progress to done on the Sprint 2018 12 board.
ardumont moved this task from done to deployed on the Sprint 2018 12 board.