Skip to content

Migrate dvcs loader tests code to pytest

They are hard to maintain (and even read) in their current state.

It's an entangled boil of inherited classes whose perimeter is conflated together (initialize svn repository, initialize loader with a given state, etc...).

It's complicated to the point of adding a new scenario about multiple visits loading scenario is not a simple task [1].

It'd be nice to disentangle this and also aligns with what other loaders are now using, pytest fixtures!


Migrated from T2481 (view on Phabricator)

Edited by Phabricator Migration user
To upload designs, you'll need to enable LFS and have an admin enable hashed storage. More information