- Download
- Download failures
Close T1431
Details
Details
- Reviewers
vlorentz - Group Reviewers
Reviewers - Maniphest Tasks
- T1431: Add tests for the Tar loader.
- Commits
- rDLDTAR33fc89ac2bc9: tests: Improve docstring declaration
rDLDTARc375e9f79492: tests: Rename test class
rDLDTAR2fea4e0a4dc7: test_loader: Add download failure scenario
rDLDTARa0eb9791dbf0: tests: Remove blank spaces
rDLDTAR1e5b9cb3ce73: tests: Move fs marker to the upper class
rDLDTARaabf1010cf3e: tar.loader: Reuse swh.model.hashutil.HASH_BLOCK_SIZE
rDLDTARc3577a11cc67: test_loader: Refactor tests
rDLDTAR03df15fba7d7: loader.tar: Cover the remote use case in test
tox
Diff Detail
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-jenkins Jenkins Build 3778: arc lint + arc unit
Event Timeline
Comment Actions
Build is green
See https://jenkins.softwareheritage.org/job/DLDTAR/job/tox/61/ for more details.
Comment Actions
Build is green
See https://jenkins.softwareheritage.org/job/DLDTAR/job/tox/62/ for more details.
Comment Actions
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!
Comment Actions
- tests: Move fs marker to the upper class
- tests: Remove blank spaces
- tests: Rename test class
- tests: Improve docstring declaration
Comment Actions
Build is green
See https://jenkins.softwareheritage.org/job/DLDTAR/job/tox/63/ for more details.