Skip to content

Migrate loader tests to use pytest

Antoine R. Dumont requested to merge generated-differential-D3428-source into generated-differential-D3428-target

This simplifies the base classes indirection introduced to make the tests run.

It keeps the unittest scaffolding to allow declaring tests once and reuse amongst the different loader instances (GitLoader, GitLoaderFromDisk, GitLoaderFromArchive).

This uses the pg-storage and no longer the in-memory storage.

The code coverage increased from 82% to 85%.

This duplicates a bit helper function from the loader-core (equivalent code exists in loader-svn). I'll attend to this to remove that duplication soon (the diff could land without ;)

Related to #2482 (closed)

Test Plan

tox

Migrated from D3428 (view on Phabricator)

Merge request reports