-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Closed
Labels
p3-nice-to-haveIt should be done this or next sprintIt should be done this or next sprinttestingRelated to the tests and the testing infrastructureRelated to the tests and the testing infrastructure
Description
Currently our tests for different cache types usually deal with some specific use case, and assert intended behaviour for particular cache types.
E.g.
https://github.com/iterative/dvc/blob/f20818cd7382d119eb63cdc45608ab91635b6fc9/tests/test_checkout.py#L522
checks behavior only for "symlink" cache type.
We should consider writing consistent tests that would check appropriate behavior for each cache type. Parametrization might be a way to deal with that, but sometimes it will not be enough. (E.g. corrupting cache in hardlink and copy cache type would look differently, therefore it requires separate tests or some if's for cache type)
Metadata
Metadata
Assignees
Labels
p3-nice-to-haveIt should be done this or next sprintIt should be done this or next sprinttestingRelated to the tests and the testing infrastructureRelated to the tests and the testing infrastructure