Differential D2886

svn: test_loader: Add missing assertions on visit status
ClosedPublic
Actions

Authored by ardumont on Mar 26 2020, 3:18 PM.

Details

Reviewers

vlorentz

Group Reviewers

Reviewers

Commits

rDLDSVNa43b1062b502: test_loader: Add missing assertions on visit status

Test Plan

tox

Diff Detail

Repository

rDLDSVN Subversion (SVN) loader

Branch

master

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 11356
Build 17188: tox-on-jenkins	Jenkins
Build 17187: arc lint + arc unit

Event Timeline

ardumont created this revision.Mar 26 2020, 3:18 PM

Herald added a reviewer: Reviewers. · View Herald TranscriptMar 26 2020, 3:18 PM

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

Harbormaster completed remote builds in B11356: Diff 10277.Mar 26 2020, 3:20 PM

vlorentz accepted this revision.Mar 26 2020, 3:21 PM

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

Closed by commit rDLDSVNa43b1062b502: test_loader: Add missing assertions on visit status (authored by ardumont). · Explain WhyMar 26 2020, 3:21 PM

This revision was automatically updated to reflect the committed changes.

ardumont added a commit: rDLDSVNa43b1062b502: test_loader: Add missing assertions on visit status.

vlorentz mentioned this in T2310: Make origin visits immutable.Mar 26 2020, 4:54 PM

Revision Contents
Changeset List

Path

Size

swh/

loader/

svn/

tests/

test_loader.py

14 lines

Diff 10277

View Options

swh/loader/svn/tests/test_loader.py

svn: test_loader: Add missing assertions on visit statusClosedPublicActions