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
Branch
add-coverage-scenario
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 2964
Build 3783: tox-on-jenkinsJenkins
Build 3782: arc lint + arc unit

Event Timeline

vlorentz added inline comments.
swh/loader/tar/tests/test_loader.py
46

Should be decorated with @pytest.mark.fs

97

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!

  • 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.