Depends on D664.
- Group Reviewers
- Maniphest Tasks
- T1219: add tests to git loader
- rDLDGc7bf9e26f88c: Run git loader tests on BulkUpdater too.
Build has FAILED
Link to build: https://jenkins.softwareheritage.org/job/DLDG/job/tox/24/
See console output for more information: https://jenkins.softwareheritage.org/job/DLDG/job/tox/24/console
I mean as there is is no context in that diff, it's not apparent that this diff actually test anything.
In general, having context helps.
That's not helpful. There is no docstring in the DirGitLoaderTest class which explains anything.
I know it's boring but it's useful.
To have an example, check the loader-svn's tests  or the loader-mercurial's