Page MenuHomeSoftware Heritage

Add test for date parsing of origin_visit_add.
ClosedPublic

Authored by vlorentz on Wed, Jun 5, 11:35 AM.

Details

Reviewers
douardda
Group Reviewers
Reviewers
Summary

Related: D1537

Diff Detail

Repository
rDSTO Storage manager
Branch
origin_visit_add_parse_date
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 6170
Build 8505: tox-on-jenkinsJenkins
Build 8504: arc lint + arc unit

Event Timeline

vlorentz created this revision.Wed, Jun 5, 11:35 AM
douardda requested changes to this revision.Wed, Jun 12, 11:54 AM
douardda added a subscriber: douardda.
douardda added inline comments.
swh/storage/tests/test_storage.py
1572

as I already said in another diff (IIRC), I'd like to see the tested date written as a literal string here.

TBH I do not like that much using these self.date_visit* member variables. There is no gain in using these rather than literals and it obfuscate (slightly) the code of these tests (which generally is tedious enough as is).

This revision now requires changes to proceed.Wed, Jun 12, 11:54 AM
vlorentz updated this revision to Diff 5197.Wed, Jun 12, 3:29 PM
  • rebase
  • use a literal string
vlorentz marked an inline comment as done.Wed, Jun 12, 3:29 PM
douardda accepted this revision.Thu, Jun 13, 2:02 PM
This revision is now accepted and ready to land.Thu, Jun 13, 2:02 PM