Differential D1382

Fix order of results of person_get.
ClosedPublic
Actions

Authored by vlorentz on Apr 9 2019, 4:42 PM.

Details

Reviewers

olasd

Group Reviewers

Reviewers

Commits

rDSTOC70c43bae5fca: Fix order of results of person_get.
rDSTO70c43bae5fca: Fix order of results of person_get.

Summary

The test only passed because we are "lucky" that the entries
id is in the right order.

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

vlorentz created this revision.Apr 9 2019, 4:42 PM

Herald added a reviewer: Reviewers. · View Herald TranscriptApr 9 2019, 4:42 PM

vlorentz added a child revision: D1378: Fix person name clashes in revision_add and release_add..Apr 9 2019, 5:01 PM

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

Harbormaster completed remote builds in B5244: Diff 4436.Apr 9 2019, 5:02 PM

vlorentz added a child revision: D1380: Rewrite skipped_content_missing to not use a temporary table..Apr 9 2019, 5:15 PM

olasd accepted this revision.Apr 9 2019, 5:30 PM

This revision is now accepted and ready to land.Apr 9 2019, 5:30 PM

rebase

This revision was landed with ongoing or failed builds.Apr 9 2019, 5:31 PM

Closed by commit rDSTO70c43bae5fca: Fix order of results of person_get. (authored by vlorentz). · Explain Why

This revision was automatically updated to reflect the committed changes.

Harbormaster failed remote builds in B5264: Diff 4456!Apr 9 2019, 5:38 PM

Build has FAILED

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

Revision Contents
Changeset List

Path

Size

swh/

storage/

db.py

10 lines

tests/

test_storage.py

37 lines

Diff 4457

View Options

swh/storage/db.py