Page MenuHomeSoftware Heritage

Cookers tests
ClosedPublic

Authored by seirl on Jul 20 2017, 4:22 PM.

Details

Reviewers
zack
Group Reviewers
Reviewers

Diff Detail

Repository
rDVAU Software Heritage Vault
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 961
Build 1276: arc lint + arc unit

Event Timeline

Looks good to me. Maybe some simple docstrings could be added detailing the purpose of the TestRepo class and explaining the goal of each test case.

Looks good to me. Maybe some simple docstrings could be added detailing the purpose of the TestRepo class and explaining the goal of each test case.

Docstrings in test cases show up when you do python -m nose -sv, and I think the test names are explicit enough to have those names indicate what the current test is instead of a docstring.

This revision is now accepted and ready to land.Sep 13 2017, 11:09 AM