Page MenuHomeSoftware Heritage

loader.tar: Cover the remote use cases in tests
ClosedPublic

Authored by ardumont on Dec 10 2018, 12:24 PM.

Diff Detail

Repository
rDLDTAR Tarball Loader
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

ardumont created this revision.Dec 10 2018, 12:24 PM
ardumont updated this revision to Diff 2538.Dec 10 2018, 12:29 PM

d/control: Fix tab typo

vlorentz accepted this revision.Dec 10 2018, 1:46 PM
vlorentz added inline comments.
swh/loader/tar/tests/test_loader.py
45

Should be decorated with @pytest.mark.fs

127

Remove that decorator from the test and move it to the class (all tests of the class use the FS)

This revision is now accepted and ready to land.Dec 10 2018, 1:46 PM

Should be decorated with @pytest.mark.fs
Remove that decorator from the test and move it to the class (all tests of the class use the FS)

Correct!

ardumont updated this revision to Diff 2539.Dec 10 2018, 1:52 PM
  • tests: Move fs marker to the upper class
  • tests: Remove blank spaces
  • tests: Rename test class
  • tests: Improve docstring declaration
This revision was automatically updated to reflect the committed changes.