client: tests add missing 'meta' information
ClosedPublic
Actions

Authored by haltode on Aug 19 2019, 3:21 PM.

Details

Reviewers

zack
seirl

Commits

rDGRPHb1fdd018fd5a: client: tests add missing 'meta' information

Diff Detail

Repository

rDGRPH Compressed graph representation

Branch

client-tests-missing-meta

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 7299
Build 10341: tox-on-jenkins	Jenkins
Build 10340: arc lint + arc unit

Event Timeline

haltode created this revision.Aug 19 2019, 3:21 PM

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

Harbormaster completed remote builds in B7299: Diff 6263.Aug 19 2019, 3:23 PM

Rename assert_endpoint_result to assert_endpoint_output.

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

Harbormaster completed remote builds in B7300: Diff 6264.Aug 19 2019, 3:27 PM

The "Build is green" review.

This revision is now accepted and ready to land.Aug 19 2019, 3:36 PM

Closed by commit rDGRPHb1fdd018fd5a: client: tests add missing 'meta' information (authored by haltode). · Explain WhyAug 19 2019, 3:37 PM

This revision was automatically updated to reflect the committed changes.

I'd use self.assert_endpoint_result instead of ClassName.assert_endpoint_result but apart from that the refactoring looks sane to me.

Revision Contents
Changeset List

Path

Size

swh/

graph/

tests/

test_api_client.py

126 lines

Diff 6263

View Options

swh/graph/tests/test_api_client.py

client: tests add missing 'meta' informationClosedPublicActions