Differential D2891

tests: Specify status when calling origin_visit_update
ClosedPublic
Actions

Authored by ardumont on Mar 26 2020, 4:51 PM.

Details

Reviewers

vlorentz

Group Reviewers

Reviewers

Commits

rDWAPPS97c4e2438eea: tests: Specify status when calling origin_visit_update

Summary

Preparatory work for T2310

Diff Detail

Repository

rDWAPPS Web applications

Branch

master

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 11371
Build 17212: Cypress tests for swh-web diffs	Jenkins
Build 17211: tox-on-jenkins	Jenkins
Build 17210: arc lint + arc unit

Event Timeline

ardumont created this revision.Mar 26 2020, 4:51 PM

Herald added a reviewer: Reviewers. · View Herald TranscriptMar 26 2020, 4:51 PM

vlorentz accepted this revision.Mar 26 2020, 4:54 PM

vlorentz mentioned this in T2310: Make origin visits immutable.

This revision is now accepted and ready to land.Mar 26 2020, 4:54 PM

Note: this is needed because we will make this argument mandatory

Build is green
See https://jenkins.softwareheritage.org/job/DWAPPS/job/tox/1035/ for more details.

Build is green
See https://jenkins.softwareheritage.org/job/DWAPPS/job/cypress-diff/645/ for more details.

Harbormaster completed remote builds in B11371: Diff 10290.Mar 26 2020, 5:01 PM

Rebase on latest master

This revision was landed with ongoing or failed builds.Mar 26 2020, 5:08 PM

Closed by commit rDWAPPS97c4e2438eea: tests: Specify status when calling origin_visit_update (authored by ardumont). · Explain Why

This revision was automatically updated to reflect the committed changes.

ardumont added a commit: rDWAPPS97c4e2438eea: tests: Specify status when calling origin_visit_update.

Harbormaster failed remote builds in B11373: Diff 10292!Mar 26 2020, 5:09 PM

Build has FAILED

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

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DWAPPS/job/cypress-diff/646/
See console output for more information: https://jenkins.softwareheritage.org/job/DWAPPS/job/cypress-diff/646/console

ardumont mentioned this in D2893: storage*: Update origin_visit_update to make status parameter mandatory.Mar 26 2020, 5:15 PM

Revision Contents
Changeset List

Path

Size

swh/

web/

tests/

api/

views/

test_origin.py

5 lines

data.py

1 line

Diff 10290

View Options

swh/web/tests/api/views/test_origin.py