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 2962
Build 3779: tox-on-jenkinsJenkins
Build 3778: arc lint + arc unit

Event Timeline

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

Should be decorated with @pytest.mark.fs

96

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.