Page MenuHomeSoftware Heritage

tests: Don't check the number of created 'person' objects.
ClosedPublic

Authored by vlorentz on Sep 17 2020, 2:51 PM.

Details

Summary

They are an implementation detail of storage backends; and get_stats() will
stop returning the 'person' count in the next version of swh-storage-core.

Additionally, D3976 changes the values of these counters.

Fixes [1]

[1] https://jenkins.softwareheritage.org/job/DLDG/job/tests/819/console

Test Plan

tox

Diff Detail

Repository
rDLDG Git loader
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 15239
Build 23490: Phabricator diff pipeline on jenkinsJenkins console · Jenkins
Build 23489: arc lint + arc unit

Event Timeline

Build has FAILED

Patch application report for D3978 (id=14012)

Rebasing onto 4d1cf9d96f...

First, rewinding head to replay your work on top of it...
Applying: tests: Don't check the number of created 'person' objects.
Changes applied before test
commit 7dfd1b81f9e9e3e44b2249ea45875d05d3f8cba3
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Sep 17 14:50:24 2020 +0200

    tests: Don't check the number of created 'person' objects.
    
    They are an implementation detail of storage backends; and get_stats() will
    stop returning the 'person' count in the next version of swh-storage-core.

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

This revision is now accepted and ready to land.Sep 17 2020, 2:54 PM
ardumont edited the test plan for this revision. (Show Details)
ardumont added a project: Git loader.

Build is green

Patch application report for D3978 (id=14012)

Rebasing onto d733c4c484...

First, rewinding head to replay your work on top of it...
Applying: tests: Don't check the number of created 'person' objects.
Changes applied before test
commit 1e30ec6bc3487d112593a33229d85439e3d546a4
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Sep 17 14:50:24 2020 +0200

    tests: Don't check the number of created 'person' objects.
    
    They are an implementation detail of storage backends; and get_stats() will
    stop returning the 'person' count in the next version of swh-storage-core.

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

This revision was landed with ongoing or failed builds.Sep 18 2020, 11:18 AM
This revision was automatically updated to reflect the committed changes.

Build is green

Patch application report for D3978 (id=14046)

Rebasing onto eafb0a64c4...

First, rewinding head to replay your work on top of it...
Fast-forwarded diff-target to base-revision-58-D3978.
Changes applied before test

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