Page MenuHomeSoftware Heritage
Differential D2824

model: improve a bit the TimestampWithTimezone model
ClosedPublic
Actions

Authored by douardda on Mar 12 2020, 4:09 PM.

Details

Reviewers
vlorentz
ardumont
Group Reviewers
Reviewers
Commits
rDMOD10b069921e74: model: improve a bit the TimestampWithTimezone model
Summary
  • add a default value for negative_utc,
  • add a validator for negative_utc (can be True iff offset is 0),
  • add low-level tests for it.

Depends on D2823.

Diff Detail

Repository
rDMOD Data model
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

douardda created this revision.Mar 12 2020, 4:09 PM
Herald added a reviewer: Reviewers. · View Herald TranscriptMar 12 2020, 4:09 PM
Harbormaster failed remote builds in B11100: Diff 10029!Mar 12 2020, 4:09 PM
swh-public-ci added a comment.Mar 12 2020, 4:09 PM

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DMOD/job/tox/219/
See console output for more information: https://jenkins.softwareheritage.org/job/DMOD/job/tox/219/console

vlorentz accepted this revision.Mar 12 2020, 4:16 PM
vlorentz added a subscriber: vlorentz.
vlorentz added inline comments.
swh/model/model.py
164–172

I'd prefer to keep it explicit, so loader implementers have to at least think about it

183

rename the function to check_negative_utc

This revision is now accepted and ready to land.Mar 12 2020, 4:16 PM
vlorentz requested changes to this revision.Mar 12 2020, 4:16 PM
This revision now requires changes to proceed.Mar 12 2020, 4:16 PM
douardda added inline comments.Mar 12 2020, 4:19 PM
swh/model/model.py
164–172

not sure, this attribute really is a pita workaround for a single edge case... but you are probably right here.

douardda updated this revision to Diff 10031.Mar 12 2020, 5:04 PM

apply vlorentz' comments

Harbormaster failed remote builds in B11102: Diff 10031!Mar 12 2020, 5:04 PM
swh-public-ci added a comment.Mar 12 2020, 5:04 PM

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DMOD/job/tox/221/
See console output for more information: https://jenkins.softwareheritage.org/job/DMOD/job/tox/221/console

vlorentz accepted this revision.Mar 13 2020, 11:16 AM
This revision is now accepted and ready to land.Mar 13 2020, 11:16 AM
douardda updated this revision to Diff 10273.Mar 26 2020, 11:30 AM

rebase

Harbormaster failed remote builds in B11352: Diff 10273!Mar 26 2020, 11:30 AM
swh-public-ci added a comment.Mar 26 2020, 11:30 AM

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DMOD/job/tox/227/
See console output for more information: https://jenkins.softwareheritage.org/job/DMOD/job/tox/227/console

douardda added a child revision: D2884: model: fix SkippedContent origin to be a str.Mar 26 2020, 11:32 AM
douardda added a child revision: D2905: hypothesis: split hypothesis strategies as a dict + entity instance.Mar 27 2020, 3:04 PM
douardda removed a child revision: D2884: model: fix SkippedContent origin to be a str.Mar 27 2020, 3:06 PM
ardumont accepted this revision.Mar 31 2020, 9:56 AM
douardda updated this revision to Diff 10442.Apr 1 2020, 10:59 AM

rebas

swh-public-ci added a comment.Apr 1 2020, 11:00 AM

Build is green

Patch application report for D2824 (id=10442)

Could not rebase; Attempt merge onto e9a4c7519e...

Updating e9a4c75..10b0699
Fast-forward
 mypy.ini                           |   4 +-
 requirements.txt                   |   1 +
 swh/model/hypothesis_strategies.py |  20 ++-
 swh/model/model.py                 | 324 ++++++++++++++++++++++++++-----------
 swh/model/tests/test_model.py      | 121 +++++++++++++-
 5 files changed, 369 insertions(+), 101 deletions(-)
Changes applied before test
commit 10b069921e74f0d0411fb105349471e0f9a79f29
Author: David Douard <david.douard@sdfa3.org>
Date:   Thu Mar 12 16:01:55 2020 +0100

    model: improve a bit the TimestampWithTimezone model
    
    - add a validator for negative_utc (can be True iff offset is 0),
    - update the timestamps_with_timezone hypothesis strategy,
    - add low-level tests for it.

commit ac9d4c84a115c6c8d3aea04ed55635e8e743ea1d
Author: David Douard <david.douard@sdfa3.org>
Date:   Thu Mar 12 14:27:23 2020 +0100

    tests: add low level tests for the Timestamp model entity

commit 85ca7d7848008951f2e26c55c1c72ed9fa92cefb
Author: David Douard <david.douard@sdfa3.org>
Date:   Fri Mar 20 12:59:56 2020 +0100

    model: use attrs_static to enforce type validation of model objects
    
    This ensures all instanciated model entities have valid types for attributes.
    
    Related to T2308.

See https://jenkins.softwareheritage.org/job/DMOD/job/tests-on-diff/20/ for more details.

Harbormaster completed remote builds in B11525: Diff 10442.Apr 1 2020, 11:00 AM
Closed by commit rDMOD10b069921e74: model: improve a bit the TimestampWithTimezone model (authored by douardda). · Explain WhyApr 1 2020, 2:54 PM
This revision was automatically updated to reflect the committed changes.
douardda added a commit: rDMOD10b069921e74: model: improve a bit the TimestampWithTimezone model.

Revision Contents
Changeset List

PathSize
swh/
model/
20 lines
5 lines
tests/
92 lines

Diff 10458

View Options

swh/model/hypothesis_strategies.py

Loading...
View Options

swh/model/model.py

Loading...
View Options

swh/model/tests/test_model.py

Loading...
About · Developer information · Legal