Page MenuHomeSoftware Heritage

storage*: origin_visit_get_by -> Optional[OriginVisit]
ClosedPublic

Authored by ardumont on Mon, Jul 27, 2:19 PM.

Details

Summary

Impacts:

  • swh-web
$ grep -r 'origin_visit_get_by' */swh/** | grep -v "swh-storage"
swh-web/swh/web/common/service.py:    visit = storage.origin_visit_get_by(origin_url, visit_id)
swh-web/swh/web/tests/api/views/test_origin.py:        expected_visit = archive_data.origin_visit_get_by(new_origin.url, visit_id)
swh-web/swh/web/tests/api/views/test_origin.py:    expected_visit = archive_data.origin_visit_get_by(new_origin.url, visit_ids[1])
swh-web/swh/web/tests/common/test_origin_visits.py:    expected_visit = archive_data.origin_visit_get_by(new_origin.url, visits[1])
swh-web/swh/web/tests/common/test_origin_visits.py:    expected_visit = archive_data.origin_visit_get_by(new_origin.url, visits[2])
swh-web/swh/web/tests/common/test_origin_visits.py:    expected_visit = archive_data.origin_visit_get_by(new_origin.url, visits[-1])
swh-web/swh/web/tests/common/test_service.py:    expected_visit = dict(archive_data.origin_visit_get_by(new_origin.url, visit))
swh-web/swh/web/tests/conftest.py:    def origin_visit_get_by(self, origin_url, visit_id):
swh-web/swh/web/tests/conftest.py:        visit = self.storage.origin_visit_get_by(origin_url, visit_id)

Related to T645

Diff Detail

Repository
rDSTO Storage manager
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

ardumont created this revision.Mon, Jul 27, 2:19 PM
vlorentz accepted this revision.Mon, Jul 27, 2:26 PM
This revision is now accepted and ready to land.Mon, Jul 27, 2:26 PM

Build has FAILED

Patch application report for D3625 (id=12757)

Rebasing onto 2d51be955b...

First, rewinding head to replay your work on top of it...
Applying: storage*: origin_visit_get_by -> Optional[OriginVisit]
Changes applied before test
commit ba10edb3c77d09632de1625c020b7eb3d10a1e18
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Mon Jul 27 14:18:32 2020 +0200

    storage*: origin_visit_get_by -> Optional[OriginVisit]
    
    Related to T645

Link to build: https://jenkins.softwareheritage.org/job/DSTO/job/tests-on-diff/570/
See console output for more information: https://jenkins.softwareheritage.org/job/DSTO/job/tests-on-diff/570/console

Build has FAILED

Patch application report for D3625 (id=12758)

Rebasing onto 2d51be955b...

Current branch diff-target is up to date.
Changes applied before test
commit 5043f9056638d7d8490880687804c0cbbd9ec25c
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Mon Jul 27 14:18:32 2020 +0200

    storage*: origin_visit_get_by -> Optional[OriginVisit]
    
    Related to T645

Link to build: https://jenkins.softwareheritage.org/job/DSTO/job/tests-on-diff/571/
See console output for more information: https://jenkins.softwareheritage.org/job/DSTO/job/tests-on-diff/571/console

ardumont updated this revision to Diff 12759.Mon, Jul 27, 2:44 PM

Fix missing part (I forgot to run on the storage entirely, snapshot tests were unhappy)

Build is green

Patch application report for D3625 (id=12759)

Rebasing onto 2d51be955b...

Current branch diff-target is up to date.
Changes applied before test
commit 119d01e41620f5e8a86e9a3e62c55bc0c8c3f9bd
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Mon Jul 27 14:18:32 2020 +0200

    storage*: origin_visit_get_by -> Optional[OriginVisit]
    
    Related to T645

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

vlorentz accepted this revision.Mon, Jul 27, 2:58 PM